Skip to content
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

RSpec: rewrite in C language #3202

Merged
merged 7 commits into from
Dec 19, 2021
Merged

Conversation

masatake
Copy link
Member

No description provided.

@masatake masatake mentioned this pull request Nov 28, 2021
29 tasks
@codecov
Copy link

codecov bot commented Nov 28, 2021

Codecov Report

Merging #3202 (1b09ed7) into master (2d0dda9) will increase coverage by 0.05%.
The diff coverage is 94.07%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3202      +/-   ##
==========================================
+ Coverage   84.95%   85.01%   +0.05%     
==========================================
  Files         206      206              
  Lines       49032    49127      +95     
==========================================
+ Hits        41656    41765     +109     
+ Misses       7376     7362      -14     
Impacted Files Coverage Δ
parsers/itcl.c 92.40% <ø> (ø)
parsers/ldscript.c 86.70% <ø> (-0.05%) ⬇️
parsers/rspec.c 92.20% <92.20%> (ø)
parsers/ruby.c 97.83% <96.29%> (-0.45%) ⬇️
main/entry.c 88.37% <100.00%> (ø)
main/read.c 95.93% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2d0dda9...1b09ed7. Read the comment docs.

@masatake
Copy link
Member Author

I make a pull request only for ecd30f1 first of all.

Signed-off-by: Masatake YAMATO <yamato@redhat.com>
The variable was extern'ed unexpectedly.

Signed-off-by: Masatake YAMATO <yamato@redhat.com>
Signed-off-by: Masatake YAMATO <yamato@redhat.com>
… with the target tag

A language can specify whether the main part does automagic-FQ-tag emission or not
with requestAutomaticFQTag field of parserDefinition.

To access the requestAutomaticFQTag field, the original code calls
getInputLanguage(). However, this didn't work expectedly. When a
subparser made a tag, it was stored to the corkQueue. The corkQueue
was flushed after the subparser run, within the base parser run.
As the result requestAutomaticFQTag field of the base parser was
referred when calling getInputLanguage().

This changes refers the tag itself to access the requestAutomaticFQTag
field instead of calling getInputLanguage().

Signed-off-by: Masatake YAMATO <yamato@redhat.com>
Signed-off-by: Masatake YAMATO <yamato@redhat.com>
The new parser fills the scope and end fields.

Signed-off-by: Masatake YAMATO <yamato@redhat.com>
Signed-off-by: Masatake YAMATO <yamato@redhat.com>
@masatake masatake merged commit 05e6ab4 into universal-ctags:master Dec 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant