edward0429 / arduino

Automatically exported from code.google.com/p/arduino
0 stars 0 forks source link

Compile with -pedantic #702

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Compile with -pedantic, or at least make that option available.

Then use it before releasing the next version, to catch booboos like this one:

SPI.h:25:1: warning: "SPI_CLOCK_DIV64" redefined
SPI.h:20:1: warning: this is the location of the previous definition

What version of the Arduino software are you using?
0022 (current stable)

I'm filing this as a bug, because inability to get good compiler feedback is a 
bug, not a missing feature.

Original issue reported on code.google.com by gooc...@top.geek.nz on 30 Oct 2011 at 9:46

GoogleCodeExporter commented 8 years ago
This, and many other issues, have already been fixed.  You should test with 
1.0-rc2, or the latest from github, not 0022.  You can get 1.0-rc2 here:

http://code.google.com/p/arduino/wiki/Arduino1

In 1.0-rc2, when you check the verbose output option in Preferences, compiler 
warnings are enabled.

At least one warning type, related to PROGMEM, is a compiler bug and can be 
safely ignored.  Using -pedantic would only cause perfectly good code to not 
compile due to that bug in the compiler.

This is a duplicate if issue 567

Original comment by paul.sto...@gmail.com on 30 Oct 2011 at 3:22

GoogleCodeExporter commented 8 years ago
It's also a duplicate of issue 664

Original comment by paul.sto...@gmail.com on 30 Oct 2011 at 3:27

GoogleCodeExporter commented 8 years ago

Original comment by dmel...@gmail.com on 30 Oct 2011 at 5:49