http://bugzilla.novell.com/show_bug.cgi?id=497175
User [email protected] added comment http://bugzilla.novell.com/show_bug.cgi?id=497175#c4 --- Comment #4 from Carlos Alberto Cortez <[email protected]> 2009-08-07 21:56:03 MDT --- Besides to the comment by Robert, I think that in: @@ -513,7 +525,14 @@ try { AsyncMethodResult result = data.Result; - object ret = data.Method.DynamicInvoke (data.Args); + object ret; + try { + ret = data.Method.DynamicInvoke (data.Args); + } catch (Exception ex) { + result.CompleteWithException (ex); + return; + } + result.Complete (ret); We could move the AsyncMethodResult line out of the try block, so we can use a catch block to call result.CompleteWithException, instead of having two try blocks - which I don't like very much. Besides that, the patch is fine. Thanks! -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. You are the assignee for the bug. _______________________________________________ mono-bugs maillist - [email protected] http://lists.ximian.com/mailman/listinfo/mono-bugs
