Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(268)

Issue 15740044: Update cpplint.py to #242: (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years ago by erg
Modified:
11 years ago
Reviewers:
Mark Mentovai
CC:
Mark Mentovai
Base URL:
http://google-styleguide.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Update cpplint.py to #242: - Check indentation of public/protected/private keywords. - Remove RTTI warning. - Silence warning about multiple inheritance from global namespace. - Copy ctors don't need "explicit". - Understand "const char* const&" as a const reference. - Remove runtime/sizeof. - Recognize the "catch" keyword. - List C++11 headers - Allow sscanf() - Allow for one extra level of nesting in template class decls. - False positive for semicolons after single-line nameless unions. R=mark@chromium.org Committed: https://code.google.com/p/google-styleguide/source/detail?r=113

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+582 lines, -224 lines) Patch
cpplint/cpplint.py View 24 chunks +294 lines, -97 lines 0 comments Download
cpplint/cpplint_unittest.py View 26 chunks +288 lines, -127 lines 0 comments Download

Messages

Total messages: 3
erg
First in a series to try to clear the backlog. Trying to split these up ...
11 years ago (2013-10-23 00:21:04 UTC) #1
Mark Mentovai
LGTM
11 years ago (2013-10-25 13:06:49 UTC) #2
erg
11 years ago (2013-10-25 17:39:49 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r113 (presubmit successful).
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b