[U-Boot-Users] [PATCH u-boot-fdt] FDT command improvements.

[Wolfgang Denk]

In message <46465158.5070602 at gmail.com> you wrote:
>
> > The limit on the length of lines is 80 columns and THIS IS A HARD
> > LIMIT.
...
> Does :set ts=4 count?  It fits in 80 columns then. ;-)

No, this does not count, as it violates another part of the Coding
Style document:

"Tabs are 8 characters, and thus indentations are also 8 characters.
There are heretic movements that try to make indentations 4 (or even 2!)
characters deep, and that is akin to trying to define the value of PI to
be 3."

> The nesting is getting me in trouble.  I can fix some, but some would be 
> ugly to fix because I would need to split strings.  I could move the 
> value parsing into a subroutine, that would help.

"... The answer to that is that if you need
more than 3 levels of indentation, you're screwed anyway, and should fix
your program.

In short, 8-char indents make things easier to read, and have the added
benefit of warning you when you're nesting your functions too deep.
Heed that warning."

> Since the LIMIT IS HARD, I'll work it...

Thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,    CEO: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Speed of a tortoise breaking the sound barrier         = 1 Machturtle