>> I will look over your patch and get back to you. > > there are really only two changes that were made, first, '#include <cstdlib>' > was removed since MIPSpro doesn't use it, and second, std::abs() doesn't > exist > on MIPSpro so i had to use the abs() in the C math routines, and needed to > add > '#include <math.h>' in the files that tries to use abs(). >
math.h is the wrong header, it should be stdlib.h (in C++ <cfoo> means it is the wrapper for <foo.h>). I can drop the std:: and use stdlib.h. >> As for the -Wall -W -pedantic, search for them in configure.in. If you can >> submit me code that detects the compiler is not gcc and removes the flags I >> would be greatful. Otherwise the bulk 90%+ of the users of blackbox use gcc >> so >> it is better for the few to override than to change it for the many. > > i agree 100%. it's not hard to edit that stuff out. if i can figure out how > to get configure to cope with non-gcc, would you like me to set a define so > that the MIPSpro changes in the source can be wrapped? i don't know that > there > is any sane solution to what i needed to do to get it all working. > hopefully we can avoid source changes. However I would like configure to be able to set the compiler flags properly.
