[corlib] Fix tests to compile when Thread.Suspend/Resume is not supported.

[runtime] Fix compilation warning

Merge pull request #2693 from dellis1972/master

[msbuild] Getting build error "Error initializing task XmlPeek: Not registered task XmlPeek".

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

Removes the uses of LogXXXFromResources from XmlPeek because
we do not use string resources in xbuild.

[msbuild] Getting build error "Error initializing task XmlPeek: Not registered task XmlPeek".

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

Removes the uses of LogXXXFromResources from XmlPeek because
we do not use string resources in xbuild.

Merge pull request #2692 from vargaz/process-coding-conv

[runtime] Use mono coding conventions in process.c.

[runtime] Use mono coding conventions in process.c.

Merge pull request #2690 from steffen-kiess/json-operator-uint

Fix typo in JsonValue which broke the conversion to uint and add test case

Fix typo in JsonValue which broke the conversion to uint and add test case

Bring new IKVM

Merge pull request #2580 from ItsVeryWindy/master

Updates to WebRequestHandler

Merge pull request #2672 from Numpsy/sgenexception

[sgen] catch NotSupportedException from system.xml.serialization

Merge pull request #2689 from vargaz/socket-io-cleanup

[runtime] Clean up the socket-io.c file to use the mono coding conven…

[runtime] Clean up the socket-io.c file to use the mono coding conventions. Rename 'error' to 'werror' so 'error' can be used to refer to MonoError as everywhere else. NFC.

Merge pull request #2688 from vargaz/marshal-set-pending

[runtime] Use pending exceptions instead of mono_raise_exception () i…

…n Marshal icalls.

[runtime] Use pending exceptions instead of mono_raise_exception () in Marshal icalls.

[aot] Fix the usage of MonoError in init-llvm_method ().

Merge pull request #2678 from esdrubal/filestream

[corlib] Fixed FileStream Write after Read issue

Merge pull request #2685 from ludovic-henry/monoerror-mono_object_register_finalizer

[runtime] Use MonoError for mono_object_register_finalizer

Merge pull request #2671 from lambdageek/dev/monoerror-mono_runtime_class_init

[runtime] Use MonoError for mono_class_vtable_full and mono_runtime_class_init_full

Switch from a boolean `raise_on_error` arg to a MonoError outarg and let
callers decide what to do with the error.

Also mark mono_runtime_class_init external only.  Runtime should use
mono_runtime_class_init_full.

[runtime] Use MonoError for mono_class_vtable_full and mono_runtime_class_init_full.

Switch from a boolean raise_on_error arg to a MonoError outarg and let
callers decide what to do with the error.

Also mark mono_runtime_class_init external only.  Runtime should use
mono_runtime_class_init_full.

[scripts] Point scripts like nunit-console2/mono-service2 to the 4.5 directory

The 2.0 profile doesn't ship with Mono anymore. To avoid breaking existing scripts we just point to the 4.5 version instead.

[System.ServiceModel] Fix a hardcoded port in a test

This failed with an "address already in use" error on Jenkins yesterday.

[runtime] Use MonoError for mono_object_register_finalizer

Merge pull request #2679 from lambdageek/dev/monoerror-reflection

[reflection] MonoError cleanups

[runtime] Remove some declarations from marshal.h which are not used outside of marshal.c.

Merge pull request #2683 from mfilippov/master

Add generation WepApplication.targets for v12.0 and v14.0 xbuild.

Add generation WepApplication.targets for v12.0 and v14.0 xbuild.

Merge pull request #2681 from kumpera/pedump_fix

[pedump] Fix init crash on pedump when using --verify.

@marek-safar  here's your fix

@monojenkins merge

[aot] Move a mono_error_raise_exception () call higher up the call chain.

[pedump] Fix init crash on pedump when using --verify.

[sdb] Remove some mono_error_raise_exception () calls.

[jit]: Correctly restore callee saved registers in mono_arch_get_call_filter() on amd64.

(cherry picked from commit 8c1ffe78ccf68fab9acad53eebc01b5c84fc81ec)

[corlib] Fixed MonoIO.Write loop.

[corlib] Fixed FileStream Write after Read issue

FileStream was writing to the wrong position When a value longer than
the FileStream buffer was written after a read.

This was fixed by doing a seek before doing the problematic write.

