When dynamically compiling script with a class with a nested class defined, the collection would be null.
Example:
class test
{
 class asd
 {
 }
}
[Mono.Debugger.Soft] Caching VirtualMachine threads, caching frames in ThreadMirror and fetching StackFrames in bulk for all threads
[Mono.Security] Added Encode to AuthorityKeyIdentifierExtension and SubjectKeyIdentifierExtension.
More C90 compatibility fixes

This silences the following gcc/clang warnings from MonoJenkins:
sgen-tarjan-bridge.c:395:2: warning: ISO C90 forbids mixed declarations and code
proflog.c:1942:2: warning: ISO C90 forbids mixed declarations and code
proflog.c:1999:3: warning: ISO C90 forbids mixed declarations and code
decode.c:2409:6: warning: ISO C90 forbids mixed declarations and code
decode.c:2433:6: warning: ISO C90 forbids mixed declarations and code

Moving the declarations to the beginning of the blocks is sufficient
to silence the warnings.

This change is released under the MIT license.
Fix non-prototype function declarations

MonoJenkins repeatedly complains about:
mono-counters.c:xxx:1: warning: function declaration isn't a prototype
[-Wstrict-prototypes]

This is caused by the missing declaration of the function arguments.

This change is released under the MIT license.
[eval] Add test for nested types
[mcs] Add cref handling of user operator documentation without parameters
[mcs] Don't report hiding warnings for compiler generated fields
Merge pull request #1033 from godFather89/master

When dynamically compiling script with a class with a nested class defin...
Merge pull request #1059 from ranma42/jenkins-warnings-cleanup

Cleanup of some MonoJenkins warnings
[mcs] Fix error reporting propagation for loop statements. Fixes #20034
[Mono.Security] Added AuthorityKeyIdentifierExtensionTest and SubjectKeyIdentifierExtensionTest.
[Microsoft.Build.Tasks] Add support for LogicalName in EmbeddedResources.
Fix a couple of compiler warnings.
[arm64] Fix another crash due to not using the right objc_msgSend signature.

Also simplify the code a little bit, there's no need to have separate
code paths for ARM64, since using the correct objc_msgSend signature
works on all architectures.
[mcs] Don't release field address copy before it's used. Fixes #20040
Use string.IsNullOrEmpty instead of .Length.
L_cuserid was removed from OpenBSD.

This change released under the MIT license with permission from the original
author Christian Weisgerber.
[System.ServiceModel.Web] Fix issue with types explicitly implementing IDictionary or IDictionary<,>

Issue: The previous code, with new tests [1], failed when the linker was
enabled.

That was caused because the code looks for the Keys and Item properties
getter using reflection. The linker handled that imperfectly (it left
some holes), in part because:

* System.Collections.Generic.IDictionary`2 does not implement
  System.Collections.IDictionary; and

* It preserved the explicit methods - not the implicit one that were
  used the the reflection-based code;

That later fact is also a bug (without the linker involvment) and solved
by this commit. A unit test was added to show the issue (NRE).

The patch also avoid the use of reflection for IDictionary since we can
simply typecast to get the same result (without the additional cost).
Fixed TLAB_ACCESS_INIT usages in MSVC by declaring the variables explicitly at the start of each block.