diff options
| author | Simon Hausmann <simon.hausmann@nokia.com> | 2012-06-01 22:05:29 +0200 |
|---|---|---|
| committer | Simon Hausmann <simon.hausmann@nokia.com> | 2012-06-01 22:05:29 +0200 |
| commit | 0bb233d1f7c4317bb772d72cc3f92cbcdb4027de (patch) | |
| tree | 47b40aab26fdaa15ec446b7d6de9bdf7efd7bbbb /Tools/Scripts/webkitpy/common/checkout/changelog.py | |
| parent | b1e9e47fa11f608ae16bc07f97a2acf95bf80272 (diff) | |
| download | qtwebkit-0bb233d1f7c4317bb772d72cc3f92cbcdb4027de.tar.gz | |
Imported WebKit commit f2da9451cbccb8b7921d55483aa0bc656ff9cf53 (http://svn.webkit.org/repository/webkit/trunk@119269)
New snapshot that includes fix for using -Werror only in developer builds
Diffstat (limited to 'Tools/Scripts/webkitpy/common/checkout/changelog.py')
| -rw-r--r-- | Tools/Scripts/webkitpy/common/checkout/changelog.py | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/Tools/Scripts/webkitpy/common/checkout/changelog.py b/Tools/Scripts/webkitpy/common/checkout/changelog.py index 3cfe25f80..f30fd2c2d 100644 --- a/Tools/Scripts/webkitpy/common/checkout/changelog.py +++ b/Tools/Scripts/webkitpy/common/checkout/changelog.py @@ -72,7 +72,6 @@ class ChangeLogEntry(object): reviewer_name_noise_regexp = re.compile(r""" (\s+((tweaked\s+)?and\s+)?(landed|committed|okayed)\s+by.+) # "landed by", "commented by", etc... |(^(Reviewed\s+)?by\s+) # extra "Reviewed by" or "by" - |\.(?:(\s.+|$)) # text after the first period followed by a space |([(<]\s*[\w_\-\.]+@[\w_\-\.]+[>)]) # email addresses |([(<](https?://?bugs.)webkit.org[^>)]+[>)]) # bug url |("[^"]+") # wresler names like 'Sean/Shawn/Shaun' in 'Geoffrey "Sean/Shawn/Shaun" Garen' @@ -86,6 +85,10 @@ class ChangeLogEntry(object): |(with(\s+[a-z-]+)+) # phrases with "with no hesitation" in "Sam Weinig with no hesitation" """, re.VERBOSE) + reviewer_name_noise_needing_a_backreference_regexp = re.compile(r""" + (\S\S)\.(?:(\s.+|$)) # Text after the two word characters (don't match initials) and a period followed by a space. + """, re.IGNORECASE | re.VERBOSE) + nobody_regexp = re.compile(r"""(\s+|^)nobody( ((,|\s+-)?\s+(\w+\s+)+fix.*) # e.g. nobody, build fix... |(\s*\([^)]+\).*) # NOBODY (..)... @@ -118,6 +121,7 @@ class ChangeLogEntry(object): reviewer_text = ChangeLogEntry.nobody_regexp.sub('', reviewer_text) reviewer_text = ChangeLogEntry.reviewer_name_noise_regexp.sub('', reviewer_text) reviewer_text = ChangeLogEntry.reviewer_name_casesensitive_noise_regexp.sub('', reviewer_text) + reviewer_text = ChangeLogEntry.reviewer_name_noise_needing_a_backreference_regexp.sub(r'\1', reviewer_text) reviewer_text = reviewer_text.replace('(', '').replace(')', '') reviewer_text = re.sub(r'\s\s+|[,.]\s*$', ' ', reviewer_text).strip() if not len(reviewer_text): |