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

[elextr]

> We could have a patch file similar to Scintilla which introduces the changes we need in ctags

Totally, and I support the end target of your changes, just not the process of getting there :smile:

I accidentally selected the "files changed" tab and it locked the browser for a time and used 1GB of memory  +5328 line -5619 lines.  IMNSHO its not a responsible thing to propose or to accept such a change.  I greatly admire the quality of your submissions, but even you can't be sure whats in such a change, and poor @b4n really can't be expected to review it.  There has got to be a better way, what about doing the whitespace first then?  That will have no code changes and would be more reasonably reviewable and could even be a few files at a time.

> my guts feeling is our c.c is better than the uctags one

Thats extremely disappointing since I think so much of ours that I turn off all uses of it but the symbols pane :smile:

(and the new uctags C/C++ one didn't seem to be much better when I tested, but presumably it will improve)

> but @elextr, it sounds like a challenge for you :-)

I gave up self flagellation decades ago :smile:

And given your gut feeling above, no point anyway :frowning: 

---
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#issuecomment-237504880
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160804/7ccce617/attachment.html>