Removed trailing white space.
Indents occur after open braces and line-join indicators (///), but not elsewhere.
Removed unnecessary -noisily- prefixes.
Convention is not to include -cap program drop- in the ado-file. Do not execute the do-file itself. Rather, to try a new version, type -discard- or -clear ado-, etc. on the command line.
All open braces are now preceded by a space.
Removed unnecessary -quietly- prefixes.
`m' is an important local, and -forvalues m- erases it. Better I think to use an otherwise unused local name.
The previous version of this code would result in an error if strlen("`var1'X`var2'") > 31. The specific form `var1'X`var2' isn't used later in the form, so I think it is fine to replace ``var1'X`var2'' with `temp'.
`byrep' is always the same as `by', as `by' is a varlist.