Fix generic resolution of more complicated types with noinit
Not yet tested.
Merge branch 'master' into kill_test_module_sort_future
Change a warning to an error.  Previously, we had only warned if a function
specified its return type but did not return a value along one of its control
flow paths.  This will cause unpredictable behavior when return temps do not
initialize themselves by default (relying on other assignments to initialize).
Now, the behavior will be deterministic and will correctly not allow the
program to execute.
Update the previously known test case affected.

The test functions/hilde/retTypeNoRet.chpl displayed the warning behavior
described in the previous commit.  Since the test now fails to compile, I have
altered the .good file accordingly, and modified the test to only include the
minimum calls which will generate the error message.
This test previously generated a warning because the function did not return
a value when the return type was specified.  However, the point of the test was
a valgrind error that still exists when no return type is specified.  I decided
to remove the specified return type.  This program will now complete with only
the valgrind error remaining.
These five tests were properly checking that we detect when a function fails to
return a value, so it was appropriate to alter them for an error rather than a
warning with output.
Futurizing these tests.

As the functions in question currently stand, a value will always be returned,
but the compiler is under the impression this is not the case and so is
generating an error.  With proper analysis, these two cases should pass, so the
issue is now a future for both of them.
Merge pull request #39 from lydia-duncan/changeWarningToError

Change warning to error
A whitespace cleanup in preparation for fixing some issues with module use data in ModuleSymbol and BlockStmt
Merge pull request #37 from lydia-duncan/fixNoinitGenericResolutionPatch

Fix generic resolution of more complicated types with noinit
Using lowercase 'n' instead of 'N'
Merge pull request #41 from benharsh/fannk

Using lowercase 'n' instead of 'N'
Merge pull request #38 from hildeth/kill_test_module_sort_future

Remove .future from test_module_Sort-baseline in modules/hilde.
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
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