Fixes #11699

[corlib] Test FileStream with Read and Writes

Covers #11699

Fixed the build.

[support/zlib] Added API exports to exported calls from zlib-helper.c.

This is so we can re-use this code from libmonodroid, which currently has its own duplicated copy.

[support/zlib] Remove eglib link dependencies from zlib-helper.c.

This is so we can re-use this code from libmonodroid, which currently has its own duplicated copy.

[reflection] MonoError in mono_reflection_get_custom_attrs_info

Also mark that function external only.  Runtime should use
mono_reflection_get_custom_attrs_info_checked

[reflection] MonoError for mono_reflection_type_get_underlying_system_type

Merge pull request #2655 from ludovic-henry/fix-wait-timeout

[WaitHandle] Fix Wait(One|Any|All) and SignalAndWait with timeout

[reflection] MonoError for mono_custom_attrs_data_construct

[reflection] MonoError for create_cattr_{typed,named}_arg

[reflection] MonoError for mono_reflection_method_on_tb_inst_get_handle

[jit] Remove a few mono_error_raise_exception () calls.

Fix the build.

[jit] Add a MonoError argument to mono_create_jit_trampoline (), get rid of the mono_create_jit_trampoline_in_domain () variant.

[jit] Add a MonoError argument to mono_create_jump_trampoline ().

[llvm] Remove some unused argument passing conventions.

Merge pull request #2673 from serpentblade/master

[System.Data.SqlClient] Fix for bulk copy inserts via IDataReader

[System.Data.SqlClient] Fix for bulk copy inserts via IDataReader

Resolve same with DataTable requiring name problem fixed in 1858ce4a94ab07f615a0e5ded37f8caa18473700

[sgen] catch NotSupportedException from system.xml.serialization

System.Xml.Serialization.TypeDesc.CheckSupported might throw a
NotSupportedException, so catch it and ignore the current type

[llvm] Fix the loaded llvm build.

Fix the build.

[llvm] Emit dwarf debug info when running with llvm 3.9. This is not enabled yet, since it doesn't seem to work, the dwarf info is in the object file emitted by llc, but disappears during linking.

[mcs] escaped quoted interpolated format specifier has no restrictions. Fixes #38992

Merge pull request #2664 from lambdageek/dev/monoerror-mono_rand

[runtime] Use MonoError in mono_rand_try_get_bytes

[System.IO.Compression.DeflateStream] Fixed end condition in decompression loop.

Keep track of total bytes read from input stream and break from the loop if it matches the number of bytes inflated by zlib.

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

[msvc] %PATH% is sometimes empty in Jenkins, so "powershell" doesn't run

Try using a system-standard default variable which points to the required folder instead.

Merge pull request #2666 from lambdageek/dev/kill-InternalReplaceStrComp

[runtime] Remove ves_icall_System_String_InternalReplace_Str_Comp

It appears to be dead code.

[runtime] Use MonoError in mono_rand_try_get_bytes

Bump Mono version -> 4.5.0

[runtime] Remove ves_icall_System_String_InternalReplace_Str_Comp

It appears to be dead code.

Merge pull request #2661 from ludovic-henry/fix-safehandle-abort

[SafeHandle] Avoid handle leakage in case of ThreadAbortException

If we get a ThreadAbortException while at SafeHandle.cs:124 (before this commit), we would not set release to true, even after incrementing the internal reference count of the SafeHandle. That could lead to potential leak, as the caller (following the common following pattern) would never call DangerousRelase, thus never calling PerformRelease. The finalizer or Dispose would not lead to the call of PerformRelease either, as the DangerousAddRef / DangerousRelase calls are unbalanced.

The common pattern for DangerousAddRef / DangerousRelease is the following:

bool release = false;
try {
  safeHandle.DangerousAddRef (ref release);
  /* do something with safeHandle.DangerousGetHandle () */
} finally {
  if (release)
    safeHandle.DangerousRelease ();
}

@monojenkins merge

Merge pull request #2657 from xmcclure/little-embed-fixes

Minor platform-support fixes

* Fix error in DISABLE_HW_TRAPS implementation
* Rename class in mono/tests/threadpool.cs to allow embedding

Merge pull request #2653 from lambdageek/dev/exception_type-1

