Merge pull request #901 from Blewzman/FixAggregateExceptionGetBaseException

Add [Category ("NotWorking")] to failing test.

Add [Category ("NotWorking")] to failing test.

Merge pull request #898 from UCIS/patch-3

Prevent exception in finalization of Form class

Merge pull request #899 from UCIS/patch-4

Fix error in ToolStripItem finalization

[llvm] Disable the 'dse' optimization pass as it is buggy and causes #17616. Fixes #17616.

[runtime] Add two variants of mono_gc_bzero and mono_gc_memmove. They better convey the behavior we need.

After reviewing all usage of mono_gc_bzero and mono_gc_memmove, I found that there are two usage patterns
we care for:

1)Aligned pointer, any size, have managed pointers.
This happens when directly dealing with managed objects like when cloning of boxing.
On this case, all pointers MUST be aligned.

2)Maybe aligned pointer, any size, might have managed pointers.
This happens for functions dealing with structs, specially around boxing and unboxing them.
Our data layout code ensures that structs with managed layout are aligned to at least a word.
This means that if the pointer is unaligned, no managed pointer can exists in the memory range
been processed.

Given those two scenarios, lets introduce two variants that enforce those constrains.

_aligned:
Pointers must be aligned, ensures atomicity of word copies;

_atomic:
If pointers are aligned, works like _aligned, otherwise it inherits the behavior of the
system memmove&memset.

The above split ensures we won't have to deal with any RISC unaligned load nonsense.

It also adds some sanity checks to code that must respect alignment.

We still need fast code for mono_gc_memmove_aligned and mono_gc_bzero_aligned.

We still need to verify if a version of memcpy version is needed. Maybe the extra
branch is not such a perf penalty.

Merge pull request #900 from Blewzman/FixAggregateExceptionGetBaseException

Add unit test for AggregateException.GetBaseException that works on .net but is broken on Mono.

Add unit test for AggregateException.GetBaseException that works on .net but is broken on Mono.

[mcs] Add NamespaceExpression to avoid problems with error location for misused namespaces. Fixes #17803

Properly preserve VPF registers in iOS. Fixes #17793.

[mcs] Add nested types handling to check whether types unify for some type parameter substitutions. Fixes #17647

Merge pull request #891 from DavidKarlas/listenerprefix

HttpListener now listens on default ports if not specified instead of random

[corlib] Throw CultureNotFoundException using invariant culture. Fixes #17690

HttpListener now listens on default ports if not specified instead of random

Revert "[jit] Implement OP_CARD_TABLE_WBARRIER on ios."

This reverts commit b3f36681c72a6900cd5e9df9b593d555b351669f.

Revert this as it breaks mt.

Revert "[jit] Fix a regression introduced by b3f36681c72a6900cd5e9df9b593d555b351669f."

This reverts commit d50c836721f22df74a45185d51d0b9d37c370e33.

Revert this as d50c836721f22df74a45185d51d0b9d37c370e33 breaks mt.

Fixes commit a63d0009af9a85c1d52475a1a9bffda34302f0b0

[corlib] Unicode numbers are not considers valid in number parsing. Fixes #17628

[corlib] Fix warnings

[corlib] Improve TimeSpan custom formatting. Fixes #17699

Fixed incomplete decompression bug in System.IO.Compression.DeflateStream.

Call zlib's inflate() even if no data was read from the stream, there might still be some data left to process.

Originally reported by Eric Roller, as Xamarin bugzilla #17183.
https://bugzilla.xamarin.com/show_bug.cgi?id=17183

Reviewed by Rodrigo Kumpera.

[corlib] Reset List enumerator current field. Fixes #17760

Fix typo

[system] Fix valid sorted set operations which could cause underlying tree modification

[mcs] Imported methods need to have SpecialName flag to be valid C# operators

Tests update

Put back and fix the SSL async patch.

Revert "Revert "Merge pull request #829 from symform/httpwebreq-async-ssl""

