Skip to content
This repository has been archived by the owner on Aug 5, 2022. It is now read-only.

topic/alignment #405

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

topic/alignment #405

wants to merge 3 commits into from

Conversation

ndifolco
Copy link

@ndifolco ndifolco commented May 5, 2017

@plbossart, @sguiriec, @cc6565 please review.

Alignment of internal Android to public tree.

Merging #405 into master will not change coverage.
The diff coverage is 50%.
Impacted file tree graph

@@           Coverage Diff           @@
##           master     #405   +/-   ##
=======================================
Coverage   74.67%   74.67%
=======================================
Files         214      214
Lines        6705     6705
Branches      852      852
=======================================
Hits         5007     5007
Misses       1221     1221
Partials      477      477
Impacted Files Coverage Δ
utility/posix/DynamicLibrary.cpp 80% <50%> (ø) ⬆️

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 965b2af...749e665. Read the comment docs.

  

p-diop and others added 2 commits May 5, 2017 15:49
ComponentTypes can extend one another and include a subelement
by inheritance. Hence the minimum times a sub-sequence can
occur in a componentType shall be 0.

Signed-off-by: Piotr Diop <[email protected]>
Klocwork has reported 2 critical issues in utility/posix/DynamicLibrary.cpp:

Array '(dlError!=nullptr)?dlError:"unknown dlopen error"' of size 21 may use index value(s) 21..INT_MAX
utility/posix/DynamicLibrary.cpp: | DynamicLibrary()
Code: ABV.GENERAL | Severity: Critical(1) | State: Existing | Status: Analyze

Array '(dlError!=nullptr)?dlError:"unknown dlsym error"' of size 20 may use index value(s) 20..INT_MAX
File: utility/posix/DynamicLibrary.cpp: | osGetSymbol()
Code: ABV.GENERAL | Severity: Critical(1) | State: Existing | Status: Analyze

Signed-off-by: Mohamed LAHMADI <[email protected]>
@codecov-io
Copy link

codecov-io commented May 5, 2017

Codecov Report

Merging #405 into master will not change coverage.
The diff coverage is 50%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #405   +/-   ##
=======================================
  Coverage   74.67%   74.67%           
=======================================
  Files         214      214           
  Lines        6705     6705           
  Branches      852      852           
=======================================
  Hits         5007     5007           
  Misses       1221     1221           
  Partials      477      477
Impacted Files Coverage Δ
utility/posix/DynamicLibrary.cpp 80% <50%> (ø) ⬆️

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 965b2af...526d226. Read the comment docs.

".travis.yml" and "appveyor.yml" files are now ignored by "git archive".

Signed-off-by: Di Folco, Neil <[email protected]>
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.

4 participants