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

Issue 24846002: Always convert flag value to string in set_flag (Closed)

Created:
7 years, 2 months ago by Paweł Hajdan Jr.
Modified:
7 years, 2 months ago
Reviewers:
iannucci, M-A Ruel
CC:
chromium-reviews, Dirk Pranke, cmp-cc_chromium.org, M-A Ruel, iannucci+depot_tools_chromium.org, ilevy-cc_chromium.org
Visibility:
Public.

Description

Always convert flag value to string in set_flag The callee needs it to be a string so that it can be made part of HTTP request. However, in code using Rietveld it's useful to use non-string types for flags where Rietveld actually doesn't return strings (say booleans). One example is the commit flag, which is a boolean. BUG=291335 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=225748

Patch Set 1 #

Patch Set 2 : re-upload #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M rietveld.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (0 generated)
Paweł Hajdan Jr.
7 years, 2 months ago (2013-09-26 21:31:33 UTC) #1
Paweł Hajdan Jr.
+Robbie, could you take a look?
7 years, 2 months ago (2013-09-27 18:35:46 UTC) #2
M-A Ruel
lgtm but the patch won't apply, please reupload.
7 years, 2 months ago (2013-09-27 18:37:55 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/phajdan.jr@chromium.org/24846002/8001
7 years, 2 months ago (2013-09-27 19:05:49 UTC) #4
commit-bot: I haz the power
Change committed as 225748
7 years, 2 months ago (2013-09-27 19:07:48 UTC) #5
iannucci
7 years, 2 months ago (2013-09-27 19:26:44 UTC) #6
Message was sent while issue was closed.
oops, belated lgtm

Powered by Google App Engine
This is Rietveld 408576698