[Github-comments] [geany/geany] Highlight C function names (patch available) (#1231)

elextr notifications at xxxxx
Thu Sep 15 11:52:58 UTC 2016


To summarise:

1) it looks ok for C so long as non-call parens in expressions can't be confused with calls.

2) in C++ its possible that there will be too many erroneous highlights due to C++ using the call syntax in many places.

3) I'm not sure if/how well it applies to any of the other languages `Lexcpp.cxx` highlights.

I would suggest that the best way ahead is to submit your patch to Scintilla and see what Neil says.  If he is happy then its fine.

The above discussion then becomes purely for Geany and which languages to make it visible for by making the SCE_C_FUNC a non-default style.  Irrespective of which choice is made by default (say visible for C and invisible for C++) a user can always change it in their own configuration anyway.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/1231#issuecomment-247307523
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160915/8ae37a5c/attachment.html>


More information about the Github-comments mailing list