[Github-comments] [geany] Scope completion fallout patches (#906)

[Jiří Techet]

>So, I guess what I'd prefer is

> use the new type name

Good.

> introduce a deprecated alias of the old name

The trouble is that langType would be defined in both ctags and in the header and we'd have to keep using the LIBCTAGS_DEFINED macro tricks I wanted to eventually eliminate (at the moment still needed for tagEntryInfo). And because using the lang field in the various structs isn't that meaningful and because none of the gp plugins use it (and most plugins don't use the TM stuff at all which I think is the case of external plugins too), I think it's quite safe to assume nobody uses it.

But of course can add it if you disagree.

> install tm_parser.h (possibly guarding everything but the typedef in GEANY_PRIVATE)

Done.

---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/906#issuecomment-189059814
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160225/e21105a7/attachment.html>