refactoring to range loop #2583
Conversation
OgreMain/src/OgreCamera.cpp
Outdated
| @@ -174,7 +171,7 @@ namespace Ogre { | |||
|
|
|||
| Quaternion targetWorldOrientation; | |||
|
|
|||
| if( mYawFixed ) | |||
There was a problem hiding this comment.
I would prefer not to do these whitespace changes manually. They are trivial to do with clang-format. I just refrained to apply them as such commit introduces lots of noise in the code (think of open pull requests) without adding much.
There was a problem hiding this comment.
is there a .clang-format ready to use somewhere or we can use llvm with 4 space of indentation ?
There was a problem hiding this comment.
added the clang-format I use to #2584. But still, I do not think we should apply it to existing code just like that..
There was a problem hiding this comment.
applied a quite similar format than your, I think it looks good
|
Ok I got if that keeps similar the actual style would be worth to make it, anyway I am reverting or creating a new commit. |
No description provided.