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

[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