Merge pull request #1109 from adbre/iss358

Override registry storage with MONO_CURRENTUSER_REGISTRY_PATH

[ppc] Bump instruction sizes after 51c454f6b03b468273bb93db2aa9b76e330a7e57.

Merge pull request #1014 from txdv/master

Expose full epoll_ctl functionality.

Merge pull request #1110 from tpokorra/FixImageListStreamerGeneratingValidResourcesForMSNet

Fixing ImageListStreamer generating winform resource files that work on MS.Net

Override registry storage with MONO_CURRENTUSER_REGISTRY_PATH

https://bugzilla.xamarin.com/show_bug.cgi?id=358

Merge pull request #1111 from akoeplinger/add-gtk-dllmap

Added dllmap for libgtk-x11 and libgdk-x11

Added dllmap for libgtk-x11 and libgdk-x11

Commit licensed under MIT/X11.

Expose full epoll_ctl functionality.

Now the user has access to all struct elements in EpollEvent when
using epoll_ctl. This is in particular very intersting when the
user wants to access the ptr field.

The user can use GCHandle.ToIntPtr to give the struct a GCHandle
and then handle the pointer later on when using epoll_wait.

ImageListStreamer generates winform resource files that work on Mono but fail on Microsoft.net
with exception System.InvalidOperationException: Loading of the ImageList did not succeed.
see bug https://bugzilla.xamarin.com/show_bug.cgi?id=3841 for detailed analysis

[ppc] Add missing `sync` instructions to atomic opcodes.

Note that lwarx and stwcx don't produce the necessary
load/store barrier that's needed to conform to the
`Interlocked` API.

Fixed another 8080 test port

I missed that one in f396d37c5493455f0b7fe726b408d74ba1c8fb5f.

Silence a warning in file-mmap-posix.c.

Merge pull request #1108 from akoeplinger/change-test-ports

[System.ServiceModel] Changed test port from 8080 to 30158

[System.ServiceModel] Changed test port from 8080 to 30158

Port 8080 is widely used and resulted in address conflicts on monojenkins, so we use a different port instead.

Commit licensed under MIT/X11.

[aot] Add support for overriding the alignment calculations done by the runtime during AOT. This is needed because the alignment of some types, notable i8/double can be different between the host and the target platform.

Disable generic-xdomain.2.exe on CI as it occasionally hangs.

Merge pull request #1106 from akoeplinger/fix-winforms-tests

[Managed.Windows.Forms] Fixed System.Resources tests that relied on DummyAssembly

Merge pull request #1107 from akoeplinger/fix-numericupdown

[Managed.Windows.Forms] Fix 'minimum' default value of NumericUpDown

[Managed.Windows.Forms] Fix 'minimum' default value of NumericUpDown

The 'minimum' default value of NumericUpDown was set to 0.0M, which caused the ToString() to format it as "0.0" instead of "0" as it does on .NET.
A similar change was done for the 'maximum' default value already in f6c8d48a3aed377b8a11e38c96c0d4c774664279.

Commit licensed under MIT/X11.

[mcs] Implements finally execution for await with try block exit.

[Managed.Windows.Forms] Fixed System.Resources tests that relied on DummyAssembly

Several System.Resources tests rely on a DummyAssembly.dll, which was removed in 016b0b5170fca72b977f952ee817fe06e32890a9 (possibly by accident?).
This commit adds the assembly back.

Commit licensed under MIT/X11.

Revert "[msbuild] fix some weird local result variable initialization issue."

This reverts commit 76b08f12ef69c180733a0cad99ea89b2a225ec51.

Merge pull request #1103 from nathanaeljones/patch-1

Fix AddCacheDependency signature to use params

[runtime] Add a workaround to file-mmap-posix.c to fix the android build.

[aot] Use g_print instead of printf in one place since the former works on android.

[arm] Fix alignment problems when AOTing from osx to arm linux.

Merge pull request #1104 from tpokorra/TarballIncludeMoreDllsInMonoLite

Tarball: include System.Security.dll and System.Configuration.dll in monolite directory

