<blockquote>
<p>After merging <a class="issue-link js-issue-link" data-error-text="Failed to load issue title" data-id="436850387" data-permission-text="Issue title is private" data-url="https://github.com/geany/geany/issues/2132" data-hovercard-type="pull_request" data-hovercard-url="/geany/geany/pull/2132/hovercard" href="https://github.com/geany/geany/pull/2132">#2132</a>, I think I should stop introducing new code to ctags side.</p>
</blockquote>
<p>I can only guess this is not exactly what you meant, because we certainly don't want to freeze upstream u-ctags, nor stop innovation.  If you mean try and keep things stable, yes, it's best to maintain stable interface whenever possible, but it shouldn't prevent innovation.</p>
<blockquote>
<blockquote>
<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=713965" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/techee">@techee</a> <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=77077" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/masatake">@masatake</a> what about Geany's c.c having a lot of extra stuff (even extra languages IIUC)? I don't watch uctags much, but last I looked any attempt to import that was abandoned.</p>
</blockquote>
<p>Well, that will definitely not be done by me :-). However, I think that the first step for Geany could be to switch to the new cxx parser for C/C++ and not to use the parser of these languages from c.c. Then the C/C++ portion of c.c could be removed completely and I think especially C++ contributes by the largest amount of code. The same could be done with the upstream c.c and it might be much easier to merge what remains.</p>
</blockquote>
<p>It's probably something I'll look at at some point.  I'll check, but I guess it'll end up splitting the different languages into separate parsers proper.  I do even have a standalone Vala parser lying somewhere that just await a revival [1] so that part could be fairly easy and give Vala support a boost.  Anyway, it's not my priority right now, but it might become so when we progress further enough towards a u-ctags sync.</p>
<p>Apart from that, I don't have much to add: yes, the ultimate goal should probably being able to simply link a libuctags, and in the meantime we should try and make it as easy as possible to keep Geany's copy of uctags in sync with upstream.  I trust <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=713965" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/techee">@techee</a> to layout the required interface, as he's acquired a lot of experience here :)</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/issues/2119#issuecomment-486634862">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ6XBQM5RWEFU7WQVLTPSGJADANCNFSM4HEAA74Q">mute the thread</a>.<img src="https://github.com/notifications/beacon/AAIOWJ7AUZ4IRBD5KPIR6VTPSGJADANCNFSM4HEAA74Q.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/geany/geany","title":"geany/geany","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@b4n in #2119: \u003e After merging #2132, I think I should stop introducing new code to ctags side.\r\n\r\nI can only guess this is not exactly what you meant, because we certainly don't want to freeze upstream u-ctags, nor stop innovation.  If you mean try and keep things stable, yes, it's best to maintain stable interface whenever possible, but it shouldn't prevent innovation.\r\n\r\n\u003e \u003e @techee @masatake what about Geany's c.c having a lot of extra stuff (even extra languages IIUC)? I don't watch uctags much, but last I looked any attempt to import that was abandoned.\r\n\u003e \r\n\u003e Well, that will definitely not be done by me :-). However, I think that the first step for Geany could be to switch to the new cxx parser for C/C++ and not to use the parser of these languages from c.c. Then the C/C++ portion of c.c could be removed completely and I think especially C++ contributes by the largest amount of code. The same could be done with the upstream c.c and it might be much easier to merge what remains.\r\n\r\nIt's probably something I'll look at at some point.  I'll check, but I guess it'll end up splitting the different languages into separate parsers proper.  I do even have a standalone Vala parser lying somewhere that just await a revival [1] so that part could be fairly easy and give Vala support a boost.  Anyway, it's not my priority right now, but it might become so when we progress further enough towards a u-ctags sync.\r\n\r\nApart from that, I don't have much to add: yes, the ultimate goal should probably being able to simply link a libuctags, and in the meantime we should try and make it as easy as possible to keep Geany's copy of uctags in sync with upstream.  I trust @techee to layout the required interface, as he's acquired a lot of experience here :)"}],"action":{"name":"View Issue","url":"https://github.com/geany/geany/issues/2119#issuecomment-486634862"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/issues/2119#issuecomment-486634862",
"url": "https://github.com/geany/geany/issues/2119#issuecomment-486634862",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>