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

Clean up code using clang-tidy #359

Merged
merged 3 commits into from
Mar 8, 2016
Merged

Clean up code using clang-tidy #359

merged 3 commits into from
Mar 8, 2016

Conversation

terahlunah
Copy link

Using the following checks :

llvm-namespace-comment
modernize-use-nullptr
modernize-use-auto

@@            master    #359   diff @@
======================================
Files          211     211
Stmts         7184    7182     -2
Branches         0       0
Methods          0       0
======================================
- Hit           5769    5767     -2
Partial          0       0
Missed        1415    1415

Review entire Coverage Diff as of a6d55c6

Powered by Codecov. Updated on successful CI builds.

  

Florian Poncabaré added 3 commits March 4, 2016 13:53
Formatting namespaces with clang-tidy
00e3f75 
Using : -checks=llvm-namespace-comment

Signed-off-by: Florian Poncabaré <[email protected]>
Changing NULL to nullptr with clang-tidy
ddb5173 
Using : -checks=modernize-use-nullptr

Signed-off-by: Florian Poncabaré <[email protected]>
Use auto to improve readability
3afc389 
Using : -checks=modernize-use-auto

Signed-off-by: Florian Poncabaré <[email protected]>
@codecov-io
Copy link

Current coverage is 80.29%

Merging #359 into master will decrease coverage by -0.01% as of a6d55c6

@@            master    #359   diff @@
======================================
  Files          211     211       
  Stmts         7184    7182     -2
  Branches         0       0       
  Methods          0       0       
======================================
- Hit           5769    5767     -2
  Partial          0       0       
  Missed        1415    1415

Review entire Coverage Diff as of a6d55c6

Powered by Codecov. Updated on successful CI builds.

@dawagner
Copy link
Contributor

dawagner commented Mar 4, 2016

👍 so far :) Waiting for more.

dawagner added a commit that referenced this pull request Mar 8, 2016
@dawagner
Merge pull request #359 from terahxluna/clang-tidy
e16ba60 
Clean up code using clang-tidy

Checks applied:
 - modernize-use-nullptr
 - llvm-namespace-comment
 - modernize-use-auto
@dawagner dawagner merged commit e16ba60 into intel:master Mar 8, 2016
@terahlunah terahlunah deleted the clang-tidy branch March 8, 2016 09:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@terahlunah @codecov-io @dawagner