Please review: [nunc-stans] Remove extra curly brace

[Lukas Slebodnik]

ehlo,

I'm too lazy to follow all contribution steps for one liner :-)
So simple patch is attached.

LS

[Noriko Hosoi]

On 11/10/2015 01:29 AM, Lukas Slebodnik wrote:
>  ehlo,
>
>  I'm too lazy to follow all contribution steps for one liner :-)
>  So simple patch is attached.
>
>  LS
>
>
>  --
>  389-devel mailing list
>  389-devel(a)lists.fedoraproject.org
>  https://admin.fedoraproject.org/mailman/listinfo/389-devel 
Thank you for fixing the
syntax error, Lukas.

I hope you don't mind to modify the comment as follows.

commit cc1c55cf542fc6114e308390d91c5b094b3aa07a
Author: Lukas Slebodnik <lslebodn(a)redhat.com&gt;
Date:   Tue Nov 10 10:06:55 2015 -0800

     Ticket #48257 - Fix coverity issues - 08/24/2015

     Description: Remove extra curly brace

     Introduced by commit 4b9a54f7ba5f78b0402c0695998cca049ea3f095.

--noriko

[Lukas Slebodnik]

On (10/11/15 10:25), Noriko Hosoi wrote:
> On 11/10/2015 01:29 AM, Lukas Slebodnik wrote:
> >ehlo,
> >
> >I'm too lazy to follow all contribution steps for one liner :-)
> >So simple patch is attached.
> >
> >LS
> >
> >
> >--
> >389-devel mailing list
> >389-devel(a)lists.fedoraproject.org
> >https://admin.fedoraproject.org/mailman/listinfo/389-devel
> Thank you for fixing the syntax error, Lukas.
>
> I hope you don't mind to modify the comment as follows.
>
> commit cc1c55cf542fc6114e308390d91c5b094b3aa07a
> Author: Lukas Slebodnik <lslebodn(a)redhat.com&gt;
> Date:   Tue Nov 10 10:06:55 2015 -0800
>
>     Ticket #48257 - Fix coverity issues - 08/24/2015
>
>     Description: Remove extra curly brace
>
>     Introduced by commit 4b9a54f7ba5f78b0402c0695998cca049ea3f095.
>
> --noriko 
Updated patch is attached.

If there are any other issues in commit message.
Feel free to modify it.

Thank you very much for review.

LS

[Lukas Slebodnik]

On (10/11/15 21:29), Lukas Slebodnik wrote:
> On (10/11/15 10:25), Noriko Hosoi wrote:
> >On 11/10/2015 01:29 AM, Lukas Slebodnik wrote:
> >>ehlo,
> >>
> >>I'm too lazy to follow all contribution steps for one liner :-)
> >>So simple patch is attached.
> >>
> >>LS
> >>
> >>
> >>--
> >>389-devel mailing list
> >>389-devel(a)lists.fedoraproject.org
> >>https://admin.fedoraproject.org/mailman/listinfo/389-devel
> >Thank you for fixing the syntax error, Lukas.
> >
> >I hope you don't mind to modify the comment as follows.
> >
> >commit cc1c55cf542fc6114e308390d91c5b094b3aa07a
> >Author: Lukas Slebodnik <lslebodn(a)redhat.com&gt;
> >Date:   Tue Nov 10 10:06:55 2015 -0800
> >
> >    Ticket #48257 - Fix coverity issues - 08/24/2015
> >
> >    Description: Remove extra curly brace
> >
> >    Introduced by commit 4b9a54f7ba5f78b0402c0695998cca049ea3f095.
> >
> >--noriko
> Updated patch is attached.
>
> If there are any other issues in commit message.
> Feel free to modify it.
>
> Thank you very much for review.
>
> LS
>  
Attached is updated patch which also fixes indentation of return statement.

LS