On Fri, 19 Feb 2010 12:14:33 +0100, Jon wrote:
On Fri, 19 Feb 2010 21:25:38 +1100 Lex Trotman elextr@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