[Github-comments] Re: [geany/geany] [C, C++] Calltips have missing space after comma for parameters (Issue #3474)

2023-05-03 Thread Nick Treleaven via Github-comments
> Does https://github.com/geany/geany/pull/3478 fix that problem? If so, it > should be submitted to universal-ctags first and if it gets merged No. And it's from universal-ctags :-) Thanks for the info. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issue

[Github-comments] Re: [geany/geany] [C, C++] Calltips have missing space after comma for parameters (Issue #3474)

2023-04-28 Thread Jiří Techet via Github-comments
> Calltips have missing space after comma for parameters Yeah, I've noticed that too after switching to the new parser. Does https://github.com/geany/geany/pull/3478 fix that problem? If so, it should be submitted to universal-ctags first and if it gets merged, we could then port it to Geany.

[Github-comments] Re: [geany/geany] [C, C++] Calltips have missing space after comma for parameters (Issue #3474)

2023-04-28 Thread Nick Treleaven via Github-comments
> Somebody needs to check that c-based.c contains at least the capability that > geany_c.c has, since it gathered changes during the divergence, since it > gathered changes during the divergence, then they can be synchronised Yeah, though sending the changes upstream could be a big job due to d

[Github-comments] Re: [geany/geany] [C, C++] Calltips have missing space after comma for parameters (Issue #3474)

2023-04-28 Thread Nick Treleaven via Github-comments
> A typo in cxxTokenCopy that wasn't setting bFollowedBySpace #3478 -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3474#issuecomment-1527506046 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany] [C, C++] Calltips have missing space after comma for parameters (Issue #3474)

2023-04-28 Thread elextr via Github-comments
> So looks like C#, D and Java could be changed to the c-based.c parsers from > ctags. `c-based.c` looks at first glance to be very similar to `geany_c.c` without C, C++, Vera, or Vala, just C# D and Java, though there may be detail differences. I guess both originated from the same ctags `c

[Github-comments] Re: [geany/geany] [C, C++] Calltips have missing space after comma for parameters (Issue #3474)

2023-04-28 Thread Nick Treleaven via Github-comments
BTW I was looking with meld if there were any upstream ctags changes that might fix this. I found 2 but they don't affect this: * A typo in cxxTokenCopy that wasn't setting bFollowedBySpace * Some changes in cxxTokenChainSplitOnComma setting bFollowedBySpace -- Reply to this email directly or vi

[Github-comments] Re: [geany/geany] [C, C++] Calltips have missing space after comma for parameters (Issue #3474)

2023-04-28 Thread Nick Treleaven via Github-comments
Oh OK, thanks (#3032). So looks like C#, D and Java could be changed to the `c-based.c` parsers from ctags. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/issues/3474#issuecomment-1527371338 You are receiving this because you are subscribed to this thread.

[Github-comments] Re: [geany/geany] [C, C++] Calltips have missing space after comma for parameters (Issue #3474)

2023-04-27 Thread elextr via Github-comments
> even though they use the same ctags/parsers/geany_c.c tag parser file as C > and C++. No actually, C and C++ use the parser in the `cxx` subdirectory, the legacy parsers in `geany_c.c` have `Old` on their names so they are not picked up. -- Reply to this email directly or view it on GitHub: