tools/pretty_gyp.py - Issue 10399131: Fixing pylint. - Code Review

Chromium Code Reviews

chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out

(480)

My Issues | Starred Open | Closed | All

Unified Diff: tools/pretty_gyp.py

Issue 10399131: Fixing pylint. (Closed) Base URL: http://gyp.googlecode.com/svn/trunk/

Patch Set: Created 8 years, 7 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

« no previous file with comments | « test/variables/commands/gyptest-commands-repeated.py ('k') | tools/pretty_sln.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: tools/pretty_gyp.py

===================================================================

--- tools/pretty_gyp.py (revision 1390)

+++ tools/pretty_gyp.py (working copy)

@@ -1,5 +1,7 @@

#!/usr/bin/env python

-# Copyright (c) 2011 The Chromium Authors. All rights reserved.

+#!/usr/bin/env python

Nico 2012/05/23 16:45:01 This looks wrong.

+

+# Copyright (c) 2012 Google Inc. All rights reserved.

# Use of this source code is governed by a BSD-style license that can be

# found in the LICENSE file.

@@ -16,7 +18,7 @@

# It takes into account quoted quotes, and makes sure that the quotes match.

# NOTE: It does not handle quotes that span more than one line, or

# cases where an escaped quote is preceeded by an escaped backslash.

-quote_re_str = r'(?P<q>[\'"])(.*?)(?<![^\\][\\])(?P=q)'

+QUOTE_RE_STR = r'(?P<q>[\'"])(.*?)(?<![^\\][\\])(?P=q)'

QUOTE_RE = re.compile(QUOTE_RE_STR)

Property changes on: tools/pretty_gyp.py

___________________________________________________________________

Added: svn:eol-style

+ LF

« no previous file with comments | « test/variables/commands/gyptest-commands-repeated.py ('k') | tools/pretty_sln.py » ('j') | no next file with comments »

Powered by Google App Engine

This is Rietveld 408576698