[Monodevelop-devel] Language specific expression evaluator support

Carlo Kok ck at carlo-kok.com
Fri Apr 23 03:27:40 EDT 2010


On 4/22/2010 6:16 PM, Lluis Sanchez Gual wrote:
> Hi,
>
> Thanks for the patch. It looks good in general, although I have some
> comments:
>
>        * Indenting seems to be done with spaces, it should be with tabs.
>        * Instead of having two handlers in DebuggerSession
>          (GetExpressionEvaluator and GetAllLocalsHandler), it would be
>          simpler to just have a single method which returns a
>          IExpressionEvaluator.
>        * The evaluators list in DebuggingService should be reset if there
>          is any change in the evaluators extension point (like what is
>          done for DebuggerEngines).
>

All fixed. New patch attached.

--
Carlo Kok
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: custom_expression_evaluator5.patch
Url: http://lists.ximian.com/pipermail/monodevelop-devel-list/attachments/20100423/ea5226eb/attachment-0001.pl 


More information about the Monodevelop-devel-list mailing list