ecore_time.c: do not mess with the representation of clockid_t.
authorBenjamin Jacobs <benj@spam.thsi.be>
Mon, 6 Jun 2016 18:16:44 +0000 (11:16 -0700)
committerCedric BAIL <cedric@osg.samsung.com>
Mon, 6 Jun 2016 19:05:43 +0000 (12:05 -0700)
commitd41b47f47e6bace28bb700d2aa8737cddb4b465b
tree757fa0629c927b7b99b25fd0fd3073cf3bea155d
parentb548a6396bb10f8de1c15dbe22c9b6e191b2b298
ecore_time.c: do not mess with the representation of clockid_t.

Clockid_t should be used as an opaque type. Some platform might want
to (and even do, e.g. DragonFlyBSD) declare clockid_t as an unsigned.

On such platform, testing the sign of clockid_t is never false, and
assigning it a negative value is an UB, which makes this code unlikely to
work as intended. Fixes black window on dragonfly!

Thanks to gcc for spotting this.

  CC       lib/ecore/lib_ecore_libecore_la-ecore_time.lo
  In file included from ../src/lib/eina/Eina.h:215:0,
                   from lib/ecore/Ecore.h:304,
                   from lib/ecore/ecore_time.c:18:
                   lib/ecore/ecore_time.c: In function 'ecore_time_get':

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>
src/lib/ecore/ecore_time.c