I agree with all that Glenn has said. He is very good at commenting and polishing up scripts and explaining them.

One point I don't like (and I fear that I'm alone with this one but I'll complain again anyways)

Not that it is wrong or that it doesn't do as it's supposed to but I don't like the use of a single $ for use as a variable.

$ = SetOption('Explicit', 'On')
$ = SetOption('NoVarsInStrings', 'On')
$ = SetOption('NoMacrosInStrings', 'On')
$ = SetOption('WrapAtEOL', 'On')


I find it very difficult to track down and modify single entries in a large script because often a coder will use that single $ either in script or UDF and that makes it difficult to do an automated Search & Replace and I have to confirm all changes.

In my opinion you shouldn't use a single $ and you shouldn't reuse the same var name over again if possible. Often the single $ is simply used for preventing screen output or holding a var that will be of no further use later on in the script and could easily be used again.

Most people don't do this type of change I suppose or they work differently than I do so it's not an issue for them.

Just my own personal preference I suppose.