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

Issue 260893013: JSON Schema Compiler: Following r267534, properly convert arrays of enums (Closed)

Created:
6 years, 7 months ago by not at google - send to devlin
Modified:
6 years, 7 months ago
Reviewers:
Yoyo Zhou
CC:
chromium-reviews, armansito
Visibility:
Public.

Description

JSON Schema Compiler: Following r267534, properly convert arrays of enums within types. BUG=368368 R=yoz@chromium.org

Patch Set 1 #

Patch Set 2 : better #

Unified diffs Side-by-side diffs Delta from patch set Stats (+52 lines, -13 lines) Patch
M tools/json_schema_compiler/cc_generator.py View 1 1 chunk +10 lines, -3 lines 0 comments Download
M tools/json_schema_compiler/util.h View 1 1 chunk +19 lines, -0 lines 0 comments Download
M tools/json_schema_compiler/util_cc_helper.py View 1 1 chunk +23 lines, -10 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Yoyo Zhou
LGTM, sorry I missed this earlier
6 years, 7 months ago (2014-05-06 18:01:14 UTC) #1
not at google - send to devlin
my bad I didn't send it out. I found another very subtle bug and haven't ...
6 years, 7 months ago (2014-05-06 18:09:54 UTC) #2
Yoyo Zhou
6 years, 7 months ago (2014-05-06 18:21:21 UTC) #3
woops! I didn't notice that; just saw it in my list of reviews.

Powered by Google App Engine
This is Rietveld 408576698