This reverts commit 56d9a057eb612124a6fa1566990d1002f699c744, which was a
revert of 35899c07fff7f7e4c6533364091bbac6ce86fdfd, thus putting 35889 back.

[mcs] Lift any non-nullable operand under runtime binder for lifted binary operation

Merge pull request #897 from echampet/scanbuild

Scanbuild

[System]: Reset NTLM auth state when redirecting; distinguish host and proxy.

Rewrite and fix the web stack's connection reuse and idle logic.

* ServicePoint now uses an idle timer to close idle connections.

* WebConnectionGroup has been mostly rewritten to use a linked
  list of open connections and their idle times.

  This fixes both the problem that we were leaking sockets and
  that we were not reusing existing connections.

* ServicePoint now correctly keeps track of the number of open
  connections.

* ServicePointManager now uses the correct default idle time
  of 100 seconds (previously 15 minutes).

* WebConnection uses the new WebConnectionGroup's busy logic
  to correctly flag connections as busy / reusable.

Fix WebConnectionStream.CheckComplete() logic for chunked encoding.

[corlib] Fix the 4.5 test suite.

[runtime] Initialize param_count under DISABLE_JIT

[runtime] Don't abort on null module scope, there's nothing special to do there. Fixes #5835.

The defacto behavior of a null module scope is to resolve in the current image.

[corlib] Remove obj_address debug helper, if anyone needs this, it can be done with overlapping fields.

[corlib] Fix 4.0 test suite compilation.

Add test for Bug #15572 from pull request #820

Add tests for bug #15574, pull request 819

Add tests for bug #15574, pull request 819

Merge pull request #820 from brendanzagaeski/master

Bug 15572. Lookup KnownTypeCollection element types in MSSimpleNamespace

Merge pull request #819 from brendanzagaeski/patch-1

Bug 15574. XML deserialization recursion: add array type to known_types?

Merge pull request #796 from alesliehughes/master

Support for DispIdAttribute in GetIdsOfNames

Merge pull request #778 from cmorris98/master

SqlBulkCopy Implementation

Merge pull request #835 from HorstKakuschke/master

