<p><b>@techee</b> commented on this pull request.</p>

<hr>

<p>In <a href="https://github.com/geany/geany/pull/1263">ctags/main/entry.c</a>:</p>
<pre style='color:#555'>> +    if (includeExtensionFlags ()
+           && isXtagEnabled (XTAG_QUALIFIED_TAGS)
+           && doesInputLanguageRequestAutomaticFQTag ())
+               buildFqTagCache (tag);
+
+#ifdef CTAGS_LIB
+       getTagScopeInformation((tagEntryInfo *)tag, NULL, NULL);
+
+       if (TagEntryFunction != NULL)
+       {
+               ctagsTag t;
+
+               initCtagsTag(&t, tag);
+               length = TagEntryFunction(&t, TagEntryUserData);
+       }
+#else
</pre>
<p>This is something I was considering too. We could "write" tags into Geany's structures in the callback of the writer and use normal way of ctags initialization and parsing.</p>
<p>What however worries me is the MIO (TagFile.mio from entry.c) into which a writer is supposed to write - right now it's automatically opened/closed by ctags for any writer and that's not something we want. If we somehow avoid it and decide to keep it NULL, we have to be very careful that nothing from entry.c that refers it. That would be quite a big change and would make the diff bigger against uctags (and more fragile when something upstream changes).</p>
<p>I agree this is probably the right way it should be done and this is also the way it should probably be introduced into uctags. But I think having this shortcut in Geany which makes sure nothing with unexpected side-effects from ctags is called is better for now.</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/pull/1263">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABDrJ8vSTWI_ed7AIIgaraHWZINelsM-ks5rcb6SgaJpZM4KXvBh">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJ99DG0EzF9zxtF81vNAEJLnzzzb4ks5rcb6SgaJpZM4KXvBh.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany/pull/1263"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<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://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/geany/geany"}},"updates":{"snippets":[{"icon":"PERSON","message":"@techee commented on #1263"}],"action":{"name":"View Pull Request","url":"https://github.com/geany/geany/pull/1263"}}}</script>