Tarball: include System.Security.dll and System.Configuration.dll in monolite directory
this is related to commit https://github.com/mono/mono/commit/993f4773638bcb36abf8b9660ee737d6445c5200

[bcl] Fix the mobile build.

Fix AddCacheDependency signature to use params

The .NET AddCacheDependency method signature is `public void AddCacheDependency(params CacheDependency[] dependencies);` Mono should mirror the signature (even if it isn't implemented) to prevent build errors. Conditional code, of course, will need to be used for an alternative mono path.

Merge pull request #1102 from tpokorra/TarballFixAvoidCDataTextReaderTests

Remove a reference to the file AvoidCDataTextReaderTests.cs so that tarballs can be built again

Remove a reference to the file AvoidCDataTextReaderTests.cs so that tarballs can be built again.
The file was removed in https://github.com/mono/mono/commit/d8d1b0be6baf6f8881871592b040d85be790926c

Merge pull request #1101 from jariq/master

Support both unencrypted PKCS#12 and PKCS#12 encrypted with an empty pas...

[mcs] Run new initializer await checks only in async context. Fixes #20614

Merge pull request #1099 from jsportaro/master

Added fix for Bug 11630

[system.data] Parse numeric expressions using invariant culture. Fixes #20582

[mcs] Reachability analysis should ignore local variables declarations. Fixes #20603

Merge pull request #1100 from BrzVlad/master

[x86] Fix initialization of stackalloc-ed memory in no pushes mode.

[runtime] Enable support for arm64-darwin in dolt.

Support both unencrypted PKCS#12 and PKCS#12 encrypted with an empty password in X509Certificate2 constructor

[x86] Align stackalloc-ed memory in no pushes mode.

[arm] Add an enum for gsharedvt argument marshalling conventions.

[x86] Fix initialization of stackalloc-ed memory in no pushes mode.

Monodoc: Fix for CDATA bug (https://bugzilla.xamarin.com/show_bug.cgi?id=17850). Reverts 3f361e1a263fc23680beab62eadde0cd3e4ac64d

Removed code that stripped CDATA statements (but not blocks) from input when showing a namespace page:

The function Ecma2Html.WrapStream(TextReader,Dictionary<String,String>) very explicitly returns a new `AvoidCDataTextReader` when the system is showing a namespace. The `AvoidCDataTextReader` strips the CDATA statement, but not it's internal contents. The intent, according to AvoidCDataTextReaderTests, is to allow class summaries of this form:

"<elements><summary>Addressbook APIs.</summary><remarks /><class name=""ABAddressBook"" fullname=""MonoTouch.AddressBook.ABAddressBook"" assembly=""monotouch""><summary><![CDATA[
      Provides access to the system Address Book.]]>
    </summary></class></elements>"

But that form is not desired.

The bug arose because the input to `WrapStream` is very large (it actually includes the entire *description* for each class and member in the namespace!). If any CDATA-block contained entities such as, e.g., < or >, the resulting XML would (obviously) be ill-formed:

"<xml>This should be valid <![CDATA[<T>]]> </xml>"

would throw because the <T> would be attempted to be parsed as XML. Obviously, samples that contain generics or lambda or XAML would cause this problem.

Although an XMLException was thrown, Macdoc swallows the exception in DocTools.GetHTML(). That is why this caused a silent failure on macdoc and was visible on Webdoc.

[System.Core] Add new tests for MemoryMappedFile.

[runtime] Fix share/private flags when doing file mmap.

[runtime] Rewrite MemoryMappedFile to support named mappings.

Added fix for Bug 11630

[mcs] Always store hidden catch variable when clause contains await

[corlib] Set AggregateException message text in all overloads

[mcs] Emit correct value type collection initialization when second initializer uses await

[mcs] Keep stack populated when array initializer contains await

[x86] Pop the vtype return addr from the stack in no pushes mode too.

Merge pull request #1098 from tpokorra/TarMissing_vasprintfh

tarball is missing file vasprintf.h

since gerror.c requires vasprintf.h, we need to add that file so that it becomes part of the tarball

[x86] Reduce some code duplication in the x86 call opcodes.

Disable push-less x86 code for now.

Breaks stuff: https://wrench.mono-project.com/Wrench/ViewLane.aspx?lane_id=10&host_id=8&revision_id=68534

Merge pull request #1097 from BrzVlad/master

[jit] Fix bug when using pushless code and reenable the feature.

[mcs] Handle catch filter with await-ed rethrow and exception filters

[jit] Fix bug when using pushless code and reenable the feature.

[mcs] Add support for await in finally clause

[Mono.Debugger.Soft] Handle cases where the instruction list is empty

[jit] Don't emit a write barrier when writing to the delegate->method field, its not needed.

[Http]: Fix a potential race condition in the WebConnectionGroup's connection list.

Remove the "ConnectionState" from the connection list before closing the connection,
this should avoid ServicePoint.SendRequest() ever crashing with a NullReferenceException
due to getting a null return value from WebConnectionGroup.GetConnection().

[mcs] Check for missing dependencies of container types. Fixes #20382

[mcs] Resolve event expression when testing left side expression. Fixes #20493

[mcs] Inflate nested pointer containers too.

[mcs] Flow analysis of binary expressions not using logical operators. Fixes #20515

Merge pull request #890 from xoofx/master

[Mono.Linker] Add support for <Module> with static method and pre-mark t...

[Mono.Linker] Add support for using the linker with assemblies referencing WPF

[Mono.Linker] Add support for <Module> with static method and pre-mark them if necessary

[sgen] Fix an infinite loop in find_previous_pointer_fragment due to remove CAS'ing the wrong value.

The fragment list has one invariant, the next pointer is tagged if the current fragment is been deleted,
which means the list head can never point to a tagged pointer.

If this invariant is broken, the code in find_previous_pointer_fragment will end up in an infinite loop.

The infinite loop happens in this simplified fragment:

try_again:
prev = &allocator->alloc_head;
1) cur = unmask (*prev);
while (1) {
2) if (*prev != cur)
goto try_again;

So assume allocator->alloc_head points to 0x121, which is tagged.

So execution goes like this:

1) cur is set to 0x120
2) *prev (value 0x121) is different than cur (value 0x120) so retry.

The above will loop forever as the fixup code never touches allocator->alloc_head.

Now that we understand the infinite loop, let's see how the remove code can trigger it.

Use use the following notation in the event sequence:

-> a non tagged pointer
-*> a tagged pointer
first CAS: InterlockedCompareExchangePointer ((volatile gpointer*)&frag->next, mask (next, 1), next)
second CAS: InterlockedCompareExchangePointer ((volatile gpointer*)prev_ptr, next, frag)

Assuming the following initial fragment list and two threads removing two separate fragments:

fragment list; -> A -> B -> C

thread 1 removes A
thread 2 removes B

thread 2 succeed the first CAS
-> A -> B -*> C

thread 1 succeed the first CAS
-> A -*> B -*> C

thread 2 fail the second CAS because of thread 1
-> A -*> B -*> C

thread 1 succeed the second CAS
-> B -*> C

thread 2 succeed the second CAS and set prev_ptr to next,
which at this point is a tagged pointer. Leaving the fragment list like this:
-*> C

After the last CAS, we end up with alloc_head with a tagged pointer, which will cause
the infinite loop we previously explained.

The fix is to unmask the value we store in the second CAS, as the objective is to link
two live fragments and not flag the previous fragment for deletion.

The exact same bug exists in mono-linked-list-set and was fixed there.

[Http]: Cosmetic, remove some dead code.

The is no reason to ever invoke the callback on the ThreadPool,
it would in fact only cause trouble.

Marek's commit db92449f was correct, I removed the commented out
code and the no longer used method.

[mcs] Implement rethrow support in awaited catch clause

