2010-10-01  Veerapuram Varadhan  <vvaradhan@novell.com>

** Fixes #561667
* SqlCommand.cs (Dispose): While Disposing, set connection's
datareader to null so the connection can be reused with another datareader.

2010-09-29  Veerapuram Varadhan  <vvaradhan@novell.com>

** Fixes failed test for smallmoney
* Tds70.cs (WriteParameterInfo): Round money values to 4
decimals, for smallmoney too.

2010-09-20  Veerapuram Varadhan  <vvaradhan@novell.com>

* SqlDataReaderTest.cs: Add test for bug#613087

test fixes

2010-09-16  Veerapuram Varadhan  <vvaradhan@novell.com>

* SqlCommandTest.cs: Add test for bug#569543

2010-09-16  Veerapuram Varadhan  <vvaradhan@novell.com>

* SqlCommandTest.cs: Add test for bug#584833

Rework TCP listening model, and fix AcceptTcpClient() to not drop timed out requests that actually steals next requests in the air.

WebResponse.Headers is not always available (e.g. moonlight browser stack)

* System.ServiceModel.Channels/HttpRequestChannel.cs: For Moonlight use
the WebResponse.SupportsHeaders before accessing the Headers collection.
This is needed because (the default) browser stack does not expose the
headers from its responses. Fix bug #642193

Turn Is*LockHeld properties into thread-specific values in ReaderWriterLockSlim. Add corresponding unit tests.

[asp.net] Get rid of a possible (although unlikely) race condition when acquiring locks

The patch removes a small race condition where a boolean flag is set after acquiring a lock to
indicate to code executing in the finally {} block that it should release the lock. The boolean
variable is now removed and the lock is released unconditionally. It carries a potential to
throw an exception when the lock is not held, but it's better than to fail to release it and
lead the application to a deadlock.

Avoid a culture-specific check for the end of an URI

* System/Uri.cs: Avoid the culture-specific String.EndsWith when
reducing an URI

Use same socket family as the endpoint when retrieving the policy

* System.Net.Policy/CrossDomainPolicyManager.cs: Use same socket family
as the endpoint when retrieving the policy. Also do not try to create a
policy out of an empty stream (avoid misleading exceptions in logs)

Fix Socket.RemoteEndPoint after ConnectAsync on Moonlight

* System.Net.Sockets/SocketAsyncEventArgs.cs: Set the Socket
RemoteEndPoint to our RemoteEndPoint on Connect operation.
* System.Net.Sockets/Socket_2_1.cs: Return the 'candidate' EndPoint
provided by SocketAsyncEventArgs in ConnectAsync (intead of null)
until the connection is completed.

[wcf] Make simplified configuration for behaviors work (4.0)

* NamedServiceModelExtensionCollectionElement.cs: minimum length of the Name property value is 0 for 4.0
* ServiceBehaviorElementCollection.cs: added an override of BaseAdd which makes sure that the element being
  added has its Name set to String.Empty if the element's name is null
* ServiceHostBase.cs: support for simplified behavior configuration. Refactored behavior addition code to a
  separate method. Check whether behavior is not in the collection before adding it.

Add runtime test for delegates with null target.

Add comment on the reason of a non obvious check.

Enable test that has been fixed in the runtime.

Raise ArgumentException for delegate on closed methods.

* method-to-ir.c (mono_method_to_ir): Check for null
on delegate construction so closed-on-null is not allowed
when using the delegate .ctor.

If the delegate been constructed is open-on-target then
the null check must not be applied as the IL pattern is:

ldnull
ldftn void Bla::Foo(int)
newobj Bar (object, native int) // delegate void Bar(Bla,int)

Mov the jit map generation code from tramp-x86.c to mini-trampolines.c, add amd64 support.

Ignore all non static fields.

* icall.c (ves_icall_get_enum_info): All instance fields
should be ignored here.

Fixes #604218

Record all trampolines to the jitmap.

Fix the net_4_0 build

Fix the moonlight build

[fix] #631810: Form.DialogResult needs to call its close events *before* closing.

* Form.cs: When setting DialogResult call RaiseCloseEvents
instead of directly setting 'closing', since we need to have our
FormClosing event invoked *before* the form dissapears, so the
user can properly cancel the close operation if desired.
Fixes #631810.

[fix] #639939: Manually and properly convert the bytes returned by XLookupString.

