Page MenuHomePhabricator

Evas gesture: do not use abs when dealing with double, use fabs instead
ClosedPublic

Authored by vtorri on May 25 2019, 10:51 PM.

Details

Summary

clang reports a warning here. We compare the result of abs (casting to an int, returning an int) to a double value. Hence we should use fabs instead

Test Plan

compilation

Diff Detail

Repository
rEFL core/efl
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
vtorri created this revision.May 25 2019, 10:51 PM

It seems that this patch has no reviewers specified. If you are unsure who can review your patch, please check this wiki page and see if anyone can be added: https://phab.enlightenment.org/w/maintainers_reviewers/

vtorri requested review of this revision.May 25 2019, 10:51 PM
vtorri edited the summary of this revision. (Show Details)May 25 2019, 10:55 PM
vtorri edited the test plan for this revision. (Show Details)
vtorri added a reviewer: Jaehyun.
Hermet accepted this revision.May 26 2019, 11:55 PM
This revision is now accepted and ready to land.May 26 2019, 11:55 PM
This revision was automatically updated to reflect the committed changes.