Add full NuGet package restore to Test.cmd and fix Open samples to direct packages up to the solution packages directory. (changeset 1359108)

Port of changeset #1357667 to VB.

Bonus: Update the VB IL visualizer to match C# in a case where it was using ToString instead of ToDisplayString.

TODO: Part 3 - "this" in anonymous functions.

CR: chucks (changeset 1358824)

Remove stale items from packages.config files. None of the related projects were actually using these items. (changeset 1358488)

The size (19KB in the case of VB) is because of the initialization code for the NodeTypes static field.
While this change doesn't change the total size of generated code, it does move it out of the static constructor into its own method. The change is based on the observation that NodeTypes is used only within the GetSerializationData method. The result is transient - it's concatenated with other object sequences. So there's no need to keep this large object array in a static field either. (changeset 1358397)

During EnC, we serialize local variable signatures and keep them in LocalMap for the next time. However, we store the wrong value, so the next EnC would go wrong by using the invalid value from the previous EnC. (changeset 1358334)

When updating lambda return type inference to analyze ref returns, I noticed that BoundLambda.InferredReturnType was unnecessarily thread-safe: all callers of this method were either calling through an instance whose inferred return type had already been set or on an instance that had not been published to multiple threads. The same was true for LamdbaSymbol.SetInferredReturnType. I have refactored BoundLambda to set the inferred return type and related information at construction time (i.e. before the object is published to multiple threads) and established the invariant that InferredReturnType is only called on BoundLambda instances that have an inferred return type.
***NO_CI***
 (changeset 1358150)

Allow Extract Method, Introduce Variable on expressions in string interpolations. Fixes bugs 1064798 and 1064803. (changeset 1357978)

A boolean condition seems to have been inadvertantly flipped in CS 1317861 (portable workspace) which caused brace completion to not work after string literals. Fixing the conditional and enabling an unit test. (changeset 1357958)

Fix a crash while compiling an expression body with a lambda referencing a parameter. FIxes #313. (changeset 1357848)

Fix for bug 1066483: VS crashes when bringing up lightbulb with a compiler warning turned into an error.

Ensure that the correct warning level is returned for compiler warnings escalated to errors. (changeset 1357804)

Note this uses Tomas' proposed object id syntax "$[1-9][0-9]+" rather than the Dev12 EE syntax "ObjId[1-9][0-9]+".

If we decide to use the Dev12 syntax instead, the Expression Compiler will need to call DkmInspectionContext.GetClrObjectIdType() when binding to check if there is an object with the given id that may shadow another symbol in scope with the same name.

CR: acasey (changeset 1357770)

When the callstack tells us that we're in a state machine MoveNext method, we want to bind as though we're really in the iterator or async method that caused the state machine to be generated.  Parameters and locals are handled by special binders, but "this" and "base" won't work unless the binder has an appropriate ContainingType, which in turn depends on ContainingMemberOrLambda.

Unfortunately, the generated code (and the custom debug info) lacks sufficient information to map back to a single member.  All we get is the name of the member in the mangled name of the state machine type.  We apply a heuristic to choose an appropriate overload, if there are multiple, and accept that sometimes we will pick the wrong one.  We believe that picking the wrong one will be benign since the binder uses relatively few properties of the ContainingMemberOrLambda and the ones it does (mainly, ContainingType) are consistent across all overloads.

Bonus: merge ThisReferenceRewriter into CapturedVariableRewriter.  This saves us a pass over the bound tree and eliminates an intermediate state in which "this" has the wrong type (ThisReferenceRewriter used to break it and let CapturedVariableRewriter correct it).

TODO: Part 2 - "Me" in VB iterators and async.
TODO: Part 3 - "this" in anonymous functions.

CR: ChuckS, KevinH (changeset 1357667)

NullReferenceException in GetDiagnostics for optional parameter in security attribute (changeset 1357563)

The code for TryGetThisParameter handed the case where the method symbol represented a static method.  However, this should never be the case for an SynthesizedInstanceMethodSymbol, so instead we just assert that the method is non-static. (changeset 1357515)

***NO_CI***
 (changeset 1357023)

Remove all traces of Roslyn.Diagnostics.Analyzers.VisualBasic.BasicCodeAnalysis.dll. This assembly was recently deleted, but some references to it were missed. (changeset 1356370)

