Merge pull request #1057 from lextm/master

[Mono.Security] Added Encode to AuthorityKeyIdentifierExtension and Subj...

Merge pull request #1061 from sethjackson/L_cuserid

L_cuserid was removed from OpenBSD.

Replaced C99 variable-length array with an alloca to work with MSVC.

Fixed SGEN_HASH_TABLE_INIT usages to work in MSVC.

Fixed TLAB_ACCESS_INIT usages in MSVC by declaring the variables explicitly at the start of each block.

[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).

L_cuserid was removed from OpenBSD.

This change released under the MIT license with permission from the original
author Christian Weisgerber.

[mcs] Don't release field address copy before it's used. Fixes #20040

[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.

Fix a couple of compiler warnings.

[Mono.Security] Added AuthorityKeyIdentifierExtensionTest and SubjectKeyIdentifierExtensionTest.

[mcs] Fix error reporting propagation for loop statements. Fixes #20034

Merge pull request #1059 from ranma42/jenkins-warnings-cleanup

Cleanup of some MonoJenkins warnings

Merge pull request #1033 from godFather89/master

When dynamically compiling script with a class with a nested class defin...

[mcs] Don't report hiding warnings for compiler generated fields

[mcs] Add cref handling of user operator documentation without parameters

[eval] Add test for nested types

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.

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.

Rename configure.in -> configure.ac. Based on PR #976.

Merge pull request #1056 from ranma42/vc-compat-C90

Fix the build on Visual C++

[Mono.Security] Added Encode to AuthorityKeyIdentifierExtension and SubjectKeyIdentifierExtension.

Fix the build on Visual C++

Visual C++ does not support intermixed declarations and code and
errors out with:
error C2275: 'guint' : illegal use of this type as an expression

This also silences gcc/clang warnings such as:
mono-counters.c:xxx:3: warning: ISO C90 forbids mixing declarations
and code

In this case this can be fixed easily by skipping the intermediate
variable.

Based on pull request #1043 by J-M <jean-michel.perraud@csiro.au>

This change is released under the MIT license.

Merge pull request #1055 from josedonizetti/fix_datacontractserializer_tests

validate if attribute is defined before check IsAny

[build] the csproj files can now do a full class libraries build

[System.Runtime.Remoting] Build if DBEUG Is defined

[build] MSBuild setup: Add support for -keyfile and -delaysign, thanks to Marek Safar for the pointer

[build] Drop AssemblyInfo from the sources in Mono.Cecil.Mdb

[build] Fix Mono.Cecil.Mdb build with xbuild

validate if attribute is defined before check IsAny

Revert "[build] Make Mono.Cecil use an [assmebly:AssemblyKeyFile] to specify keyfile"

This reverts commit 20872b3abe24c0e085710ea09209a92617fdb2e5 to fix the build

[build] Make Mono.Cecil use an [assmebly:AssemblyKeyFile] to specify keyfile

Currently the flag is passed on the command line, but this is
problematic for us since our xbuild implementation does not seem to support
the attribute that generates the /keyfile: argument

Fix TimeZoneTest to build properly with new-style XI assemblies.

Remove unused namespace from LazyTest.cs

[build] Fix the reference to ecma.pub

[runtime] Don't leak memory if the custom attribute constructor throws.

[runtime] Add regression tests for custom attribute loader errors.

[reflection] Harder the custom attribute code against common type load errors.

Replace silent failures with exceptions.

[runtime] Change the semantics of mono_error_set_type_load_name to acquire alloc'd strings.

The use case for this function compared to the one that take MonoClass/MonoImage is for lookups
and the failed names are usually freshly alloc'd.

Fix a leak in class.c usage of mono_error_set_type_load_name.

[reflection] Do proper error handling in more cases.

Remove all internal usages of mono_custom_attrs_construct and replace with mono_custom_attrs_construct_by_type
that takes a MonoError.

Introduce mono_custom_attrs_get_attr_checked and replace usage of the old one where it's needed.

[build] Fresh new csproj files based on the updated build system.

[build] Fix location of the key file

[Managed.Windows.Forms] Disable TRACE code, since it is broken

[build] The ecma.pub is one directory above

[msvc/scripts] Update order.xml file with Makefile changes, update generator

[build] Ensure that staged build directories are set on LOCAL_MCS_FLAGS

The LOCAL_MCS_FLAGS are passed to the compiler before the standard
directory for the profile.  This is important both for correctness,
and also assists the solution generator, since we can now precisely
determine which intermediate project must be referenced.

Fix warnings

[system] Add SendMailAsync methods

[mscorlib] Since .NET 4.5 some of the System.IdentityModel now is in the core, add this class

[Mono.Posix] Support OS X ENOATTR

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

The original Mono.Unix.Native.Errno design used the Linux <errno.h>
constants and values...largely because it was easy.

As it turns out, this is also incomplete. The getxattr(2) defines
ENOATTR as a possible error code. On Linux, ENOATTR is an alias for
ENODATA [0]. On OS X, ENOATTR is _not_ an alias. Furthermore, since
ENOATTR is not a Linux errno value (it's an alias!),
Mono_Posix_ToErrno() couldn't check for the native ENOATTR value, so a
Syscall.GetLastError() call after a failing Syscall.getxattr() would
fail with a System.ArgumentOutOfRangeException:

$ csharp -r:Mono.Posix.dll
Mono C# Shell, type "help;" for help

Enter statements below.
csharp> using Mono.Unix.Native;
csharp> byte[] value;
csharp> Syscall.getxattr(".", "test", out value);
-1
csharp> Syscall.GetLastError();
System.ArgumentOutOfRangeException: Current platform doesn't support this value.
Parameter name: value
93
at Mono.Unix.Native.NativeConvert.ThrowArgumentException (System.Object value) [0x00000] in <filename unknown>:0
at Mono.Unix.Native.NativeConvert.ToErrno (Int32 value) [0x00000] in <filename unknown>:0
at Mono.Unix.Native.Stdlib.GetLastError () [0x00000] in <filename unknown>:0

The fix is to add an Errno.ENOATTR field [1].

The complication is one of what enum value to use: at present, the
Errno values are based on the Linux values. In the case of ENOATTR
(and others), there is no Linux value. Meanwhile, there is the
possibility of additional Linux error values; how do we future proof
ourselves?

This isn't a perfect solution, but it should punt the problem down the
road a bit: Errno is an `int` enum, and thus has large range. To
support the OS X values, we add 1000 to the OS X error value.

/* OS X: <errno.h> */
#define ENOATTR 93 /* Attribute not found */

// C# Errno enum:
public enum Errno : int {
// ...
ENOATTR = 1093,
}

This gives us 0..1000 for POSIX+Linux error codes, 1000..2000 for OS X
specific error codes, and as new platforms require additional support
they can all start at multiples of 1000.

Hopefully no single *nix platform will add more than 1000 error codes...

[0]: http://linux.die.net/man/2/getxattr
[1]: Related part of the fix: add all the OTHER <errno.h> values that
     OS X includes but weren't included in the Errno enum.

Revert "[system] Add new StreamWebSocket.cs file to the mobile profile list of sources"

This reverts commit 00ecc7392d03c817d2f52de6758195633b2d28ee.

[xbuild] fix NET_2_0 build.

[xbuild] implement ToolLocationHelper.GetPathToStandardLibraries().

[build] Add new project files based on the new staged build

[build] Use LOCAL_MCS_FLAGS to give preference to the intermediate dir in System builds, and do not use temporary files for corlib builds

Revert "Merge pull request #981 from methane/websocket"

This reverts commit 62cc262196951b67a13069444b889dc71758776c, reversing
changes made to c1028cf91286489ef075a33c91d29d1efea8d200.

[System] Fix parsing of IPv6 addresses (bug #18574)

From Dick Porter:

The IPv6 address parser misses some syntax errors, and can become confused by
addresses that at first glance look IPv4 compatible.  The attach patch contains
the following fixes:

Don't fail to parse an address if we mistakenly thought it was IPv4 compatible
or mapped, just treat it as IPv6.

Fix IPv4-compatible and mapped address formatting in ToString().

Fix parsing of colons - specifically occurrences of ':::'.

Make sure blocks of digits are no longer than 4 characters.

Add some more parsing tests to Mono's NUnit test suite.

Add a NUnit test for bogus IPv6 addresses.

Update order.xml

[system] Small optimization for Xamarin.Mac (where we know we're executing on a Mac)

[msbuild] implement CodeTaskFactory i.e. inline tasks.

[MS.Build] fix some TaskFactory related bugs uncovered by CodeTaskFactory impl.

- ITaskFactory could return null, so check the return value.
- UsingTask descendants could have property references, so use ExpandString.
- TaskBody was not appropriately passed to internal structure.

[MS.Build] fix default build targets to not return empty list in Project.

And since the code is almost the same as existing ProjectInstance one, just
make it identical. (Should be unified, but depends on members, hence TODO.)

Small fix.

Revert "Fix WebProxy.BypassProxyOnLocal logic."

This reverts commit 3f7ad8f457f67576f400a172573081818fbda2dc.

WebProxy.BypassProxyOnLocal is really broken!  There seems to be no
way of actually bypassing a proxy on local connections.

Merge pull request #1052 from sethjackson/master

Don't define _XOPEN_SOURCE on OpenBSD.

Fix tests not to hang on failure

[mcs] Expand type parameter member change using effective base type

Fix WebProxy.BypassProxyOnLocal logic.

[mcs] Handling of nested effective base classes

Merge pull request #1051 from josedonizetti/bug11916-DataContractSerializer

add IsAny=true support for DataContractSerializer

Don't define _XOPEN_SOURCE on OpenBSD. With _XOPEN_SOURCE defined mincore() is not visible.

This change is licensed under the MIT license.

[build/msbuild] Tool resolves assemblies based on exact match, not random-last

add IsAny=true support for DataContractSerializer

[arm64] Fix the definition of DWARF_PC_REG, it should be x30 (lr).

Fix casing in monotouch_Mono.CSharp.dll.sources

fix V547 Expression 'ccw->ref_count >= 0' is always true. Unsigned type value is always >= 0.

fix V547 Expression 'index >= 0' is always true. Unsigned type value is always >= 0.

fix V547 Expression 'index >= 0' is always true. Unsigned type value is always >= 0.

fix V595 The 'method' pointer was utilized before it was verified against nullptr. Check lines: 302, 303.

fix V595 The 'amodule' pointer was utilized before it was verified against nullptr

Merge pull request #924 from marcusva/master

- Use the correct absolute path for relative paths within a rooted path

Merge pull request #164 from LogosBible/AppSettingsFileChanges

Use the configuration file path directory for AppSettings File directory

Merge pull request #1045 from sethjackson/master

kinfo_proc2 was removed from OpenBSD. Use kinfo_proc instead.

Merge pull request #1046 from sethjackson/null-gc

Allow building with no GC again.

[mcs] Emit better code for null coalescing operator with nullable return type. Fixes #19702

Allow building with no GC again.

These changes are released under the MIT license.

kinfo_proc2 was removed from OpenBSD. Use kinfo_proc instead.

This change is released under the MIT license with permission from the
original author Robert Nagy.

Merge pull request #1049 from ranma42/fix-check-gdi-sibling

Fix checking when libgdi is configured as "sibling"

Fix checking when libgdi is configured as "sibling"

When libgdiplus is configured as sibling, it is available in the
libgdiplus_loc path. This path was added to runtime/mono/config, but
it was later overridden by the prefix-based path.

Using the same approach as for libMonoPosixHelper fixed the lookup of
the library (performed, for example, when testing
MonoTests.System.Drawing without installing libgdiplus).

This change is released under the MIT license.

[Http]: Cleanup and simplify the ServicePoint's connection group list.

Merge pull request #1032 from miguelzf/master

[Jay] Fix memory corruptions: out-of-bounds and uninitialized value.

Merge pull request #1044 from ranma42/fix-icc-build

Fix building mono with icc

Fix building mono with icc

When compiling mono with icc (ICC) 14.0.0 20130728 the build fails on

  CC     libmonoruntimesgen_la-sgen-stw.lo

with the following error:

Assembler messages:
Error: suffix or operands invalid for `lea'

Apparently icc (or the assembler) dislikes the rip-based LEA
operation. This can be worked around by manually assembling the
instruction and replacing it with the corresponding byte sequence.

This change is released under the MIT license.

[mcs] Add explicit array conversion involving type parameters. Fixes #18474

[mcs] Undo implicit object prefix for cref interface references

[MS.Build] fix support for MSBuildExtensionsPath* to try more than one paths.

So, it seems that where MSBuildExtensionsPath* is evaluated should be likely
evaluated multiple times, and find actually matching files in one of those.
F# build .targets file expects that.
Since it is almost impossible to assume in which usage scenario it is going
to be evaluated multiple times (those properties can be used anywhere by
simple $(...) syntax), it limits the attempts only for Import() so far.
Old xbuild actually does so too. That way, the particular F# issue is fixed.

[MS.Build] Fix metadata item parsing wrong indexing.

[mcs] Explicit type parameter conversion to generic parameter

[GC] Make size dependent vars use size_t instead of glong or int.

This opens the door for huge nurseries.

[vsbuild] Update Makefiles to not generate the VS metadata for the 'basic' profile