[Http]: Fix digest authentication (bug #18799).

We did not correctly parse key-value pairs without quotes in the
challenge.

[System.Net.Http]: Always add "Content-Length" in HttpClientHandler.SendAsync().

This has been reported on the forums:
http://forums.xamarin.com/discussion/17770/length-required-error-in-http-post-since-latest-release

[Mono.Debugger.Soft] Improved ILInterpreter to eval properties that return a primitive constant

[windows] Return early in the vectored exception handler if the thread is not managed by the runtime.

[Http]: Only recycle ServicePoints from the idle timer (fixes #19823).

Thanks a lot for Martin Potter to find and report the race condition
which was causing the problem.

ServicePointManager.RecycleServicePoints() was a left-over from the
old times when we didn't have an idle-timer based approach.

[Mono.Debugger.Soft] minor code cleanup

Fix mono exception handler crashing on windows.

Here's the scenario that causes the crash:

Exception is fired, which invokes mono's vectored exception handler. It
is not an exception mono recognizes, so falls through to the default:
case in the exception handler. The problem is the return value has been
initialized to EXCEPTION_CONTINUE_EXECUTION. This causes SEH not to
process any other handlers, and just attempt to continue execution at
the point the exception was thrown.

The problem is much of windows internals will catch various exceptions
internally and handle them. So instead of the exception being handled,
we would just crash in the bowels of windows.

The solution is to set mono_win_chained_exception_needs_run to TRUE in
the default case - this is what the various exception handlers do if the
exception is not in JIT'd code. This causes the handler to return
EXCEPTION_CONTINUE_SEARCH, which causes SEH to search for the next
handler, and no crash is produced.

Both x86 and x64 had this bug, so I fixed it in both places.

Fix contributed by Steve Anichini.

Changed mono_win_chained_exception_needs_run to be a JIT TLS variable.

This fixes a potential race condition that might happen if two threads try to handle an exception at the same time.

[sys.xaml] fix TypeId and get broken WindowsBase tests pass.

Attempt to sanitize the `vasprintf` situation in eglib.

The problem is that on some platforms we call the function
without having provided a prototype for it. This breaks when
building with `-Werror`.

Now we declare it in the private `vasprintf.h` header when
the platform doesn't have a declaration in `stdio.h`. This
should also let us remove the hack in `glib.h` for MSVC.

This may or may not break the build on some platforms; I've
tried on those that I have access to with success. Let me
know if this breaks anything.

Add build system plumbing for building with -Werror.

Merge pull request #1094 from LRFalk01/master

Identities not being set in ClaimsPrincipal Ctor

Identities not being set in ClaimsPrincipal Ctor

Class' identities member is not set in constructor ClaimsPrincipal
(IEnumerable<ClaimsIdentity> identities)

Remove unnecessary MPH_INTERNAL applied to a function definition.

This should be applied to prototypes, and it already is for
this function (in mph.h).

Remove the architecture conditions for using `perf` in the log profiler.

We now use generic atomics which work on all platforms, so there's
no reason to keep this constraint here. This allows the `perf`-backed
profiler to be used on mips, ppc, s390x, etc.

Revert "Disable `perf` support in the profiler when running on Android (for now)."

This reverts commit 8e35b7d01e14371851a806dad6196b36d9163c91.

The fallback implementation (as used on other platforms) appears to hang
the runtime. Having a maybe-working `perf`-backed sample profiler is better
than a not-working sample profiler.

Revert "[x86] Reenable the no pushes code."

This reverts commit 5093efd55883e8a8a093fdb43d938d7b9dde7a4e.

Revert this as it still breaks the build.

[x86] Fix the build.

[x86] Reenable the no pushes code.

[sys.xaml] remove extraneous NET_2_1 and use __MOBILE__ to make it build for XA/XiOS.

[jit] Align the stack when using the pushless code on x86.

Merge pull request #1090 from BrianDurham/patch-2

Update X11Display.cs - change "Check you DISPLAY"

Merge pull request #1089 from BrianDurham/patch-1

Update Graphics.cs - change "Check you DISPLAY"

Merge pull request #1091 from BrianDurham/patch-3

Update XplatUIX11.cs - change "Check you DISPLAY"

Merge pull request #1092 from BrianDurham/patch-4

Update XplatUIX11GTK.cs - change "Check you DISPLAY"

Update XplatUIX11GTK.cs - change "Check you DISPLAY"

Update XplatUIX11GTK.cs - change "Check you DISPLAY" text to "Check your DISPLAY"