-
Notifications
You must be signed in to change notification settings - Fork 6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
rgw: make sure modelines are correct for all files #29742
Conversation
@@ -1,5 +1,5 @@ | |||
// -*- mode:C++; tab-width:8; c-basic-offset:2; indent-tabs-mode:t -*- | |||
// vim: ts=8 sw=2 smarttab | |||
// vim: ts=8 sw=2 smarttab ft=cpp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these modelines came straight from https://github.com/ceph/ceph/blob/master/CodingStyle#L76-L82 so i'd prefer not to change them; why can't vim detect the filetype from the .cc extension?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll update that too
Signed-off-by: Daniel Gryniewicz <dang@redhat.com>
@@ -79,7 +79,7 @@ by section. | |||
header is: | |||
|
|||
// -*- mode:C++; tab-width:8; c-basic-offset:2; indent-tabs-mode:t -*- | |||
// vim: ts=8 sw=2 smarttab | |||
// vim: ts=8 sw=2 smarttab ft=cpp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@liewegas any objection to this change in CodingStyle? the intent is to identify .h files as c++ instead of c, to match the 'mode:C++' for emacs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds like a great idea to me!
Signed-off-by: Daniel Gryniewicz dang@redhat.com
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test docs
jenkins render docs