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

Suppress warning #3714

Merged
merged 11 commits into from
May 12, 2023
Merged

Suppress warning #3714

merged 11 commits into from
May 12, 2023

Conversation

masatake
Copy link
Member

@masatake masatake commented May 8, 2023

No description provided.

Signed-off-by: Masatake YAMATO <yamato@redhat.com>
Signed-off-by: Masatake YAMATO <yamato@redhat.com>
@masatake
Copy link
Member Author

masatake commented May 9, 2023

.\main\mio.c(668): warning C4013: '_chsize_s' undefined; assuming extern returning int [C:\projects\ctags\win32\ctags_vs2013.vcxproj]
..\main\unwindi.c(294): warning C4018: '>=' : signed/unsigned mismatch [C:\projects\ctags\win32\ctags_vs2013.vcxproj]
..\parsers\sql.c(838): warning C4018: '>' : signed/unsigned mismatch [C:\projects\ctags\win32\ctags_vs2013.vcxproj]
..\parsers\verilog.c(885): warning C4018: '<' : signed/unsigned mismatch [C:\projects\ctags\win32\ctags_vs2013.vcxproj]

Signed-off-by: Masatake YAMATO <yamato@redhat.com>
Signed-off-by: Masatake YAMATO <yamato@redhat.com>
Signed-off-by: Masatake YAMATO <yamato@redhat.com>
Signed-off-by: Masatake YAMATO <yamato@redhat.com>
Signed-off-by: Masatake YAMATO <yamato@redhat.com>
…tion"

Signed-off-by: Masatake YAMATO <yamato@redhat.com>
…ction "

Signed-off-by: Masatake YAMATO <yamato@redhat.com>
…r 'l' in format [-Wformat=]"

Signed-off-by: Masatake YAMATO <yamato@redhat.com>
@codecov
Copy link

codecov bot commented May 9, 2023

Codecov Report

Patch coverage: 91.30% and project coverage change: -0.01 ⚠️

Comparison is base (6b06565) 82.99% compared to head (fd601e0) 82.99%.

❗ Current head fd601e0 differs from pull request most recent head ceda2dc. Consider uploading reports for the commit ceda2dc to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3714      +/-   ##
==========================================
- Coverage   82.99%   82.99%   -0.01%     
==========================================
  Files         226      226              
  Lines       54972    54993      +21     
==========================================
+ Hits        45626    45642      +16     
- Misses       9346     9351       +5     
Impacted Files Coverage Δ
main/lregex.c 82.87% <73.07%> (-0.23%) ⬇️
main/field.c 92.60% <94.11%> (-0.14%) ⬇️
dsl/es.c 54.81% <100.00%> (-0.04%) ⬇️
dsl/optscript.c 88.13% <100.00%> (+0.13%) ⬆️
dsl/sorter.c 64.82% <100.00%> (+0.24%) ⬆️
main/entry.c 86.75% <100.00%> (+0.01%) ⬆️
main/fmt.c 86.74% <100.00%> (ø)
main/keyword.c 78.67% <100.00%> (ø)
main/options.c 83.83% <100.00%> (-0.02%) ⬇️
main/parse.c 95.60% <100.00%> (ø)
... and 5 more

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@masatake
Copy link
Member Author

masatake commented May 9, 2023

I should make another pull request for ceac68d...fd601e0.
I will drop these three commits when merging this pull request.

@masatake masatake modified the milestones: 6.2, 6.1 May 12, 2023
@masatake
Copy link
Member Author

..\main\mio.c(668): warning C4013: '_chsize_s' undefined; assuming extern returning int [C:\projects\ctags\win32\ctags_vs2013.vcxproj]
..\main\unwindi.c(294): warning C4018: '>=' : signed/unsigned mismatch [C:\projects\ctags\win32\ctags_vs2013.vcxproj]
..\parsers\sql.c(838): warning C4018: '>' : signed/unsigned mismatch [C:\projects\ctags\win32\ctags_vs2013.vcxproj]
..\parsers\verilog.c(885): warning C4018: '<' : signed/unsigned mismatch [C:\projects\ctags\win32\ctags_vs2013.vcxproj]

@masatake masatake merged commit e1abef6 into universal-ctags:master May 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant