iamandi / nanopb

Automatically exported from code.google.com/p/nanopb
zlib License
0 stars 0 forks source link

Required extensions cause previously working .proto file to fail #83

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
From P. Stanek:

> Hi,
> Including of extensions support makes me some problems in my code.
> ( usage of "extend google.protobuf.MessageOptions" with required field)
> 
> What about making new generator option to not process extensions?
> or
> Making generator to only produce some warnings that required fields in
> extension will not be checked and process them as optional etc...
> 
> Thanks

--

Having an option to disable extensions does make sense.

I must wonder how sensible "required extensions" are though.. because the 
"required" part would not be evaluated for anyone who doesn't include the file 
in question.

Original issue reported on code.google.com by Petteri.Aimonen on 29 Aug 2013 at 4:56

GoogleCodeExporter commented 9 years ago
This issue was updated by revision d7f3a74388b4.

Original comment by Petteri.Aimonen on 8 Sep 2013 at 8:10

GoogleCodeExporter commented 9 years ago
Fix released in nanopb-0.2.3.

Original comment by Petteri.Aimonen on 18 Sep 2013 at 9:51