[go: nahoru, domu]

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

improve performance when using BracePair #48

Merged
merged 5 commits into from
May 2, 2018
Merged

Conversation

izhangzhihao
Copy link
Owner

No description provided.

@izhangzhihao izhangzhihao self-assigned this May 1, 2018
@codecov
Copy link
codecov bot commented May 1, 2018

Codecov Report

Merging #48 into IC-2017.2 will increase coverage by 0.36%.
The diff coverage is 75%.

Impacted file tree graph

@@               Coverage Diff               @@
##             IC-2017.2      #48      +/-   ##
===============================================
+ Coverage        74.82%   75.18%   +0.36%     
+ Complexity         120      119       -1     
===============================================
  Files               14       14              
  Lines              282      270      -12     
  Branches            73       69       -4     
===============================================
- Hits               211      203       -8     
+ Misses              11        9       -2     
+ Partials            60       58       -2
Impacted Files Coverage Δ Complexity Δ
.../rainbow/brackets/visitor/DefaultRainbowVisitor.kt 70.73% <66.66%> (+0.55%) 6 <0> (-1) ⬇️
...github/izhangzhihao/rainbow/brackets/BracePairs.kt 76% <83.33%> (-0.2%) 3 <0> (ø)

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 e84816b...2136fad. Read the comment docs.

@YiiGuxing YiiGuxing merged commit a9b531f into IC-2017.2 May 2, 2018
@izhangzhihao izhangzhihao deleted the performance-improve branch May 15, 2018 02:29
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.

None yet

2 participants