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

Conversation

@tcahuzax
Copy link
Contributor

@tcahuzax tcahuzax commented Dec 4, 2015


   

Copy link
Contributor

Choose a reason for hiding this comment

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

Wrongfully aligned comment.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

This patch introduces a clang-format configuration
file that matches team coding style.

Signed-off-by: Thomas Cahuzac <[email protected]>
@tcahuzax tcahuzax changed the title Using clang-format to format code [Review] Using clang-format to format code Dec 9, 2015
@tcahuzax
Copy link
Contributor Author

tcahuzax commented Dec 9, 2015

👍
@dawagner @krocard Plz review

@krocard
Copy link
Contributor

krocard commented Dec 9, 2015

:shipit:

Some pieces of code were not formatted
properly by clang-format.

This patch changes the code to help
formatting.

Signed-off-by: Thomas Cahuzac <[email protected]>
Some 'Catch' macros are treated as control
instructions, but in fact they are not.

This patch removes them.

Signed-off-by: Thomas Cahuzac <[email protected]>
@tcahuzax
Copy link
Contributor Author

tcahuzax commented Dec 9, 2015

👍
@dawagner @krocard Plz review

@dawagner
Copy link
Contributor

dawagner commented Dec 9, 2015

:shipit:

krocard added a commit that referenced this pull request Dec 9, 2015
Using clang-format to format code
@krocard krocard merged commit cf68468 into intel:master Dec 9, 2015
@krocard krocard changed the title [Review] Using clang-format to format code Using clang-format to format code Dec 29, 2015
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