Skip to content

Style fixes - #505

Merged
justinemarie merged 1 commit into
NetSys:masterfrom
vivi:clang
Jun 7, 2017
Merged

Style fixes#505
justinemarie merged 1 commit into
NetSys:masterfrom
vivi:clang

Conversation

@vivi

@vivi vivi commented Jun 7, 2017

Copy link
Copy Markdown
Contributor

Ran clang-format on all module and core files using our
.clang-format file.

Ran `clang-format` on all module and core files using our
`.clang-format` file.
@justinemarie

Copy link
Copy Markdown
Contributor

Looks good -- need to wait for Travis to merge.

@codecov

codecov Bot commented Jun 7, 2017

Copy link
Copy Markdown

Codecov Report

Merging #505 into master will decrease coverage by <1%.
The diff coverage is 79%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #505    +/-   ##
=======================================
- Coverage      64%     64%   -<1%     
=======================================
  Files         198     198            
  Lines       12071   12054    -17     
=======================================
- Hits         7766    7752    -14     
+ Misses       4305    4302     -3
Impacted Files Coverage Δ
core/modules/nat.cc 70% <ø> (ø) ⬆️
core/opts.cc 82% <ø> (ø) ⬆️
core/module.h 88% <ø> (ø) ⬆️
core/bessctl.h 100% <ø> (ø) ⬆️
core/modules/arp_responder.h 100% <ø> (ø) ⬆️
core/modules/noop.cc 13% <0%> (ø) ⬆️
core/modules/queue_inc.cc 5% <0%> (ø) ⬆️
core/dpdk.cc 78% <0%> (ø) ⬆️
core/modules/port_inc.cc 5% <0%> (ø) ⬆️
core/traffic_class.cc 91% <100%> (ø) ⬇️
... and 10 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 b75a71a...a29be9e. Read the comment docs.

@justinemarie
justinemarie merged commit 7b5d7a7 into NetSys:master Jun 7, 2017
@justinemarie

Copy link
Copy Markdown
Contributor

Thank you!
image

justinemarie pushed a commit that referenced this pull request Jun 9, 2017
Ran `clang-format` on all module and core files using our
`.clang-format` file.
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.

2 participants