Skip to content

Conversation

@mped-oticon
Copy link
Contributor

Adds documentation-commentary to some infrastructure used by the LL.
It is a long-term effort to better document the LL.

Notably, ticker and mayfly require more documentation; this will be
done later.

@mped-oticon mped-oticon self-assigned this Dec 18, 2018
@mped-oticon mped-oticon requested a review from cvinayak December 18, 2018 15:22
@codecov-io
Copy link

codecov-io commented Dec 18, 2018

Codecov Report

Merging #12167 into master will not change coverage.
The diff coverage is 66.66%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #12167   +/-   ##
=======================================
  Coverage   48.27%   48.27%           
=======================================
  Files         295      295           
  Lines       44282    44282           
  Branches    10601    10601           
=======================================
  Hits        21377    21377           
  Misses      18636    18636           
  Partials     4269     4269
Impacted Files Coverage Δ
subsys/bluetooth/controller/util/util.c 100% <ø> (ø) ⬆️
subsys/bluetooth/controller/util/mem.c 53.94% <ø> (ø) ⬆️
subsys/bluetooth/controller/util/memq.c 69.23% <66.66%> (ø) ⬆️

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 c716f9c...d90979f. Read the comment docs.

@cvinayak
Copy link
Contributor

recheck

@cvinayak
Copy link
Contributor

Documentation looks good to me. Will wait for the CI to succeed.

Copy link
Member

@carlescufi carlescufi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice job.

@carlescufi
Copy link
Member

@mped-oticon can you please rebase?

Adds documentation-commentary to some infrastructure used by the LL.
It is a long-term effort to better document the LL.

Notably ticker and mayfly require more documentation; this will be
done later.

Signed-off-by: Mark Ruvald Pedersen <[email protected]>
@mped-oticon
Copy link
Contributor Author

@mped-oticon can you please rebase?

Done

@carlescufi carlescufi merged commit 076fd6a into zephyrproject-rtos:master Jan 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants