[Mono-dev] [PATCH] WCF multithreaded and property handling
atsushieno at veritas-vos-liberabit.com
Tue Mar 23 08:58:56 EDT 2010
As for the properties test case, never mind. I happened to get one at
hand (as my reply on DestinationUnreachable implies).
On 2010/03/23 21:49, Atsushi Eno wrote:
> Thanks for the patch. They are looking like a great set of attempts
> for cool bugfixes :) However there is a lot of other changes that your
> description cannot explain. So, please first split the changes into
> further smaller patches for each purpose, and give explanation for
> each change. For example,
> - // FIXME: this should not be required, but it somehow saves some
> failures wrt concurrent calls.
> - Thread.Sleep (100);
> This kind of change should not be made without explanation. (you
> aren't really sure about why it exists, right?)
> As for ChannelListenerBase.Properties, I'd rather make the changes
> much smaller like the attached change. Isn't it enough? There's no
> test case that I can verify your (and-my) changes.
> Atsushi Eno
> On 2010/03/23 20:28, Matt Dargavel wrote:
>> The included patches fix the following:
>> multithreaded_fixes.patch: ObjectDisposedException,
>> InvalidOperationException("Another async TryReceiveRequest operation
>> is in progress") and other multithreaded timing fixes. Also includes
>> change to make GET ?wsdl case insensitive.
>> properties_handling.patch: MetadataPublishingInfo not available in
>> TransactionChannelListener’s inner_listener. I created a new
>> RetrieveProperty function as overriding GetProperty<T> didn’t work-
>> the ChannelListenerBase implementation was still called. Perhaps
>> there’s a bug with generic function overrides or maybe I’ve done
>> something silly there?
>> properties_and_wsdl.patch: patch for ServiceMetadataExtension.cs that
>> goes with the properties changes and the ?wsdl change.
>> Let me know if you have any questions. :-)
> Mono-devel-list mailing list
> Mono-devel-list at lists.ximian.com
More information about the Mono-devel-list