PTAL at patch 2.
https://codereview.chromium.org/65933003/diff/1/tools/push-to-trunk/common_includes.py
File tools/push-to-trunk/common_includes.py (right):
https://codereview.chromium.org/65933003/diff/1/tools/push-to-trunk/common_includes.py#newcode102
tools/push-to-trunk/common_includes.py:102: out = filter(lambda x:
re.search(r"^[ \t]*BUG[ \t]*=", x), out)
On 2013/11/19 13:39:29, Jakob wrote:
I don't think we need any of these three "filter" lines any more.
AddIssues()
below should handle all of that anyway.
Done.
https://codereview.chromium.org/65933003/diff/1/tools/push-to-trunk/common_includes.py#newcode109
tools/push-to-trunk/common_includes.py:109: def AddSafe(bugs, bug):
On 2013/11/19 13:39:29, Jakob wrote:
I think you don't need this function at all, if below in the regexes
you match
against (\d+) instead of (.*).
Done.
https://codereview.chromium.org/65933003/diff/1/tools/push-to-trunk/common_includes.py#newcode116
tools/push-to-trunk/common_includes.py:116: ref = re.match(r"^[ \t]*BUG[
\t]*=[ \t]*(.*?)[ \t]*$", text)
Regarding the inner whitespace I had
https://code.google.com/p/chromium/codesearch#chromium/tools/depot_tools/git_cl.py&sq=package:chromium&type=cs&l=862
as reference. That's what is allowed when depot tools use it.
Agreed on the outer whitespace. Added a strip.
https://codereview.chromium.org/65933003/diff/1/tools/push-to-trunk/common_includes.py#newcode121
tools/push-to-trunk/common_includes.py:121: match = re.match(r"^v8[
\t]*:[ \t]*(.*)$", bug)
On 2013/11/19 13:39:29, Jakob wrote:
similarly, we don't need to support whitespace around ':'
Done.
https://codereview.chromium.org/65933003/diff/1/tools/push-to-trunk/common_includes.py#newcode141
tools/push-to-trunk/common_includes.py:141: FormatIssues("", v8bugs)
On 2013/11/19 13:39:29, Jakob wrote:
V8 bugs first, please :-)
Done. That required some test changes...
https://codereview.chromium.org/65933003/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
