Hi Fabian,
On Wed, Mar 11, 2015 at 6:44 PM, Fabian Frederick <[email protected]> wrote:
> @@ -1002,13 +999,9 @@ static int clean_an_unclean_leb(struct ubifs_info *c,
>
> dbg_rcvry("LEB %d len %d", lnum, len);
>
> - if (len == 0) {
> + if (len == 0)
> /* Nothing to read, just unmap it */
> - err = ubifs_leb_unmap(c, lnum);
> - if (err)
> - return err;
> - return 0;
> - }
> + return ubifs_leb_unmap(c, lnum);
While technically the curly braces are no longer needed, I'd keep them
because of the comment line.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
