Closed GoogleCodeExporter closed 9 years ago
ssuominen,
I reviewed the threads you linked to, as well as information about c99.
It seems to me that the basic problem is faac is not compiling with -std=c99.
This does not seem to be an issue with mp4v2, because mp4v2 is written in C++.
-std=c99 informs gcc that mp4v2 conforms to the c99 specification for the C
programming language, which is simply wrong given that mp4v2 is written in C++.
The GNU gcc documentation even makes it clear that you cannot compile C++ with
std=c99 (see http://gcc.gnu.org/onlinedocs/gcc-3.0.4/gcc_6.html, for example).
Furthermore, platform.h is a system header, and altering a system header could
potentially break other packages.
See http://www.ibm.com/developerworks/library/l-c99.html for more info, but
it's pretty clear to me that c99 applies only to the C language. Please let me
know if you disagree with this, but after reviewing all material, I believe
this change needs to happen in faac and not mp4v2.
Original comment by kid...@gmail.com
on 22 Jul 2010 at 8:02
Original issue reported on code.google.com by
ssuomi...@unk.fi
on 4 Jul 2010 at 12:26