[webkit-reviews] review granted: [Bug 28370] check-webkit-style: add check for 'using std::foo; ' statements : [Attachment 34940] Patch v1

[bugzilla-daemon at webkit.org]

David Levin <levin at chromium.org> has granted David Kilzer (ddkilzer)
<ddkilzer at webkit.org>'s request for review:
Bug 28370: check-webkit-style: add check for 'using std::foo;' statements
https://bugs.webkit.org/show_bug.cgi?id=28370

Attachment 34940: Patch v1
https://bugs.webkit.org/attachment.cgi?id=34940&action=review

------- Additional Comments from David Levin <levin at chromium.org>
> +	     'Use \'using namespace std;\' instead of \'using std::%s;\'.' %
method_name)

Consider using "" for the outer quotes to avoid the \' because it makes it
easier to read. Like this:

	  "Use 'using namespace std;' instead of 'using std::%s;'." %
method_name)


> diff --git a/WebKitTools/Scripts/modules/cpp_style_unittest.py
b/WebKitTools/Scripts/modules/cpp_style_unittest.py
> +	       'Use \'using namespace std;\' instead of \'using std::min;\'.'

Consider using "" for the outer quotes to avoid the \'.