* X11Keyboard.cs: Manually handle the conversion of the string
returned by XLookupString, just as we were already doing for XIM
support - using StringBuilder directly was somehow returning a
corrupt buffer from time to time.
Fixes #639939.

[wcf] System.ServiceModel.Configuration.CustomBindignCollectionElement.GetDefault returns an instance of CustomBinding

Set LastOperation properly

SocketAsyncEventArgs.LastOperation is set when the operation is
invoked.
Reset the error status before each new *Async() call.

Commit some stuff I had implemented in my SVN tree.

Merge branch 'master' of github.com:mono/mono

[bug workaround] Fix the build of System.Web.Mvc by renaming internal StringComparer class to InternalStringComparer

The build fails because of what seems to be a bug in gmcs. The compiler reports that
the StringComparer.OrdinalIgnoreCase property doesn't exist and it does so because
gmcs loads the internal System.Collections.Generic.StringComparer from EqualitComparer.cs instead of the
System.StringComparer one.

Update the comment explaining why the small_id_table uses conservative
GC tracking.

Bypass BeginInvoke for asynch. operations

The code involved in a delegate BeginInvoke is too heavy and of no use
for socket asynchronous operations. With these changes, we bypass all
that and send the work items directly to the asynchronous IO queue,
improving performance and lowering memory use.

Fix EqualityComparer for strings.

It turns out that Strings need to explicitly use a version that does
implement EqualityComparer<string> since string is an IEquatable.

Do this, and to avoid the reflection mess, cast to object, then cast
to the real type

Optimize EqualityComparer for strings

This shortcut prevents the reflection system to kick into action
for compiler generated string switch statements.

ConnectAsync() security checks

-ConnectAsync() changed to make security checks for all endpoints before
trying to connect for the first time.
-Use the socket Connect() call instead of repeating the same stuff.
Connect should be splitted in 2 pieces (DNS + Connection attempts).

Added support for ArraySegment

-Removed NET_2_0
-Implemented support for the Receive/Send overloads that take a list of
ArraySegments.

[Messaging] Conditionally drop dependency on Wnforms

If you pass the option MESSAGING_NO_WINFORMS=1 to make, System.Messaging will be compiled
without the dependency to System.Windows.Forms. The dependency is needed just by the
System.Messaging.Design.QueuePathDialog class which is both a stub and not used anywhere
by Mono.

Enable support for wrapping non Exception subclasses in RuntimeWrappedException objects, implement support for RuntimeCompatibilityAttribute.

Made the moonlight profile happy.

Disabled operations not allowed under moonlight.

Move async IO code to Socket_2_1

Moved internal classes to Socket_2_1.

Rework SocketAsyncEventArgs internals

-Removed #if NET_2_0
-All the *Async() calls but ConnectAsync() are routed through the
asynch IO threadpool instead of creating a new short-lived thread for
every operation.
-When reusing a SocketAsyncEventArgs only 1 new object, a delegate, is
created.

Implemented more combinations of nullable boolean logical operators

Better error message for syntax error in namespace name

Improve the Uri parsing speed

This affects MonoTouch startup as our dictionaries are a little bit
slow to be created now that we use Dictionary<> hashes for switch
statements.

Some timings:

Using a simple block of if's is twice as slow as the compiler generated
switch statement.   But using this tuned code is faster than the
compiler generated code, with a million loops on x86-64:

With "http": .10 vs .51 (first check)
with "https": .16 vs .51 (second check)
with "foo": .22 vs .31 (never found)
with "mailto": .12 vs .51  (last check)

Fix UriParser.GetComponents for the PathAndQuery case

PathAndQuery should always return the path starting with /,
even when it's empty

Fix AOT support for OP_GC_CARD_TABLE on amd64.

Fix the setting of pointer type fields using reflection. Fixes #641590.

Implement amd64 support for OP_CARDTABLE.

* amd64-codegen.h (amd64_alu_reg_membase_size): Add support
for RIP based addressing.

* cpu-amd64.md: Add card_table_wbarrier.

* mini-amd64.c (mono_arch_output_basic_block): Emit the
new OP.

* mini-amd64.c (mono_arch_emit_exceptions): Handle another
kind of patch-info - GC_CARD_TABLE_ADDR. This is required
because we can neither have 64bits immediates with amd64
or 2 scratch regiters with current regalloc.

* mini-amd64.h: Define MONO_ARCH_HAVE_CARD_TABLE_WBARRIER.