Fixes Xamarin Bug 16742:  XmlConvert.ToDateTime(string, XmlDateTimeSeria...

[xbuild] Fix dependency in C# targets

There was a typo in DependsOnTargets attribute of the
_AddCorlibReference target definition. It didn't break any
behaviour, since the target it depended on was always built
earlier in the build anyway, so it was really a sanity check.

However, it broke the tests for the new (unused) engine,
Microsoft.Build.dll, which checks for unexpected attributes.

Revert "[jit] Align the stack in mono_arch_restore_context () on x86 since it might be unaligned if it was captured asynchronously."

This reverts commit 7798b94706e5552cd1abd80227631f3f9b1bfdad.

Revert this as it breaks debugging on android/x86 (#17410).

[mscorlib] Path: fix GetFullPath (/../bin/bash) to return /bin/bash, not bin/bash, fixes 17083

[HttpListener] Only bind to the specified hosts (and support '*' as Any), fixes #17204, #10818

[xbuild] Fix 2.0 tests to handle mcs sdk argument

[Uri] Fix the ParserUser feature of UriParseComponents

The current implementation was wrong, did not bother with pct-encoding
(despite being listed in the comments) and did not use the actual set
of characters listed on the spec, instead of using a white-list, it
had its own incorrect black-list.

#17636: Allow Int32.Parse with/AllowExponents to take negative exponents.  This is a change in behavior from 2009, when we fixed this: Novell#542464, now we match .NET 4.5

[DataAnnotation.Validator] Add support for nullable types, patch from lmarinucci@yahoo.com, fixes #17318

WebRequest: Provide stub ImpersonationLevel

The old implementation threw an exception requesting that the method be implemented
in a derived class, but the method is not virtual, so it could have never been
implemented.

Since Mono does not support this, at least we keep the token around and avoid
throwing.

Fix error in ToolStripItem finalization

The Dispose function in ToolStripItem should not remove itself from its owner during finalization. Finalization implies that the owner has also been garbage collected, and accessing it may result in undesirable behaviour. More specifically, the call to owner.Items.Remove ultimately may boil down to a call to GDI+ throwing an Exception like the following:

Program.UnhandledException: System.ArgumentException: A null reference or invalid value was found [GDI+ status: InvalidParameter]
  at System.Drawing.GDIPlus.CheckStatus (Status status) [0x00000] in <filename unknown>:0
  at System.Drawing.Graphics.GdipMeasureString (IntPtr graphics, System.String text, System.Drawing.Font font, System.Drawing.RectangleF& layoutRect, IntPtr stringFormat) [0x00000] in <filename unknown>:0
  at System.Drawing.Graphics.MeasureString (System.String text, System.Drawing.Font font, Int32 width, System.Drawing.StringFormat format) [0x00000] in <filename unknown>:0
  at (wrapper remoting-invoke-with-check) System.Drawing.Graphics:MeasureString (string,System.Drawing.Font,int,System.Drawing.StringFormat)
  at System.Windows.Forms.TextRenderer.MeasureTextInternal (IDeviceContext dc, System.String text, System.Drawing.Font font, Size proposedSize, TextFormatFlags flags, Boolean useMeasureString) [0x00000] in <filename unknown>:0
  at System.Windows.Forms.TextRenderer.MeasureText (System.String text, System.Drawing.Font font, Size proposedSize, TextFormatFlags flags) [0x00000] in <filename unknown>:0
  at System.Windows.Forms.ToolStripItem.CalculateAutoSize () [0x00000] in <filename unknown>:0
  at System.Windows.Forms.ToolStripItem.set_InternalOwner (System.Windows.Forms.ToolStrip value) [0x00000] in <filename unknown>:0
  at (wrapper remoting-invoke-with-check) System.Windows.Forms.ToolStripItem:set_InternalOwner (System.Windows.Forms.ToolStrip)
  at System.Windows.Forms.ToolStripItemCollection.Remove (System.Windows.Forms.ToolStripItem value) [0x00000] in <filename unknown>:0
  at System.Windows.Forms.ToolStripItem.Dispose (Boolean disposing) [0x00000] in <filename unknown>:0
  at System.Windows.Forms.ToolStripDropDownItem.Dispose (Boolean disposing) [0x00000] in <filename unknown>:0
  at System.Windows.Forms.ToolStripMenuItem.Dispose (Boolean disposing) [0x00000] in <filename unknown>:0
  at System.ComponentModel.Component.Finalize () [0x00000] in <filename unknown>:0

[xbuild] Fix Tooltask.ToolPath behaviour to match .NET

[xbuild] Make ToolLocationHelper work better for 2.0/3.5

Mono keeps unversioned tools in the 4.5 directory, so look there
for 2.0. Additionally, most 3.5 tools are in the 2.0 directory.

[xbuild] Fix typo that broke 2.0/3.5 targets

[xbuild] Fix mscorlib resolution for custom frameworks

Now that mcs supports multi-targeting, we can use the MSBuild 4.0+
system of finding mscorlib.dll in the target framework folders and
passing it to the compiler, along with the /nostdlib argument.

Like MSBuild, the old NoStdLib property can be set to true to disable
passing mscorlib.dll to the compiler, and the new NoCompilerStandardLib
property can be set to false to disable passing /nostdlib to the compiler.

This makes custom target frameworks work correctly without having to
override CscExe with a wrapper script.

[mcs] Don't report any not applicable operator errors during constant folding.

[NTLM]: Add new 'NtmlSettings' class to hold the 'DefaultAuthLevel' property.

When the NTLM sources are compiled into System.dll, then we need to store the
configuration variables there and peek/poke it in Mono.Security.

[system] Updates to WebUtility decoders and encoders

[system.xml] Add more invalid argument checks

int a[0], malloc(0), memcpy(a,b,0), memcmp(c,d,0) doesn't make much sense

This remove a scan-build warning

Signed-off-by: Etienne CHAMPETIER <etienne.champetier@fiducial.net>

fix typo in mono_counters_cleanup leading to a double free

Signed-off-by: Etienne CHAMPETIER <etienne.champetier@fiducial.net>

[arm] Use d0/d1 as fp scratch regs on non-hard float platforms. Hopefully fixes #17597.

[system.xml] Support more primitive types in XmlReader::ReadContentAs

[runtime] Remove overly pessimistic test.

The test asserts that we can't allocate int arrays with 2G elements.
With SGen on 64 bits, however, we can, which is a feature, not
a bug.

[sgen] new T[-1] must throw OverflowException, not OOM.

Fixes failure in mini/arrays.cs.

[sgen] Hint OOM exception branch as not taken.

[sgen] Fix overflow occurring with large object allocations.  Fixes #17590.

[corlib] Mark awaited task exception observed when GetResult does the wait

Revert "Merge pull request #829 from symform/httpwebreq-async-ssl"

This reverts commit 35899c07fff7f7e4c6533364091bbac6ce86fdfd, reversing
changes made to 80166fd02fadc470e40b59df0c8c65c19945ed21.

Merge pull request #892 from initram/master

Implemented Parse() and TryParse() for System.Numerics.BigInteger and DaysInYearBeforeMinSupportedYear for Calendars

Merge pull request #895 from ranma42/fix-tuple-hash-codegen

[corlib] Fix implementation of Tuple hash generator

[corlib] Fix implementation of Tuple hash generator

The Tuples.cs code generator dies with:

Unhandled Exception:
System.NotImplementedException: The requested feature is not implemented.
  at TupleGen.computeHash (Int32 destVar, Int32 start, Int32 count) [0x00000] in <filename unknown>:0
  at TupleGen.WriteHash (Int32 destVar, Int32 start, Int32 count) [0x00000] in <filename unknown>:0
  at TupleGen.Main () [0x00000] in <filename unknown>:0

This is caused by the incorrect definition of WriteHash(), which
instead of recursively calling itself is calling computeHash().

Thanks to Marek Safar for reporting the issue in
https://github.com/fsharp/fsharp/issues/188#issuecomment-34351850

Implemented DaysInYearBeforeMinSupportedYear for all Calendars that should have it in .Net 4.5

Cleanup and better comment.

Merge remote-tracking branch 'upstream/master'

[jit] Fix a regression introduced by b3f36681c72a6900cd5e9df9b593d555b351669f.

[numerics] Fix type in Cosh

Merge pull request #829 from symform/httpwebreq-async-ssl

Convert blocking operations in HttpWebRequest and SslClientStream to non-blocking

[corlib] Make primitive types smaller than int compare result compatible

[corlib] Fixes formatting of nested tuples

[corlib] Default comparer uses IComparable interface only

Merge branch 'BigIntegerParse'

Implemented overloaded versions of Parse and TryParse functions for BigInteger.

Implementation is heavily inspired by Int32.
Added unit tests for the new functions.

Fixes build

[corlib] Update fixed test

Merge pull request #889 from slluis/serializer-fix

Fix serialization issue

Fix serialization issue

Fixes BXC 5667 - SerializationException: The object with ID 1 could
not be resolved

[System] Fix process creation when UseShellExecute is set to false.

This fixes the behavior of process spawning when UseShellExecute is
set to true.

The right behavior is to return no process only when we used the system
launcher. So, spawning /bin/cat should behave in the same way independent
of how UseShellExecute is set.

Revert "[runtime] Don't pass SEE_MASK_NOCLOSEPROCESS to ShellExecuteEx as we don't care for the launched process. Part 1/3 of Bug #16119."

This reverts commit 6c06d109952acb28a8198dcf85a6e8f4d4393985.

[corlib] Harden CustomAttributeData::GetHashCode

[runtime] Put back ensure_synch_cs_set () which was removed in a157ac79e8f404f0eb9b263f5de697f13528deec to work around some shutdown crashes.

[mcs] Tests update

[runtime] Filter out events with same name

Fixes build

[corlib] Cleanup type ToString formatting.

[corlib] Fix some AssemblyName tests