[MonoDevelop] [Patch] On-the-fly parsing support for MonoDevelop

Lluis Sanchez lluis at ximian.com
Thu Sep 13 10:49:09 EDT 2007


Hey Jérémie,

The code looks great, thanks! A few comments:

      * Commented code like this should be removed:

+                       /*string errorType = (type ==
ErrorType.Syntaxic) ? "synError" : "semError";*/

      * You should unsubscribe ParseInformationChanged event when the
        SourceEditorView is disposed, or you'll be holding the editor
        object in memory forever.
      * I can't see the tooltip, it's never shown.
      * To commit the code we need the corresponding ChangeLog entries.

Thanks,
Lluis.

El dj 13 de 09 del 2007 a les 13:16 +0200, en/na Jérémie / Garuma / Kiri
va escriure:
> Hello everybody,
> 
> This patch implements the necessary bits to retrieve parsing errors
> from NRefactory and the associated methods to show them to the user by
> both underlining the errors in the text editor and displaying error's
> description in the normal editor tooltips. 
> 
> Thanks.
> 
> LAVAL Jérémie
> jeremie.laval at gmail.com
> http://garuma.wordpress.com
> _______________________________________________
> Monodevelop-list mailing list
> Monodevelop-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/monodevelop-list



More information about the Monodevelop-list mailing list