[runtime] Don't access MonoClass::exception_type directly.

Use `mono_class_get_failure ()` and `mono_class_has_failed ()`

This is mostly just cleanup refactoring at this point that will make it easier to change how exception_type is used.

Also from this exercise we've learned that most of the class handling code just cares whether the class has an error or not, not what sort of error it was.

Merge pull request #2646 from ludovic-henry/fix-processwatch-dispose

[Microsoft.Build] Fix XS build error + Fix expected output newline from ProcessWrapper.OutputStreamChanged and ProcessWrapper.ErrorStreamChanged

[SafeHandle] Avoid handle leakage in case of ThreadAbortException

If we get a ThreadAbortException while at SafeHandle.cs:124 (before this commit), we would not set release to true, even after incrementing the internal reference count of the SafeHandle. That could lead to potential leak, as the caller (following the common following pattern) would never call DangerousRelase, thus never calling PerformRelease. The finalizer or Dispose would not lead to the call of PerformRelease either, as the DangerousAddRef / DangerousRelase calls are unbalanced.

The common pattern for DangerousAddRef / DangerousRelease is the following:

bool release = false;
try {
  safeHandle.DangerousAddRef (ref release);
  /* do something with safeHandle.DangerousGetHandle () */
} finally {
  if (release)
    safeHandle.DangerousRelease ();
}

[WaitHandle] Fix Wait(One|Any|All) and SignalAndWait with timeout

In the case where we would call for example WaitHandle.WaitOne with a timeout, we could run in the case where we would wait way longer than the timeout would allow it (240s instead of 90s for example). This would be due to spurious wake, which would not be considered to be an error, but which would lead calling the `mono_os_cond_timedwait` function with the same timeout every time. That would mean that `mono_os_cond_timedwait` would return that it timedout, if and only if, it would timed out after N milliseconds, with N the millisecondsTimeout parameter to WaitOne in this example. That mean that any spurious wake would effectively lead to rewaiting with a full timeout every time.

That could lead to the following scenario:
 - T1 calls waitHandle.WaitOne(timeout = 1000ms) at T=0ms
 - T1 calls mono_os_cond_timedwait(timeout = 1000) at T=0ms
 - GC suspend T1 at T=500ms <- here it's not necessarily the GC, it can be anything suspending/signaling the thread
 - T1 spurious wake, and call mono_os_cond_timedwait(timeout = 1000) at T=500ms
 - GC interrupt T1 at T=1250ms
 - T1 spurious wake, and call mono_os_cond_timedwait(timeout = 1000) at T=1250ms
 - etc.

As you can see, T1 should have returned from waitHandle.WaitOne before the second spurious wake, but as it recalled mono_os_cond_timedwait with a timeout of 1000ms again, it didn't.

The fix consists in keeping track of when the call is supposed to finish, and pass an adaptable timeout to mono_os_cond_timedwait, which decreases as time goes on, to enventually reach 0, and return in a timely manner.

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

Merge pull request #2620 from ludovic-henry/referencesource-waithandle

[corlib] Import System.Threading.WaitHandle

Merge pull request #2651 from akoeplinger/dmcs-warning

[scripts] Add deprecation warning to dmcs

[runtime] Don't access MonoClass::exception_type directly.

Use mono_class_has_failure () or mono_class_get_failure ()

[System.IO.Compression] Lower minimum version in ZIP headers for compatibility.

This fixes compatibility with tools like MS Excel, file tool, and others that do not know how to recognize newer ZIP version formats.

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

Merge pull request #2593 from ludovic-henry/coop-fix-native-to-managed

[coop] Fix native-to-managed wrapper

[WaitHandle] Call SafeWaitHandle DangerousAddRef / DangerousRelease around native calls

We do not want to wait on a handle that have been cleaned up, as that will lead to a warning message from the io-layer, without any error in managed code. That mean we could not detect this error case and recover from it.

By doing this DangerousAddRef / DangerousRelease before going to native, we ensure that the WaitHandle is and will stay alive while waiting on it. This same is done in CoreCLR, but in the marshalling code.

[corlib] Import System.Threading.WaitHandle

[io-layer] Fix process_wait return value for current process

