Skip to content

Add wildcards to reduce IDEA warnings#63

Merged
wenshao merged 1 commit intoalibaba:mainfrom
kraity:main
Apr 23, 2022
Merged

Add wildcards to reduce IDEA warnings#63
wenshao merged 1 commit intoalibaba:mainfrom
kraity:main

Conversation

@kraity
Copy link
Copy Markdown
Collaborator

@kraity kraity commented Apr 23, 2022

JSON Class 代码中泛型增加通配符, 减少IDEA警告

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 23, 2022

Codecov Report

Merging #63 (db7dac1) into main (a27fb2d) will increase coverage by 0.00%.
The diff coverage is 88.46%.

@@            Coverage Diff            @@
##               main      #63   +/-   ##
=========================================
  Coverage     64.91%   64.92%           
- Complexity     8061     8062    +1     
=========================================
  Files           420      420           
  Lines         45443    45442    -1     
  Branches       9168     9170    +2     
=========================================
+ Hits          29499    29501    +2     
+ Misses        12389    12384    -5     
- Partials       3555     3557    +2     
Impacted Files Coverage Δ
...rc/main/java/com/alibaba/fastjson2/JSONWriter.java 73.12% <ø> (ø)
...alibaba/fastjson2/reader/ObjectReaderImplEnum.java 72.50% <ø> (-1.31%) ⬇️
...ibaba/fastjson2/writer/ObjectWriterBaseModule.java 79.16% <0.00%> (-0.24%) ⬇️
.../alibaba/fastjson2/writer/ObjectWriterCreator.java 78.21% <0.00%> (-0.62%) ⬇️
...ibaba/fastjson2/writer/ObjectWriterCreatorASM.java 96.49% <0.00%> (-0.12%) ⬇️
core/src/main/java/com/alibaba/fastjson2/JSON.java 85.18% <97.87%> (-0.79%) ⬇️
...ain/java/com/alibaba/fastjson2/JSONReaderUTF8.java 72.35% <0.00%> (+0.01%) ⬆️
...rc/main/java/com/alibaba/fastjson2/JSONReader.java 73.40% <0.00%> (+0.10%) ⬆️
...in/java/com/alibaba/fastjson2/JSONWriterJSONB.java 57.62% <0.00%> (+0.27%) ⬆️
... and 2 more

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 8589ec0...db7dac1. Read the comment docs.

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.

3 participants