-
Notifications
You must be signed in to change notification settings - Fork 620
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pascal: extract type #3212
base: master
Are you sure you want to change the base?
Pascal: extract type #3212
Conversation
@ntrel reviewed the case in universal-ctags#2241. Thank you.
Close universal-ctags#2241. @mastake's note: This pull chnage is based on universal-ctags#2241 submitted by @ntrel. I adjust the change not to fill "signature" field for tags of "type" kind. I also adjust the change to avoid regression.
Codecov Report
@@ Coverage Diff @@
## master #3212 +/- ##
=======================================
Coverage 84.94% 84.95%
=======================================
Files 206 206
Lines 49030 49044 +14
=======================================
+ Hits 41650 41665 +15
+ Misses 7380 7379 -1
Continue to review full report at Codecov.
|
{ | ||
if (tail ("type")) /* check for type declaration */ | ||
{ | ||
found_tag = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These lines are not covered by the test cases.
If more than two types are defined under the https://www.gnu-pascal.de/gpc/Type-Declaration.html
|
This is mostly based on #2241 submitted by @ntrel.
I adjusted the changes to fit the latest codebase.