Skip to content

Conversation

@ulfalizer
Copy link
Contributor

Bool symbols implicitly default to 'n'.

A 'default n' can make sense e.g. in a Kconfig.defconfig file, if you
want to override a 'default y' on the base definition of the symbol. It
isn't used like that on any of these symbols though.

Bool symbols implicitly default to 'n'.

A 'default n' can make sense e.g. in a Kconfig.defconfig file, if you
want to override a 'default y' on the base definition of the symbol. It
isn't used like that on any of these symbols though.

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

Codecov Report

Merging #8578 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8578      +/-   ##
==========================================
- Coverage   63.97%   63.96%   -0.01%     
==========================================
  Files         429      429              
  Lines       41290    41290              
  Branches     6957     6957              
==========================================
- Hits        26414    26413       -1     
- Misses      11719    11720       +1     
  Partials     3157     3157
Impacted Files Coverage Δ
samples/philosophers/src/main.c 96.87% <0%> (-1.57%) ⬇️

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 80e02a9...1e43a7b. Read the comment docs.

@MaureenHelm MaureenHelm merged commit 19d7803 into zephyrproject-rtos:master Jun 27, 2018
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.

3 participants