Allow multi-line enum entries in java_cpp_enum.py.

Allow multi-line enum entries in java_cpp_enum.py.

clang-format wraps long lines, so java_cpp_enum.py should also
handle multi-line enum entries.

Also changed the HeaderParser to ignore empty comments.

BUG=652018
Committed: https://crrev.com/9066edf6a2199aa3390cc55e89a93f858b50761b
Cr-Commit-Position: refs/heads/master@{#422863}

[estevenson, 2016-10-04 15:09:15]

estevenson@chromium.org changed reviewers:
+ agrieve@chromium.org

[estevenson, 2016-10-04 15:09:15]

ptal Andrew!

[estevenson, 2016-10-04 15:34:03]

The CQ bit was checked by estevenson@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-04 15:34:25]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[agrieve, 2016-10-04 15:57:29]

lgtm /w one nit. Thanks for adding tests :)

https://codereview.chromium.org/2396533002/diff/1/build/android/gyp/java_cpp_...
File build/android/gyp/java_cpp_enum.py (right):

https://codereview.chromium.org/2396533002/diff/1/build/android/gyp/java_cpp_...
build/android/gyp/java_cpp_enum.py:182: def _ParseCurrentEnumEntry():
Looks like these helpers don't use their outer context. You should make them
siblings rather than nested.

[estevenson, 2016-10-04 16:15:25]

https://codereview.chromium.org/2396533002/diff/1/build/android/gyp/java_cpp_...
File build/android/gyp/java_cpp_enum.py (right):

https://codereview.chromium.org/2396533002/diff/1/build/android/gyp/java_cpp_...
build/android/gyp/java_cpp_enum.py:182: def _ParseCurrentEnumEntry():
On 2016/10/04 15:57:28, agrieve wrote:
> Looks like these helpers don't use their outer context. You should make them
> siblings rather than nested.

Done.

[estevenson, 2016-10-04 16:15:40]

The CQ bit was checked by estevenson@chromium.org

[estevenson, 2016-10-04 16:15:41]

The patchset sent to the CQ was uploaded after l-g-t-m from agrieve@chromium.org
Link to the patchset: https://codereview.chromium.org/2396533002/#ps20001
(title: "Make inner functions siblings")

[commit-bot: I haz the power, 2016-10-04 16:16:15]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-04 18:27:53]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-04 18:30:19]

Description was changed from

==========
Allow multi-line enum entries in java_cpp_enum.py.

clang-format wraps long lines, so java_cpp_enum.py should also
handle multi-line enum entries.

Also changed the HeaderParser to ignore empty comments.

BUG=652018
==========

to

==========
Allow multi-line enum entries in java_cpp_enum.py.

clang-format wraps long lines, so java_cpp_enum.py should also
handle multi-line enum entries.

Also changed the HeaderParser to ignore empty comments.

BUG=652018

Committed: https://crrev.com/9066edf6a2199aa3390cc55e89a93f858b50761b
Cr-Commit-Position: refs/heads/master@{#422863}
==========

[commit-bot: I haz the power, 2016-10-04 18:30:20]

Patchset 2 (id:??) landed as
https://crrev.com/9066edf6a2199aa3390cc55e89a93f858b50761b
Cr-Commit-Position: refs/heads/master@{#422863}