Commit 01b7e5cd authored by Karandeep Bhatia's avatar Karandeep Bhatia Committed by Commit Bot

JsonSchemaCompiler: Rename ConstantName To FeatureNameToConstantName.

This fixes an existing TODO. The current name doesn't clearly specify
what the function does.

BUG=1113513

Change-Id: Id08cd8e56e72abc0ad1059aacf20f989bb99d8f5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2342754
Auto-Submit: Karan Bhatia <karandeepb@chromium.org>
Reviewed-by: default avatarDevlin <rdevlin.cronin@chromium.org>
Commit-Queue: Karan Bhatia <karandeepb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#802801}
parent e38f198a
...@@ -126,8 +126,8 @@ def CloseNamespace(cpp_namespace): ...@@ -126,8 +126,8 @@ def CloseNamespace(cpp_namespace):
return c return c
# TODO(karandeepb): Rename to FeatureNameToConstantName. def FeatureNameToConstantName(feature_name):
def ConstantName(feature_name): # type: (str) -> str
"""Returns a kName for a feature's name. """Returns a kName for a feature's name.
""" """
return ('k' + ''.join(word[0].upper() + word[1:] return ('k' + ''.join(word[0].upper() + word[1:]
......
...@@ -47,7 +47,8 @@ class _Generator(object): ...@@ -47,7 +47,8 @@ class _Generator(object):
) )
for feature in self._feature_defs: for feature in self._feature_defs:
c.Append('features_["%s"] = %s;' c.Append('features_["%s"] = %s;'
% (feature.name, cpp_util.ConstantName(feature.name))) % (feature.name,
cpp_util.FeatureNameToConstantName(feature.name)))
(c.Eblock() (c.Eblock()
.Append('}') .Append('}')
.Append() .Append()
...@@ -62,7 +63,7 @@ class _Generator(object): ...@@ -62,7 +63,7 @@ class _Generator(object):
) )
for feature in self._feature_defs: for feature in self._feature_defs:
c.Append('case %s: return "%s";' % c.Append('case %s: return "%s";' %
(cpp_util.ConstantName(feature.name), feature.name)) (cpp_util.FeatureNameToConstantName(feature.name), feature.name))
(c.Append('case kUnknown: break;') (c.Append('case kUnknown: break;')
.Append('case kEnumBoundary: break;') .Append('case kEnumBoundary: break;')
.Eblock() .Eblock()
......
...@@ -93,7 +93,7 @@ class _Generator(object): ...@@ -93,7 +93,7 @@ class _Generator(object):
.Append('kUnknown,') .Append('kUnknown,')
) )
for feature in self._feature_defs: for feature in self._feature_defs:
c.Append('%s,' % cpp_util.ConstantName(feature.name)) c.Append('%s,' % cpp_util.FeatureNameToConstantName(feature.name))
c.Append('kEnumBoundary') c.Append('kEnumBoundary')
return c return c
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment