iamandi / nanopb

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

Clean up compiler warnings #8

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I'm compiling on Windows, using a Microsoft compiler set to use the  highest 
warning level.  The code generates quite a few warnings.

Not a very exciting bug - but it would be nice to clean this up.

Attached a patch that resolves all warnings on my machine.  Patch is against 
the nanopb-0.1.1 release.   

Thanks!

David

Original issue reported on code.google.com by david.ho...@googlemail.com on 18 Apr 2012 at 3:35

Attachments:

GoogleCodeExporter commented 9 years ago
This issue was closed by revision d1ca88d20ec1.

Original comment by Petteri.Aimonen on 18 Apr 2012 at 5:16

GoogleCodeExporter commented 9 years ago
Thanks for the patch, I applied it with minor changes.

Hopefully there is no clash for the "UNUSED" macro name, I added #ifdefs around 
it. Could call it "PB_UNUSED", but on the other hand that may be excessively 
cautious.

Original comment by Petteri.Aimonen on 18 Apr 2012 at 5:18