Re: coding standards

2018-03-06 Thread don fong
L A Walsh wrote: 1) I see no benefit in the use of extra braces. It diminishes > comprehension in this case. really? to me the braces make the code easier to read, in the context of the surrounding function. they clarify the intention. *if (THE VALUE IS OK) {* *USE THE VALUE* *

Re: coding standards

2018-03-06 Thread L A Walsh
don fong wrote: my patch (form (A)): -report_error (_("%s: parameter null or not set"), name); +{ + if (check_nullness) + report_error (_("%s: parameter null or not set"), name); + else + report_error (_("%s: parameter is not set"), name); +} the new

Re: coding standards

2018-03-06 Thread Greg Wooledge
On Tue, Mar 06, 2018 at 05:15:36PM +0800, Clark Wang wrote: > I don't know much about bash's source code so I cannot comment much. And > this kind of arguments are quite opinion based which are not simple yes/no > questions. That's precisely what a "coding standard" is -- an opinion. If one

Re: coding standards

2018-03-06 Thread Clark Wang
I don't know much about bash's source code so I cannot comment much. And this kind of arguments are quite opinion based which are not simple yes/no questions. And I believe one thing - the world is not perfect. :) -clark On Tue, Mar 6, 2018 at 8:26 AM, don fong wrote: > Clark,