Agreed with Jason -- test_bad_identifiers.proto in the Java tree should be
updated.
On Thu, Apr 7, 2011 at 9:19 AM, <jasonh@google.com> wrote:
> code looks fine, can you add a test?
>
>
> http://codereview.appspot.com/4383043/
>
Done. Sorry I was looking at net/proto2/compiler/java (didn't search the java
tree..). It turns out we also need to fix 2 usages in java_message.cc to pass
the test.
On 2011/04/07 17:29:01, kenton wrote:
> Agreed with Jason -- test_bad_identifiers.proto in the Java tree should be
> updated.
>
> On Thu, Apr 7, 2011 at 9:19 AM, <mailto:jasonh@google.com> wrote:
>
> > code looks fine, can you add a test?
> >
> >
> > http://codereview.appspot.com/4383043/
> >
LGTM
On Thu, Apr 7, 2011 at 8:30 PM, <liujisi@google.com> wrote:
> Done. Sorry I was looking at net/proto2/compiler/java (didn't search the
> java tree..). It turns out we also need to fix 2 usages in
> java_message.cc to pass the test.
>
>
> On 2011/04/07 17:29:01, kenton wrote:
>
>> Agreed with Jason -- test_bad_identifiers.proto in the Java tree
>>
> should be
>
>> updated.
>>
>
> On Thu, Apr 7, 2011 at 9:19 AM, <mailto:jasonh@google.com> wrote:
>>
>
> > code looks fine, can you add a test?
>> >
>> >
>> > http://codereview.appspot.com/4383043/
>> >
>>
>
>
>
> http://codereview.appspot.com/4383043/
>
Issue 4383043: Use java.lang.Object instead of Object to allow "Object" to be used as message name.
Created 13 years ago by Jisi Liu
Modified 13 years ago
Reviewers: kenton, jasonh
Base URL:
Comments: 0