[AD] Allegro 4.2.0: hsv_to_rgb needless assert

[ Thread Index | Date Index | More lists.liballeg.org/allegro-developers Archives ]


The assert in hsv_to_rgb is superfluous:

	void hsv_to_rgb(float h, float s, float v, int *r, int *g, int *b)
	{
[snip]
	   ASSERT(h >= 0 && h <= 360);
[snip]
	      h = fmod(h, 360.0f) / 60.0f;


Either the assert stays and fmod() goes away, or fmod() stays and the assert goes.

My vote is to keep fmod() around and remove the assert, to be compatible with the release version.





Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/