Re: staging: lustre: Delete unnecessary goto statements in six functions

From: SF Markus Elfring
Date: Tue Dec 15 2015 - 13:03:18 EST


> This is the original code:
Really …?
> result = baz();
> if (result)
> goto label;
>
> label:
> go on...

I do not see such a source code structure
at the six places I propose to clean-up.


> I don't find the test->goto label; label: use offensive,
> but if he does, I think keeping a blank line in place of
> the test->goto might be better.

I find this an interesting view on source code layout.
Are there any more opinions around such implementation details?

Regards,
Markus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/