made VS not crash on invalid dll reference (BadImageFormatException). especially for find all reference cache generation (#24452)

When adding the reference to System.IO.Pipes.AccessControl for the
compiler server to use on CoreCLR, I unified the pathway for the desktop
and CoreCLR server access control code. This means that
System.IO.Pipes.AccessControl needed to be added as a dependent DLL for
desktop too, but I forgot to do that. This change adds
System.IO.Pipes.AccessControl as a dependent DLL in all the places where
the build task is deployed.

Revert "moved waiter from diagnostics.dll to features.dll"

This reverts commit 823d9730.

Add UseCultureAttribute to help with culture-dependent unit tests

Disable NuGet package restore in Visual Studio for Roslyn.sln

'Use local function' suppress if used in expression tree.

Fixed VS2017 crash with :: operator

This PR fixes the CoreCLR compiler server on Windows and changes our bootstrap 
build to act as a test case. The build was previously failing during pipe ownership verification
because the required API was not available through reflection. This change
uses the System.IO.Pipes.AccessControl API to do pipe ownership verification
instead. However, System.IO.Pipes.AccessControl is operating system dependent 
and thus ends up in the runtimes subdirectory next to the build task.
Unfortunately, there is no mechanism for automatic assembly resolution
of plugin DLLs like MSBuild tasks, so we have to implement something
similar ourselves. This change hooks assembly resolution and, when we
see a well-known DLL request, we redirect it to the appropriate path.

Fixes #20900 and #24072

Added support for pattern matching syntax to smart variable completion provider

Suppress analyzer for properties with explicit interface implementation

Constrain availability of ref conditional expressions to language versions 7.2 and higher

Fixes:#24306

Update string text for keybindings reset.

* make sure OOP inherit same culture as VS otherwise culture sensitive resource such as xml doc comment will not work properly

* don't crash on expected issue

Fix a case where the stack spiller spills more than needed. (possibly resulting in false errors)

* dotnet/dev15.6.x:
  Fix symbol completion after 'in' (#24335)
  use PascalCase for const name
  Limit compiler server pipe name length (#24265)
  Test ConvertedType on LHS of deconstruction-assignment (#24158)
  remove unused usings
  use .editorconfig files
  address more comments
  address code review comments
  move newly added text into resource file
  add text and hyperlink to C# code style page

Add a link to Tools>Options>…>Code Style to link to EditorConfig documentation. Bug#23513

On MacOS pipes are implemented using Unix domain sockets. Unix domain
sockets must have a valid file path for their endpoint. The best place
is in the temp folder. Unfortunately, domain sockets on Unix also have a
very small path length limit of 104 characters. In addition, Mac temp
paths can be very long since they include randomly generated characters
from the OS. In total, this means that Roslyn has barely 50 characters
worth of space for its pipe identifiers.

This change cuts down on the bytes of the SHA256 hash used. This shouldn't
affect the security of the pipe, since the hash isn't used as a security boundary,
but should fit the pipe name into the length limitations.

Fixes #24137