Comment 0 for bug 918554

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

The macro DEG2RAD is no longer universally defined as has now moved to core/deg2rad.h. But some plugins don't know this yet...
lp:compiz-expo-plugin / compiz-plugins-main
lp:compiz-cubeaddon-plugin / compiz-plugins-extra

One could resolve this by simply including core/deg2rad.h in the affected code. But looking at deg2rad.h I suggest the file shouldn't exist at all and should be removed from compiz-core. And plugins should replace DEG2RAD with an explicit (M_PI / 180.0f) which is more readable and reduces coupling.