search

timschmidt / repsnapper

RepSnapper RepRap control software
http://reprap.org/wiki/RepSnapper_Manual:Introduction
154 stars 52 forks source link

[amf tools] replace abs() with std::abs() for gcc6 compatibility #119

Closed tdaitx closed 7 years ago

tdaitx commented 7 years ago

Building with GCC 6 fails with "call of overloaded ‘abs(double)’ is ambiguous" - for some reason only on PowerPC and PPC64.

I have reported this upstream at the AMF Tools forum (for some reason that SF project does not allow someone to create tickets for it).

This pull request is mostly to make you aware this issue exist within AMF Tools, I'm not sure if you want to maintain a "fork" with different patches for it. Hopefully AMF Tools upstream will catch up on this.

hurzl commented 7 years ago

I did a similar fix here: e05de0c1b12a9802ddf036dddf44f44e621f3e6e so I don't think we need an extra branch for it ...