[Github-comments] [geany/geany] Ctags sync, part 1 (#1160)

[Jiří Techet]

> @@ -2234,12 +2232,12 @@ static boolean skipPostArgumentStuff (statementInfo *const st,
>  	{
>  		switch (c)
>  		{
> -			case ')':				break;
> -			case ':': skipMemIntializerList (token);break;	/* ctor-initializer */
> -			case '[': skipToMatch ("[]");			break;
> -			case '=': cppUngetc (c); end = TRUE;	break;
> -			case '{': cppUngetc (c); end = TRUE;	break;
> -			case '}': cppUngetc (c); end = TRUE;	break;
> +		case ')':                               break;
> +		case ':': skipMemIntializerList (token);break;  /* ctor-initializer */
> +		case '[': skipToMatch ("[]");           break;
> +		case '=': cppUngetc (c); end = TRUE;    break;
> +		case '{': cppUngetc (c); end = TRUE;    break;
> +		case '}': cppUngetc (c); end = TRUE;    break;

> that one looks odd, but well.

Yeah, I should have probably fixed the indentation in uctags but it's not that important.

> though, I highly doubt we'll merge c.c from U-CTags, as we have different things in it, and that they have another C++ one anyway. I'll probably try to extract the Vala part to a separate parser (I already started a while back IIRC, experimenting with other stuff in the way), and we'll "only" be left with like 3 parsers in there, that have probably diverged a lot less.
Though OTOH maybe yeah, when it's down to C#/Java/Vera/D it might start and be mergable.

Would it be too hard to do it the other way round and add Vala to uctag's c.c so it supports the same set of languages? I also think our function parameter recording works better than the uctags one so it might be worth porting too.

---
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/pull/1160/files/11dcc5e3e5a0014905fbb76457b52991135dc410..9745d470c6288d88b455436a918f92e13e271df4#r73955181
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160808/28a96689/attachment.html>