satyenr / protobuf-perlxs

Automatically exported from code.google.com/p/protobuf-perlxs
Apache License 2.0
0 stars 0 forks source link

Nested messages have incorrect class #2

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Use a nested message construct.
2. After compilation and use, do an ISA on the nested message and the
   class is that of the nesting class, not the nested class.

What is the expected output? What do you see instead?

Here's a proto that will show the issue:

------------------
package Report;

message Statistic {
    optional string key = 1;
    optional string val = 2;
}

message Query {
    optional string    name  = 1;
    repeated Statistic stats = 2;
}
------------------

What version of the product are you using? On what operating system?

0.2 build on Linux 2.6.21/gcc 4.2.4/perl 5.8.8

Please provide any additional information below.

I attached a patch - simple fix. Thanks for coding this up.

-John

Original issue reported on code.google.com by jwpalmi...@gmail.com on 4 Nov 2008 at 5:21

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks John!  I will include this in the next release (0.3, due this weekend).

-dave

Original comment by dave%dav...@gtempaccount.com on 6 Nov 2008 at 12:19

GoogleCodeExporter commented 9 years ago

Original comment by dave%dav...@gtempaccount.com on 19 Nov 2008 at 5:42