Assign identical operation name for each contract version so that they don't conflict each other.

Fix duplex proxy constructor arg names.

JsonSerializationWriter: handle char values

Filter requests only by paths, excluding host and port.

Override IsContentTypeSupported() in per-encoding-binding-element implementation to work as expected.

Avoid making the small_id table precisely GC tracked, some runtime code can't handle MonoInternalThread's being moved.

On 64bit this is turning into a 16 byte struct, this works around the issue.

Disable make_root_descr_all_refs () for now since it seems to cause crashes.

Use g_malloc0 in mono_gc_make_root-descr_all_refs ().

Remove a debug printf.

Add a few variants of the MONO_GC_REGISTER_ROOT () to make most roots precisely tracked and to avoid registering locations pointing to memory allocated by mono_gc_alloc_fixed () as roots under SGEN.

Disable the wrapped exception code for now, it break some of our tests.

Handle verification of types in non canonical forms.

* verify.c (mono_type_is_valid_type_in_context): Handle
types such as VAR and MVAR encoded as CLASS or VALUETYPE.
This encoding is not the canonical one, but works and
must be properly verified.

Fixes #601970

Remove unwanted debug spew.

Fix WebRequest.RegisterPrefix / Create to match SL in some corner cases

* System.Net/WebRequest_2_1.cs: (Create) Find the prefix that is the
closest match to the specified Uri and use it's associated creator.
(RegisterPrefix) We cannot register 'http[s]:' even if registering
'http[s]' is possible (but does not work as expected) as are the (fully)
valid 'http[s]:/[/]' values.

Add test case to show a different behavior (from Windows) wrt Socket buffers

* Test/System.Net.Sockets/SocketTest.cs: Add test case where buffers can
be set to 0 under Windows (but has a 256 bytes minimum on Linux)

[moonlight] Fully honor WebResponse.SupportsHeaders

* System.Net.Policy/CrossDomainPolicyManager.cs: Use InternalHeaders,
instead of Headers, since we always use the browser stack to download
the security policies.

* System.Net/WebClient_2_1.cs: Throw a NotImplementedException from
ResponseHeaders if the response does not support them.
* System.Net/WebResponse_2_1.cs: Make Headers property smart to throw a
NotImplementedException if SupportsHeaders return false. Add a new
InternalHeaders property since the response headers are needed,
internally (e.g. policies), even if not exposed to the outside world.

move DynamicData to mvc because it depends on Extensions

Fix the default exception message of RuntimeWrapperException.

Wrap thrown objects which do not derive from Exception into a RuntimeWrappedException. Fixes #640687.

epoll_ctl takes an int, not IntPtr

Ditto.

[641146] Implemented IStructuralComparable.CompareTo

[WCF] System.Data.Services.Client must be signed with the ECMA public key

[WCF] Changes to System.Data.Services.Client to compile it for 4.0

It appears Microsoft didn't make any changes to the assembly in .NET 4.0 - they simply
changed its version number to 4.0.0.0 and added few assembly attributes.

Fix exceptions wrt IPAddress.ScopeId

* System.Net/IPAddress.cs: ScopeId can only be set on IPv6 or it throws
a SocketException (not an Exception). ScopeId is also limited from 0 to
2^32.

Add some missing validations to Socket

* System.Net.Sockets/Socket_2_1.cs: Add checks for invalid AddressFamily,
SocketType and ProtocolType (to throw ArgumentException). Add negative
check for Ttl property. Add AddressFamily mismatch checks to ConnectAsync
methods

Add missing SocketOptionName_HopLimit definition for IPv6

* socket-io.c|h: Add definition/case for SocketOptionName_HopLimit.
Note that this is already being used by managed code (to set the TTL
value on IPv6 sockets)

Fix support of non-ASCII host names in System.Uri

* System/Uri.cs: Fix escaping non-ascii host names. Avoid memory
allocations (UTF8.GetBytes) when it is not necessary while escaping.

* Test/System/UriTest2.cs: Add test case where host name is β (beta)

Don't allow EventType.Breakpoint in vm.EnableEvents ().

Avoid extracting tar.gz files in the get-cygwin-deps.sh script, we download only zip files now.

Convert ERROR_PATH_NOT_FOUND to ENOENT too.

Don't link against missing libiconv on windows.

    * configure.ac: iconv check creates variable ICONV_LIBS
    * src/Makefile.am: only link eglib to iconv when iconv is found

