Hey Mike, On 20 December 2016 at 01:12, Mike Blumenkrantz < [email protected]> wrote:
> discomfitor pushed a commit to branch master. > > http://git.enlightenment.org/core/efl.git/commit/?id= > 25792d64165ad4f5f647a36f087af2d2206a6618 > > commit 25792d64165ad4f5f647a36f087af2d2206a6618 > Author: Mike Blumenkrantz <[email protected]> > Date: Mon Dec 19 11:08:43 2016 -0500 > > build: enable -Wfloat-equal for compiling > > #WarningOfTheMonth > Urgh. What the hell. Those warnings have a very limited value. How many real issues have you seen that resulted from this unsafe comparison? I'm pretty sure that float == 0 or float != 0 will succeed when we expect them to, and the equality tests will also work whenever we do direct assignments (without arithmetic). Now I understand why cedric made a series of patches with the float comparison thing. I'm sure he didn't have anything more important to do. Are you going to fix the remaining 523 warnings? Please don't send 100 patches for that... PS: I had only 2 warnings before that. Potentially not fixable (va_start with char arguments in EAPI functions). But at least I could easily spot anything new. -- Jean-Philippe André ------------------------------------------------------------------------------ Developer Access Program for Intel Xeon Phi Processors Access to Intel Xeon Phi processor-based developer platforms. With one year of Intel Parallel Studio XE. Training and support from Colfax. Order your platform today.http://sdm.link/intel _______________________________________________ enlightenment-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
