[Geany-devel] Adding an R parser

Enrico Tröger enrico.troeger at xxxxx
Sun Feb 21 14:13:02 UTC 2010


On Fri, 19 Feb 2010 12:14:33 +0100, Jon wrote:

>On Fri, 19 Feb 2010 21:25:38 +1100
>Lex Trotman <elextr at gmail.com> wrote:
>
>> Thats because sum is listed as a keyword in Geanys filetypes.r file.
>> Someone who is knowledgeable in R should have a look and check the
>> filetypes.r file and submit a patch.
>
>OK. I've got a basic patch for that. I've not gone through the whole
>packages list, but I've removed everything that's not syntax ("if",
>"then", "else", etc.) from the primary list. I've had a look at the
>scintilla code, but I'm not sure about it. It seems to correctly
>include "." as part of a word, but then later excludes it (If I'm
>reading the code correctly). I'll try playing with a little later since
>my efforts thus far haven't changed anything.
>
>I think I need a better understanding of what Scintilla is doing before
>I can properly debug that one.
>
>Should I send the geany patch to the mailing list or just put it in the
>tracker? And should I open a new issue in the tracker, or add it to my
>R-Parser patch that's already there?

What exactly do you mean with 'the geany patch'?
I'm going to add your R symbol parser to SVN.

For the syntax highlighting issue, please open a bug report at the
Scintilla project as Lex said and attach the patch their if you have
one.
We'll get the changes into Geany when we update our Scintilla copy or
we can also manually backport the change. But, also as Lex said, we'd
like to stay as close as possible with Scintilla upstream.


Regards,
Enrico

-- 
Get my GPG key from http://www.uvena.de/pub.asc
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.geany.org/pipermail/devel/attachments/20100221/1e349709/attachment.pgp>


More information about the Devel mailing list