Call ImmutableList<T>.AddRange from ConvertAll and FindAll

Replace any remaining calls to ImmutableList<T>+Node.Add and
ImmutableList<T>+Builder.Add from within a loop to calls to the
equivalent AddRange method
Remove import of System.Linq, reference namespace explicitly.
Formalized coding style guidelines.

It is possible to capture most of the coding style guidelines in a corefx.vssettings VS 2013 settings file.

Examples of settings that were not captured: avoiding double free lines, or spurious blank spaces.
Fix race condition in metadata reader

See http://roslyn.codeplex.com/workitem/299

In the case where a PEReader was backed by a memory-mapped file,
concurrent calls to PEReader.GetMetadataReader were unreliable.
They  led to concurrent calls to MemoryMappedFile.CreateFromFile
on the  same FileStream, which is not safe.

The fix is to use the same streamGuard lock used elsewhere to
avoid concurrent use of the stream.
Fix spelling error in rarely used pre-release API

MetadataTokens.ExportedTypeHandle was misspelled with two o's.

This is of course a breaking change, but this API is still in
pre-release. Furthermore, there are no usages at all of this API
in any partner code that I can find and it is unlikely to be used
yet elsewhere. The API is in the Ecma335 sub-namespace for getting
handles from raw metadata tokens, but most consumers would find
ExportedTypes via the main token-agnostic API.
ImmutableArray<T>.Builder.Sort should not create Comparer instance if count<=1
Tweaking Sort(int index, int count, IComparer<T> comparer) to check for count instead of Count. Checking for count > 1 will ensure both of these
i. The no. of elements intended to be sorted is > 1
ii. The underlying array contains > 1 elements
Add clarifying comment (responding to PR feedback)
Merge pull request #189 from richamsft/master

ImmutableArray<T>.Builder.Sort should not create Comparer instance
1. GoodbyeHello() 2. Invert FEATURE_TRACING
Cleanup PAL comments.
Merge pull request #186 from nguerrera/mdreader-fixes

Fix race condition and spelling error in metadata reader
Merge pull request #192 from Priya91/dataflow

1. GoodbyeHello() 2. Invert FEATURE_TRACING
Merge pull request #190 from mj1856/master

Remove unused variables
Remove ImmutableArrayInterop and work around it

The public ImmutableArrayInterop class has been deemed too dangerous
to be out there for general use.

Remove it and implement a byte[] only version internally in the
metadata reader using an unverifiable union.

Retain the documentation and add a warning to anyone thinking about
reusing the code to do likewise.
Merge pull request #196 from nguerrera/internalize-unsafe-interop

Remove ImmutableArrayInterop and work around it
Update buildtools reference to 1.0.15-prerelease
Enabling innerloop dataflow tests
Merge pull request #198 from FiveTimesTheFun/buildtools-15

Update buildtools reference to 1.0.15-prerelease
Merge pull request #197 from Priya91/enableddataflowtests

Enabling innerloop dataflow tests
Add thread-safety test for ImmutableArray`1.Add

Repro for issue found in #191