When waiting on the current process, we would return WAIT_FAILED, as waitpid would return -1, with errno set to ECHILD.
But the semantic of WaitForSingleObjectEx and WaitForMultipleObjectsEx when waiting on current process is to return WAIT_TIMEOUT. This is then a special case that we need to add to this code.

[scripts] Add deprecation warning to dmcs

dmcs was deprecated in favor of mcs a while back, yet people are still creating new scripts with it.
Adding a message that shows up on each dmcs invocation should hopefully make this more obvious.

Merge pull request #2648 from BrzVlad/fix-cprop-vregs

[jit] Fix init of cprop defs for vregs

[llvm] Avoid setting the mono.nofail.load flag with the new llvm branch, its not supported there.

Merge pull request #2660 from kumpera/loader-error-cleanup6

[mono-error] This PR eliminates mono_class_from_name and a couple of assorted fixes.

[runtime] Ban mono_class_from_name mono_class_from_name_case.

They are no longer in use by the runtime. Begone!

[tests] Pass cli args to test runner so we can pick which test to run

[util] When creating the TLE exception only provide aditional data if both type and assembly names are provided. Otherwise one would fail and we'd get an OOM instead.

[runtime][jit] Replace usage of mono_class_from_name with mono_class_load_from_name for variable type lookups.

[runtime] Switch probing usage of mono_class_from_name to mono_class_try_load_from_name.

Those are call sites that can handle missing types correctly.

[runtime] Add MonoError to search_modules. Fix signature of mono_class_from_name_checked_aux (MonoError MUST be last arg).

[runtime] Switch GENERATE_GET_CLASS_WITH_CACHE to use mono_class_load_from_name. Kill dead macro.

[runtime] Add manually coded version of GENERATE_GET_CLASS_WITH_CACHE that uses System.dll

[runtime] Replace mono_class_from_name with mono_class_load_from_name for explicit loads that must not fail.

[runtime] Add type load caching to one missing thing appdomain.c

[runtime] Use mono_class_load_from_name to load types in mono_defaults.

We can use mono_class_load_from_name in all cases that we previously used to assert after loading.

All optional types have being moved over to use GENERATE_TRY_GET_CLASS_WITH_CACHE.

All but System.Runtime.ConstrainedExecution.CriticalFinalizerObject which is used within
sgen and we must ensure we don't try to lazy load it there.

I'm not 100% sure whether it should be optional, maybe it's a leftover from the 1.0 -> 2.0 timeframe?

The problem is that marking more types as required would be a burden on linkers as they might not force
it to be preserved.

I'm not particularly happy with the current large set of types that are loaded upfront. A significant
number of them could be lazy loaded.

[runtime] Replace explicit lookup with GENERATE_GET_CLASS_WITH_CACHE in cominterop.c

Last place with adhoc lookup & caching.

[runtime] Add GENERATE_TRY_GET_CLASS_WITH_CACHE macro that generates a probing function.

This function generates probing functions that rely on mono_class_try_load_from_name.

In addition replace usage of hand crafted loading code from all around.

[runtime] Add mono_class_load_from_name and mono_class_try_load_from_name.

Those are variants of mono_class_from_name_checked that implement two common type loading scenarios in the rutime.

The first is mono_class_load_from_name, that loads critical types that the runtime cannot function without.
This means we abort if the type is missing or broken. We should use it in places where today we crash when
either of those would happen.

The other is mono_class_try_load_from_name, which is ok if the type is not found, but will abort if the type is broken.
The reasoning for aborting on a broken type is that it's unclear on whether the runtime should be using the type or not,
specially given most of those optional types change behavior of the application. This, in practice, forces the linker to
not produce a broken corlib.

Merge pull request #2647 from qmfrederik/fixes/eventsources

WSDualHttpBinding: Additional constructor overloads to match .NET 4.5

Merge pull request #2659 from tritao/system_io_compression_reopen

[System.IO.Compression] Fixed re-opening of disposed Zip entry streams.

Revert "Revert "[jit] Don't allocate memory for unused vreg""

This reverts commit c4037092ac415665b4b2aa639b389deb5bcfc12d.

[jit] Clear defs only for valid vregs used by the bblock's instructions

Also clean the most significant part of long vregs.

[System.Web.ApplicationServices] Add new file to EXTRA_DISTFILES

It was added in 383d80a7850c2737e931a6d9ca8b26c98eefd79c. Fixes tarball.