Additional cleanup of scopeResolve in preparation for moving
module InitFunction from scopeResolve to docs

1) A number of "whitespace" changes aimed at fitting the majority
of the longer lines to fit the preferred 80 character limit

2) Simple refactoring of the code that converts use statements into
the internal ModuleSymbol::modUseList and BlockStmt::modUses

3) Several static functions are defined to take optional arguments;
on further inspection these functions have two calling patterns

   a) An entry call in which only the required arguments are used.
      The function implementation tests against the default values,
      determines it is an entry call, performs initialization and
      then proceeds, often using a recursive call.

   b) Recursive calls with all of the arguments specified.  These
      calls again rely on tests against the specified defaults to
      bypaass the initialization and perform the actual work

Split these functions into two functions; one to do initialization
and the other to handle the work.