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

Conversation

@cc6565
Copy link
Contributor

@cc6565 cc6565 commented Apr 6, 2017

Use correct blackboard index while calling dumpElement (logValue
for ArrayParameter class) to fix the eratic behavior of dumpElement
for ArrayParameters.

Change-Id: I255af6969283f91cdd2efd810c7d453f3765a16d
Tracked-On:
Signed-off-by: François Gaffie [email protected]

Merging #403 into master will increase coverage by <.01%.
The diff coverage is 0%.
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #403      +/-   ##
==========================================
+ Coverage    74.6%   74.61%   +<.01%
==========================================
Files         214      214
Lines        6675     6677       +2
Branches      845      845
==========================================
+ Hits         4980     4982       +2
Misses       1224     1224
Partials      471      471
Impacted Files Coverage Δ
parameter/ArrayParameter.cpp 66% <0%> (ø) ⬆️
utility/NonCopyable.hpp 100% <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 e678467...d4ac41d. Read the comment docs.

  

Use correct blackboard index while calling dumpElement (logValue
for ArrayParameter class) to fix the eratic behavior of dumpElement
for ArrayParameters.

Change-Id: I255af6969283f91cdd2efd810c7d453f3765a16d
Tracked-On:
Signed-off-by: François Gaffie <[email protected]>
@codecov-io
Copy link

codecov-io commented Apr 6, 2017

Codecov Report

Merging #403 into master will increase coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #403      +/-   ##
==========================================
+ Coverage    74.6%   74.61%   +<.01%     
==========================================
  Files         214      214              
  Lines        6675     6677       +2     
  Branches      845      845              
==========================================
+ Hits         4980     4982       +2     
  Misses       1224     1224              
  Partials      471      471
Impacted Files Coverage Δ
parameter/ArrayParameter.cpp 66% <0%> (ø) ⬆️
utility/NonCopyable.hpp 100% <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 e678467...d4ac41d. Read the comment docs.

@sguiriec
Copy link

sguiriec commented Apr 6, 2017

@cc6565 without test case it cannot be merged. Please refer to the next new pull request
#404

@sguiriec sguiriec closed this Apr 6, 2017
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