<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=793526" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/b4n">@b4n</a> I'm done, phew! It's your turn :-)</p>
<p>The current state is this: if you delete geany/ctags/main, replace it with upstream-ctags/main and apply the 6-line patch (which I might eliminate in the future, it just requires some help on the uctags side and some more work on our side), you are done. This is the very last manual ctags/main sync so you'd better enjoy reviewing it - such an opportunity won't happen in the future ;-)</p>
<p>In any case, it doesn't make sense to review the patches as they are in this pull request - most of it is just how I tried to reduce the number of diffs against uctags and also originally I didn't even dream about having a full sync so there are some patches trying to simplify future manual syncs which are pointless now.</p>
<p>I think to review this it's best to first look at the corresponding upstream changes - it's these pull requests:</p>
<p><a class="issue-link js-issue-link" data-error-text="Failed to load issue title" data-id="438112415" data-permission-text="Issue title is private" data-url="https://github.com/universal-ctags/ctags/issues/2088" data-hovercard-type="pull_request" data-hovercard-url="/universal-ctags/ctags/pull/2088/hovercard" href="https://github.com/universal-ctags/ctags/pull/2088">universal-ctags/ctags#2088</a><br>
<a class="issue-link js-issue-link" data-error-text="Failed to load issue title" data-id="446716942" data-permission-text="Issue title is private" data-url="https://github.com/universal-ctags/ctags/issues/2102" data-hovercard-type="pull_request" data-hovercard-url="/universal-ctags/ctags/pull/2102/hovercard" href="https://github.com/universal-ctags/ctags/pull/2102">universal-ctags/ctags#2102</a></p>
<p>Then, the majority of Geany changes is inside src/tagmanager/tm_ctags.c and this file should be reviewed as a whole. It however resembles the mini-geany.c test mostly, it just does our specific things with the collected tags. To avoid extra callbacks, I moved tag creation functions from tm_source_file_c to this file in</p>
<p><a class="commit-link" data-hovercard-type="commit" data-hovercard-url="https://github.com/geany/geany/commit/266283c9a1aa3aa22bc384e2c61acb61135926fe/hovercard" href="https://github.com/geany/geany/commit/266283c9a1aa3aa22bc384e2c61acb61135926fe"><tt>266283c</tt></a></p>
<p>Other patches from this pull request worth looking at are:</p>
<p><a class="commit-link" data-hovercard-type="commit" data-hovercard-url="https://github.com/geany/geany/commit/2413f2e170bdd6b708cad715de006932bb7e8dd9/hovercard" href="https://github.com/geany/geany/commit/2413f2e170bdd6b708cad715de006932bb7e8dd9"><tt>2413f2e</tt></a><br>
<a class="commit-link" data-hovercard-type="commit" data-hovercard-url="https://github.com/geany/geany/commit/f6139d3b26e3bcfe847f94a9546636c37185fad7/hovercard" href="https://github.com/geany/geany/commit/f6139d3b26e3bcfe847f94a9546636c37185fad7"><tt>f6139d3</tt></a><br>
<a class="commit-link" data-hovercard-type="commit" data-hovercard-url="https://github.com/geany/geany/commit/75c8c6c829f481d0b75c5d70eb1f51b3334234c9/hovercard" href="https://github.com/geany/geany/commit/75c8c6c829f481d0b75c5d70eb1f51b3334234c9"><tt>75c8c6c</tt></a> (and its bugfix <a class="commit-link" data-hovercard-type="commit" data-hovercard-url="https://github.com/geany/geany/commit/5c35fa79575a80498b18500c2d0c8e1904418a91/hovercard" href="https://github.com/geany/geany/commit/5c35fa79575a80498b18500c2d0c8e1904418a91"><tt>5c35fa7</tt></a>)<br>
<a class="commit-link" data-hovercard-type="commit" data-hovercard-url="https://github.com/geany/geany/commit/0182f7d1f0d254388fa9746c98dcb9941f6d488b/hovercard" href="https://github.com/geany/geany/commit/0182f7d1f0d254388fa9746c98dcb9941f6d488b"><tt>0182f7d</tt></a><br>
<a class="commit-link" data-hovercard-type="commit" data-hovercard-url="https://github.com/geany/geany/commit/170b388ec857983c3bba0489633a1c68e13a5227/hovercard" href="https://github.com/geany/geany/commit/170b388ec857983c3bba0489633a1c68e13a5227"><tt>170b388</tt></a></p>
<p>The rest is just intermediate garbage.</p>
<p>I'd suggest that you have a look at what I described above, play with the ActionScript and Cobol regex parsers to see they work and if everything is OK, I'll rebase this pull request this way:</p>
<ol>
<li>I remove the ActionScript and Cobol regex parsers and the related code</li>
<li>I'll make the ctags changes a single commit - basically "sync to upstream ctags version" where I copy-over the ctags main and apply the patch</li>
<li>I'll make a single commit containing all the Geany changes (something like "make Geany compile with the upstream ctags") so it will be easier to see what changes were made on the Geany side</li>
</ol>
<p>Does it sound OK to you?</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/2132?email_source=notifications&email_token=AAIOWJ6ANTGTWEIC44B3UI3PWU5TJA5CNFSM4HIG4G52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV65IUA#issuecomment-494785616">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJYQBG3IJZHBJL36RVTPWU5TJANCNFSM4HIG4G5Q">mute the thread</a>.<img src="https://github.com/notifications/beacon/AAIOWJZH3PS247QTJM5FXDDPWU5TJA5CNFSM4HIG4G52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV65IUA.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/2132?email_source=notifications\u0026email_token=AAIOWJ6ANTGTWEIC44B3UI3PWU5TJA5CNFSM4HIG4G52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV65IUA#issuecomment-494785616",
"url": "https://github.com/geany/geany/pull/2132?email_source=notifications\u0026email_token=AAIOWJ6ANTGTWEIC44B3UI3PWU5TJA5CNFSM4HIG4G52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV65IUA#issuecomment-494785616",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>