Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

fix issue 2638[解析非法字符串JSON.parseObject("}",JaveBean.class ) 没有抛出异常,测试"] " 可以抛出] #2641

Merged
merged 2 commits into from
Sep 4, 2019

Conversation

JacceYang
Copy link
Contributor

No description provided.

@codecov-io
Copy link

Codecov Report

Merging #2641 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #2641      +/-   ##
============================================
- Coverage     81.26%   81.24%   -0.03%     
+ Complexity     7031     7027       -4     
============================================
  Files           143      143              
  Lines         26246    26248       +2     
  Branches       6274     6275       +1     
============================================
- Hits          21330    21324       -6     
- Misses         3105     3110       +5     
- Partials       1811     1814       +3
Impacted Files Coverage Δ Complexity Δ
...com/alibaba/fastjson/parser/DefaultJSONParser.java 88.09% <0%> (-0.19%) 314 <0> (+1)
...ava/com/alibaba/fastjson/serializer/DateCodec.java 87.58% <0%> (-1.97%) 37% <0%> (-1%)
...n/java/com/alibaba/fastjson/util/JavaBeanInfo.java 87.65% <0%> (-0.19%) 194% <0%> (-1%)
...java/com/alibaba/fastjson/parser/ParserConfig.java 82.66% <0%> (-0.18%) 197% <0%> (-1%)
...ava/com/alibaba/fastjson/parser/JSONLexerBase.java 71.25% <0%> (-0.04%) 807% <0%> (-1%)

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 c4ccd8b...92c9e4a. Read the comment docs.

@JacceYang JacceYang changed the title fix issue 2638[解析非法字符串"}" 没有抛出异常,测试"] " 可以抛出] fix issue 2638[解析非法字符串JSON.parseObject("}",JaveBean.class ) 没有抛出异常,测试"] " 可以抛出] Aug 17, 2019
@wenshao wenshao merged commit 35ed2f1 into alibaba:master Sep 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants