On Fri, 2014-02-14 at 18:30 +0100, David Sterba wrote:
> On Thu, Feb 13, 2014 at 11:16:38AM +0800, Gui Hecheng wrote:
> > @@ -6460,6 +6460,7 @@ int cmd_check(int argc, char **argv)
> > !extent_buffer_uptodate(info->dev_root->node) ||
> > !extent_buffer_uptodate(info->chunk_root->node)) {
> > fprintf(stderr, "Critical roots corrupted, unable to fsck the FS\n");
> > + close_ctree(info->fs_root);
> > return -EIO;
>
> Can you please convert it to the 'goto + single return' pattern?
Thanks for your comments, David. I'll rework it soon.
Gui
> The other patches are ok, adding them to integration.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
