>There is good in the code:
>- Liberal use of white space. Too many people don't use blank lines, which makes the code hard to read. Using a space after the comma in a parameter list and around the equal sign in the assignment statements are other important ones.
>- Using the full function/command name rather than the first four letters

mostly agree - but not sure "liberal" is the best word to describe my POV for blank lines.
I prefer having code blocks easily grokked separated from others - but prefer opening { on the same line,
as otherwise too much screen space is wasted without information passed as well.
Most coding is done with { alone on separate lines - giving half of those to group blocks IMO reads easiest.

Sometimes C or Java/C# is too spaced to follow even on large screen, forcing unneccassary scrolling.

YMMV

thomas