Skip to content

Conversation

@masatake
Copy link
Member

Close #3382

Signed-off-by: Masatake YAMATO yamato@redhat.com

@masatake masatake requested a review from pragmaware May 12, 2022 18:51
@codecov
Copy link

codecov bot commented May 12, 2022

Codecov Report

Merging #3383 (f05982e) into master (3873e70) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #3383      +/-   ##
==========================================
- Coverage   83.21%   83.20%   -0.01%     
==========================================
  Files         218      218              
  Lines       52392    52393       +1     
==========================================
  Hits        43596    43596              
- Misses       8796     8797       +1     
Impacted Files Coverage Δ
parsers/cxx/cxx_parser_tokenizer.c 92.48% <100.00%> (+0.02%) ⬆️
parsers/cpreprocessor.c 95.21% <0.00%> (-0.10%) ⬇️

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 3873e70...f05982e. Read the comment docs.

@masatake
Copy link
Member Author

Failures unrelated to this change are reported on the ubi8 container, a platform I cannot ignore.

masatake added 2 commits May 13, 2022 08:49
Though iconv command is available, there is a case
that encodings used in test cases are not available.

e.g. the ubi8 container doesn't provide shift_jis
encoding unless installing glibc-langpack-ja package.

Signed-off-by: Masatake YAMATO <yamato@redhat.com>
Close universal-ctags#3382

Signed-off-by: Masatake YAMATO <yamato@redhat.com>
@masatake masatake force-pushed the cxx--skip-declspec branch from 5fd2ae5 to f05982e Compare May 12, 2022 23:50
@masatake
Copy link
Member Author

The change for fixing is enough compact and self-closed. Even when @pragmaware returns and says, "I don't like this fix," we can revert this commit quickly.

I don't like this change because it throws away the information found in the source code in the early stage of parsing. The change makes it impossible to use the information in the upper stages of parsing and/or to store the information in fields of tag entries. However, this is the same as __attribute__().

@masatake masatake merged commit 62bf9fc into universal-ctags:master May 13, 2022
kraj pushed a commit to YoeDistro/meta-openembedded that referenced this pull request May 23, 2022
Changelog:
==========
Merge pull request #3383(universal-ctags/ctags#3383)

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
kraj pushed a commit to YoeDistro/meta-openembedded that referenced this pull request May 23, 2022
Changelog:
==========
Merge pull request #3383(universal-ctags/ctags#3383)

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
kraj pushed a commit to YoeDistro/meta-openembedded that referenced this pull request May 24, 2022
Changelog:
==========
Merge pull request #3383(universal-ctags/ctags#3383)

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
Signed-off-by: Khem Raj <raj.khem@gmail.com>
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.

C/C++: problem with __declspec in the parser

1 participant