A whitespace cleanup in preparation for fixing some issues with module use data in ModuleSymbol and BlockStmt
Remove some dead code
Fix an error for ordering #includes
Fix minor alignment issue in ModuleSymbol
Switch pointer comparisons to use NULL rather than 0
Revert to class initializer-lists for ModuleSymbol and BlockStmt
See what the implications are for turning off qthreads affinity

[more qthreads testing stuff, discussed with sung]

There are some for-loop and while-loop tests that are worse on chap04 (multi
socket) but not on chap03 (single socket.) The current guess is that because
qthreads pins shepherds to cores, if a core switches to another pthread, the
shepherd (and it's workers) can't make progress, meaning that qthreads could
perform worse in single threaded cases because. If shepherds are not pinned to
cores, that should mean that if the OS swaps the shepherds thread, that
shepherd should be able to move to a different core and continue making
progress but on a different core.

However, in cases where qthreads is using all systems resources, it seems
likely that we would want affinity (shepherds pinned to cores) but we're not
really sure of the implications yet, hence trying this.

Also it's possible that since we switched to using the nemesis scheduler, we
want affinity off for that, and only leave it on for configurations not using
nemesis (numa, using sherwood.)
Merge pull request #43 from noakesmichael/mod-use-2

Preliminary "whitespace" cleanup in preparation for tackling a few simple issues with Module uses

Module use information is logically located in 4 places; use-statements in the AST, the modUses property of BlockStmt, the modUseList/modUseSet on ModuleSymbol.  These are being managed at multiple points in the code and there are some consistency issues right now.  I am going to fix a few of these issues to advance the interpreter.  This is a "whitespace cleanup" so that the code reviews for the functional changes will be easier.

compiled: clang on STD, gnu on STD/LLVM
tested:     paratest without --verify
reviewed: Elliot
Merge pull request #45 from ronawho/qthreads-testing

See what the implications are for turning off qthreads affinity
inital check-in of ref vars, round 2
Ref vars now work properly with PRIM_CREATE_REF