Re: [U-Boot] Continuation line alignment

2011-11-08 Thread Gerlando Falauto
On 11/08/2011 12:58 AM, Scott Wood wrote: > I like aligning based on which level of nested parens the line break is > in (and removing unnecessary parens when precedence is obvious, to make > it easier to track the relevant ones): > > if ((day_of_week() % 2 == 0&& > (temperature()<

Re: [U-Boot] Continuation line alignment

2011-11-07 Thread Scott Wood
On 11/07/2011 05:32 PM, Gerlando Falauto wrote: > What bothers me more is, for instance, the condition under which my > smartphone will work correctly: > > if (((day_of_week() % 2 == 0) && >(temperature() < 14.4 || temperature() > 15.3)) >|| ((sky_color() == E_BLUE)

Re: [U-Boot] Continuation line alignment

2011-11-07 Thread Scott Wood
On 11/07/2011 05:32 PM, Gerlando Falauto wrote: > On 11/08/2011 12:02 AM, Scott Wood wrote: >> Documentation/CodingStyle calls for something like this: >> >> if (himport_ex(&env_htab, (char *)default_environment, >> sizeof(default_environment), '\0', >>

Re: [U-Boot] Continuation line alignment

2011-11-07 Thread Gerlando Falauto
On 11/08/2011 12:02 AM, Scott Wood wrote: > On 11/07/2011 04:05 PM, Wolfgang Denk wrote: >> Dear Gerlando Falauto, >> >> In message<4eb84859.6000...@keymile.com> you wrote: >>> >>> -int _do_env_set (int flag, int argc, char * const argv[]) >>> +int env_check_apply(const char *name, const char *old