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

Issue 250773010: [NaCl SDK] easy_template: fix bug when using % in an expression template. (Closed)

Created:
6 years, 8 months ago by binji
Modified:
6 years, 7 months ago
Reviewers:
bradnelson, bradn
CC:
chromium-reviews, binji+watch_chromium.org, Sam Clegg
Visibility:
Public.

Description

[NaCl SDK] easy_template: fix bug when using % in an expression template. BUG=none R=bradnelson@google.com, bradnelson@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266627

Patch Set 1 #

Patch Set 2 : fix % after {{}} bug #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -3 lines) Patch
M native_client_sdk/src/build_tools/easy_template.py View 1 1 chunk +12 lines, -3 lines 0 comments Download
M native_client_sdk/src/build_tools/tests/easy_template_test.py View 1 1 chunk +7 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
binji
6 years, 8 months ago (2014-04-26 00:29:19 UTC) #1
bradn
lgtm
6 years, 8 months ago (2014-04-26 00:55:15 UTC) #2
binji
fixed bug. Forgot to replace % with %% after expression match {{}}
6 years, 7 months ago (2014-04-28 17:21:23 UTC) #3
bradn
still lg
6 years, 7 months ago (2014-04-28 18:28:55 UTC) #4
binji
6 years, 7 months ago (2014-04-28 19:28:41 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 manually as r266627 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698