first, nobody in portable side is using Roslyn ETW event source.
second, we don't have a way to distinguish events from portable side and desktop side if two data are mixed.
third, nobody is enabling ETW in portable side.
last, ETW in portable side just doesnt work with our other tools.

so, I am moving this out to Workspace.Desktop.
 (changeset 1356327)

now, we need another layer called Desktop that override services in portable layer (changeset 1356294)

***NO_CI***
 (changeset 1356181)

Performance traces showed some lock contention (spinning) while acquiring Workspace's stateLock field.
Use interlocked operations to access Workspace's latestSolution field. This eliminates lock contention on the hot ".CurrentSolution" path.
Replace Workspace's ReaderWriterLockSlim stateLock with a NonReentrantLock which is much more lightweight. (changeset 1355891)

Delete BasicCodeAnalysisRoslynDiagnosticAnalyzers.vbproj. This project is now empty. (changeset 1355874)

 (changeset 1355776)

CallerLineNumber value was set to beginning of expression rather than method name (changeset 1355770)

The former assembly contains a single diagnostic, BasicUseSiteDiagnosticsCheckEnforcer, that only applies to the BasicCodeAnalysis (Portable) project. Currently, we do some MSBuild magic to prevent other assemblies from picking up that analyzer when they consume the Microsoft.Net.RoslynDiagnostics NuGet.

The right way to do this is to have one analyzer assembly for VB diagnostics, and have this particular diagnostic be off by default. BasicCodeAnalysis.vbproj gains a rule set file which turns it back on.

A subsequent change will actually delete the now empty BasicCodeAnalysisRoslynDiagnosticAnalyzers project, and another will move us forward to a newer version of Microsoft.Net.RoslynDiagnostics that includes this change. (changeset 1355761)

I think that at one point, it contained Win32 API calls that were not portable, but that is not the case now.  Moving it alongside FatalError simplifies project authoring (fewer linked files and aliases). (changeset 1355678)

This also updates Microsoft.Net.ToolsetCompiler to the corresponding version. (changeset 1355122)

Fix for bug 1046530: Should the strings in Microsoft.CodeAnalysis.DiagnosticFormatter.GetMessagePrefix be localized?

Localize the diagnostic severity in DiagnosticFormatter.GetMessagePrefix. (changeset 1355104)

FIx for bug 1038237: Incorrect warnings in Error List for #pragma warning disable'd code with Closed File Diagnostics enabled

The issue here is that root of the syntax trees recovered from temporary storage stream didn't have the ContainsDirective flag set, even when it had pragma directives. Hence, while computing diagnostics for closed files, we intermittently see diagnostics which have been suppressed by pragma directives appear in the error list.
Fix is to ensure that the stream based constructor for DirectiveTriviaSyntax node has this flag set, just like it's other constructors. (changeset 1355099)

Update assert in TypeUnification.CanUnify for substitutions from type parameter to type parameter (changeset 1354882)

Moved default host services logic from MSBuildWorkspace to new type DesktopMefHostServices type.

Changed CommandLineProject to make workspace parameter optional. You only need to specify workspace if want to override behavior of some of the services (beyond default).

Added WorkspaceServices.GetRequiredService and LanguageServices.GetRequiredService API that throws InvalidOperationException if service is not available.

Changed Workspace.AddDocument and Workspace.AddAdditionalDocument to take DocumentInfo instead of multiple parameters (for future proofing signature.)

Added ApplyChangesKind enum values for analyzers and additional documents

Changed Workspace TryApplyChanges to check for all unsupported changes before starting to apply.
Changed separate *apply* methods to default to calling equivalent OnXXX method instead of throwing.

Changed CustomWorkspace.CanApplyChanges to default to true for all kinds.
Added error messages for each ApplyChangesKind

Added missing WithXXX methods from DocumentInfo (changeset 1354869)

Hide events and property backing fields from the expression-level completion list (changeset 1354088)

Stream.ReadByte() allocates a byte[1] each time. This change uses a thread-local allocation to avoid that allocation. (changeset 1354061)

Fixes a couple options that were not being serialized to the registry and adds several OptionProviders to enumerate options in portable projects that are no longer exported, allowing us to properly reset all options to their default values during integration tests. (changeset 1353921)