[MonoDevelop] [Patch] On-the-fly parsing support for MonoDevelop
Lluis Sanchez
lluis at ximian.com
Fri Sep 21 08:21:12 EDT 2007
Can you provide the ChangeLog for those changes? I'm going to commit.
Thanks!
Lluis.
El dj 20 de 09 del 2007 a les 16:19 +0200, en/na Jérémie LAVAL va
escriure:
> Attached is an updated version which comply to Mono coding Guidelines
> (no code change).
>
> Le Thu, 20 Sep 2007 14:57:20 +0200,
> "Jérémie Laval" <jeremie.laval at gmail.com> a écrit :
>
> > Some up-to-date things :) .
> >
> > Attached is the patch that add the "visual" part of autocorrection to
> > Monodevelop (as the core changes are already in Monodevelop SVN). To
> > apply it go to "$basedir/monodevelop/Extras/MonoDevelop.SourceEditor"
> > and patch with "cat SourceEditor.patch | patch -p0".
> >
> > If you have any suggestion either on the code or the visual aspect
> > ping me ;) .
> >
> > 2007/9/13, Lluis Sanchez <lluis at ximian.com>:
> > >
> > > 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
> > >
> > >
> _______________________________________________
> 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