Set errno correctly when loading an image fails on windows. Fixes #362607.

EpollEvent size adjustment

EpollEvent has to be one uint32 plus one int64, i.e. 12 bytes long.
Manos libres now.

Do not build mcs/docs if the 4.0 build is disabled.

Revert "mdoc 2.0-only profile build support, take 3!"

We'll instead just disable the docs build entirely if --with-profile4=no
is specified [future commit, not this one].

mdoc 2.0-only profile build support, take 3!

Further discussion suggested that, even if mdoc 2.0 works, it hasn't gotten
widespread testing (as it's been a 4.0 app for months), so it would be
preferable to keep it as a 4.0 app.

However, it still needs to be built under 2.0 so that a --with-profile4=no
build can...build...without errors.

Thus, the compromise: build mdoc twice, once for each profile.  This will
ensure that *some* version of mdoc is around when mcs/docs is built.

Unfortunately we can't place mdoc into $(per_profile_dirs), as these are
built before $(net_2_0_dirs) (and thus before monodoc, which needs to be
built before mdoc).  Thus the compromise of listing mdoc in *both*
$(net_2_0_dirs) and $(net_4_0_dirs), allowing build dependencies to be met.

LocalEndPoint works now for non-blocking connects

We didn't keep the endpoint around for non-blocking connects and later,
when Poll() was called, we didn't have the information. Also 'isbound'
wasn't being set in a few places and UDP sockets needed to be marked as
not connected after a call to Connect() with an Any address.

Removed an old test for Connect(Any) with UDP sockets, since this is
used to disconnect them now.

Fixes bug #640749.

The Hashtable is back

But this time it is using a hash code provider and a comparer that
don't invoke the object's methods.
Thanks to Paolo for pointing this out.

Call the thread cleanup at the end as it was done before, call it even if handle_remove () fails.

Add tidbits of documentation to ReaderWriterLockSlim.

Also set readerDoneEvent when exiting an upgradeable lock and reader condition is correct.

It could cause a deadlock when a writer tested the rwlock value before RwRead was removed and another Upgradeable operation happened at the same time.

Revert "mdoc 2.0-only profile build support, take 2!"

This reverts commit 313633fc67cf1479514019a56143e5ad1643d28f because it breaks the build.

mdoc 2.0-only profile build support, take 2!

Further discussion suggested that, even if mdoc 2.0 works, it hasn't gotten
widespread testing (as it's been a 4.0 app for months), so it would be
preferable to keep it as a 4.0 app.

However, it still needs to be built under 2.0 so that a --with-profile4=no
build can...build...without errors.

Thus, the compromise: build mdoc twice, once for each profile.  This will
ensure that *some* version of mdoc is around when mcs/docs is built.

NET_4_0 fixes for System.Uri also apply to Moonlight (SL4)

* System/Uri.cs: Update some NET_4_0 to include MOONLIGHT too

Do not allow + sign when specifying the port of an URI

* System/Uri.cs: A bit more strict when parsing the port number

* Test/System/UriTest2.cs: Add test cases with different, mostly
invalid, ports (e.g. negative, float, too big...)

Fix some host name checks, delay port check and fix some exceptions messages

* System/Uri.cs: Fix host name checks. Delay port check until the host
name is checked (compatibility). Fix exception messages to match the
expected values from the NCL tests.

* Test/System/UriTest2.cs: Add some test cases for host names

Don't store objects in a Hashtable.

Storing the objects in Hashtable causes the GetHashCode() and may be
Equals() methods to be called on the object, which MS does not do.
I'm using a list holding WeakReferences. Not sure if hard references
would be ok.

Fixes bug #640688.

[corlib] Prevent BinaryFormatter to throw on missing fields

This commit includes unit tests that were run in MS.NET > 2.0.
The old behaviour of MS.NET 1.1 is still included in the patch
as an ONLY_1_1 define, to be backport-friendly to other branches
(but can be removed from now on in next commits, as mono master
doesn't include this profile anymore). Fixes BNC#324144.

Enable cardtable for parallel mark-sweep.

* sgen-marksweep.c (mono_sgen_marksweep_init): Enable
cardtable for the parallel collector.

This closes by 50% the gap in binary-tree against the
serial collector.

Add support for break on unverifiable for verifier failure.

* mini.c (mini_method_compile): Break on verification
failure.

New tests.