<p dir="auto">Parsers for R and verilog from ctags are token-based instead of our line-based parsers and are definitely superior and all development should go towards these parsers.</p>
<p dir="auto">The sh parser is not token based but the uctags version seems to handle more things.</p>
<p dir="auto">Other 2 parsers which I think we should eventually adopt are VHDL and tex parsers (both are token-based) - they generate a slightly different set of tags and tag mappings should be adopted to those for the sidebar so I skipped them for now.</p>
<p dir="auto">Finally, I tried also switching to uctags <code>c.c</code> for java, D and C# but our c.c seems to be actually better there.</p>
<hr>
<h4>You can view, comment on, or merge this pull request online at:</h4>
<p> <a href='https://github.com/geany/geany/pull/3035'>https://github.com/geany/geany/pull/3035</a></p>
<h4>Commit Summary</h4>
<ul>
<li><a href="https://github.com/geany/geany/pull/3035/commits/cf7f9b3c1419e8423c59db7c8ef255e8d9e416a4" class="commit-link">cf7f9b3</a> Use uctags R parser</li>
<li><a href="https://github.com/geany/geany/pull/3035/commits/e4416f211e3d2944af42aa83c456cd6927e6d045" class="commit-link">e4416f2</a> Update R kind mappings</li>
<li><a href="https://github.com/geany/geany/pull/3035/commits/ceddba0ba705b3b5f0b00d804ff494f26f920b1a" class="commit-link">ceddba0</a> Use uctags parser for sh</li>
<li><a href="https://github.com/geany/geany/pull/3035/commits/55b1668f09e7d20212ee37f9f84d6c62a11a29e3" class="commit-link">55b1668</a> Update mappings for sh parser</li>
<li><a href="https://github.com/geany/geany/pull/3035/commits/433d8dc329d9b73d502c74157fac9030c85fd812" class="commit-link">433d8dc</a> Use uctags verilog parser</li>
<li><a href="https://github.com/geany/geany/pull/3035/commits/13d389df6ac323c6fd156c91773b58993d3f749a" class="commit-link">13d389d</a> Update mappings for verilog parser</li>
<li><a href="https://github.com/geany/geany/pull/3035/commits/205c7d30e90444278b1f33a3cba2a0cb3687827a" class="commit-link">205c7d3</a> Update verilog unit tests</li>
<li><a href="https://github.com/geany/geany/pull/3035/commits/81d6afd09e92504ddfb6dbe44bec5b68b93dce1e" class="commit-link">81d6afd</a> Add R and Verilog to languages reporting context</li>
</ul>
<h4 style="display: inline-block">File Changes </h4> <p style="display: inline-block">(<a href="https://github.com/geany/geany/pull/3035/files">11 files</a>)</p>
<ul>
<li>
<strong>M</strong>
<a href="https://github.com/geany/geany/pull/3035/files#diff-e0ce68205f4ca660762ddbc1f13f556330aea31c7de950df4da1d4ce6f95e6d6">ctags/Makefile.am</a>
(7)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany/pull/3035/files#diff-df4a70fa42e6f58ce9dfe30f6cf1357cebdbf96e03192c917fa05e5da851eab3">ctags/parsers/geany_r.c</a>
(177)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany/pull/3035/files#diff-b472afebbb049c222e0d9cf3af9b5b783306ddb4ce488b37ffe851490ad41706">ctags/parsers/geany_sh.c</a>
(110)
</li>
<li>
<strong>D</strong>
<a href="https://github.com/geany/geany/pull/3035/files#diff-3e93c21e1382f86c4808f31747c2f195e45e9f056cfff64f1b0b040f2d7e2f50">ctags/parsers/geany_verilog.c</a>
(332)
</li>
<li>
<strong>A</strong>
<a href="https://github.com/geany/geany/pull/3035/files#diff-2d189145870bda73e2570aae700be88080c6c402c3075c87aba997d3ef2b2041">ctags/parsers/r.c</a>
(1470)
</li>
<li>
<strong>A</strong>
<a href="https://github.com/geany/geany/pull/3035/files#diff-1d3a6814c9328d78a828169b39b6312af8b918a17667028d0e61696abe3b51a3">ctags/parsers/r.h</a>
(111)
</li>
<li>
<strong>A</strong>
<a href="https://github.com/geany/geany/pull/3035/files#diff-9e5d480144184540c4e34835f82eb4b035c179d2b7ed5767a0cca68b86d56b4f">ctags/parsers/sh.c</a>
(490)
</li>
<li>
<strong>A</strong>
<a href="https://github.com/geany/geany/pull/3035/files#diff-e4b4f7350e176aefdb5d874f17fbc1e64b7376aa332f97bef559721a4992c31e">ctags/parsers/verilog.c</a>
(2024)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/geany/geany/pull/3035/files#diff-52c3ba1a920ce03ae524e393cf4d0ff582b13d9eafef29393f9841c19adf31f8">src/tagmanager/tm_parser.c</a>
(14)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/geany/geany/pull/3035/files#diff-4e85656d913413fbbdb9da7cbbcf9048f1c86e4c738b8be315fdf455831249d3">tests/ctags/bug1111214-j-chan.v.tags</a>
(4)
</li>
<li>
<strong>M</strong>
<a href="https://github.com/geany/geany/pull/3035/files#diff-684fc3428ce6768cb9748e1ca5eaba862da4fa6aa1f80651c60589998aefedea">tests/ctags/traffic_signal.v.tags</a>
(24)
</li>
</ul>
<h4>Patch Links:</h4>
<ul>
<li><a href='https://github.com/geany/geany/pull/3035.patch'>https://github.com/geany/geany/pull/3035.patch</a></li>
<li><a href='https://github.com/geany/geany/pull/3035.diff'>https://github.com/geany/geany/pull/3035.diff</a></li>
</ul>
<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/3035">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ7LGBST7S3KBDB3BV3UOTLVXANCNFSM5JB5KPRQ">unsubscribe</a>.<br />Triage notifications on the go with GitHub Mobile for <a href="https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675">iOS</a> or <a href="https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub">Android</a>.
<img src="https://github.com/notifications/beacon/AAIOWJ7P3GBOKJ5C76NRA5DUOTLVXA5CNFSM5JB5KPR2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4P45PALA.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/3035",
"url": "https://github.com/geany/geany/pull/3035",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>