SOLL icon indicating copy to clipboard operation
SOLL copied to clipboard

[AST] Enhance Struct Decl

Open LFsWang opened this issue 4 years ago • 4 comments

LFsWang avatar Sep 01 '21 19:09 LFsWang

Codecov Report

Merging #117 (d748632) into master (821b1f6) will decrease coverage by 1.74%. The diff coverage is 61.46%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #117      +/-   ##
==========================================
- Coverage   66.47%   64.73%   -1.75%     
==========================================
  Files          98      106       +8     
  Lines       13679    14095     +416     
  Branches     2041     2162     +121     
==========================================
+ Hits         9093     9124      +31     
- Misses       3567     3888     +321     
- Partials     1019     1083      +64     
Impacted Files Coverage Δ
include/soll/AST/Stmt.h 98.03% <0.00%> (-1.97%) :arrow_down:
include/soll/Sema/Sema.h 90.00% <ø> (ø)
include/soll/Sema/GlobalContent.h 21.42% <21.42%> (ø)
lib/AST/DeclVisitor.cpp 80.85% <33.33%> (-3.60%) :arrow_down:
lib/AST/Expr.cpp 60.21% <33.33%> (-5.38%) :arrow_down:
include/soll/AST/Decl.h 80.58% <47.82%> (-9.55%) :arrow_down:
include/soll/Sema/DeclarationContainer.h 50.00% <50.00%> (ø)
lib/AST/Decl.cpp 67.96% <51.72%> (-3.94%) :arrow_down:
include/soll/Sema/NameAndTypeResolver.h 55.55% <55.55%> (ø)
include/soll/Sema/ReferenceResolver.h 57.14% <57.14%> (ø)
... and 24 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 821b1f6...d748632. Read the comment docs.

codecov[bot] avatar Sep 01 '21 19:09 codecov[bot]

Please rebase to the latest master

hydai avatar Sep 07 '21 09:09 hydai

This pull request introduces 3 alerts when merging c3c0042ee907dbfea8d86e6c40a75b6d94fefb22 into 821b1f64ff622e4e29bb9165ca4eaf4407bdc687 - view on LGTM.com

new alerts:

  • 2 for Declaration hides parameter
  • 1 for Missing return statement

lgtm-com[bot] avatar Sep 09 '21 05:09 lgtm-com[bot]

This pull request introduces 2 alerts when merging 63132918aa69a5349e1d003c2dbd6d3ae81e6b4a into 821b1f64ff622e4e29bb9165ca4eaf4407bdc687 - view on LGTM.com

new alerts:

  • 2 for Declaration hides parameter

lgtm-com[bot] avatar Sep 15 '21 11:09 lgtm-com[bot]