[Monodevelop-devel] Language specific expression evaluator support

Lluis Sanchez Gual slluis.devel at gmail.com
Thu Apr 22 12:16:41 EDT 2010


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

Lluis.

El dt 20 de 04 de 2010 a les 19:12 +0200, en/na Carlo Kok va escriure:
> Attached is a patch that will let plugins assign their own "Expression 
> Evaluator" for the debugger. Tested with Delphi Prism (works).
> 
> if oke, I'll check it in.
> 
> --
> Carlo Kok
> RemObjects Software
> _______________________________________________
> Monodevelop-devel-list mailing list
> Monodevelop-devel-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/monodevelop-devel-list




More information about the Monodevelop-devel-list mailing list