[PATCH:libXaw] Fix abs() usage.

walter harms wharms at bfs.de
Thu Jul 17 09:54:23 PDT 2014



Am 17.07.2014 17:11, schrieb Thomas Klausner:
> On Wed, Jul 16, 2014 at 03:50:35PM -0700, Matt Turner wrote:
>> On Wed, Jul 16, 2014 at 9:44 AM, Thomas Klausner <wiz at netbsd.org> wrote:
>>> Review please!
>>>
>>> Thanks,
>>>  Thomas
>>
>> Reviewed-by: Matt Turner <mattst88 at gmail.com>
>>
>> Please apply to libXaw3d as well.
> 
> Thanks for the review. I've fixed it in libXaw3d as well and added
> your reviewed-by for it.
>  Thomas


i am sorry to interrupt,
this is a case for difftime()

  if ((abs((long) time - (long) ctx->text.lasttime) < MULTI_CLICK_TIME)

 if ( fabs(difftime(time,ctx->text.lasttime)) < MULTI_CLICK_TIME )

this avoids the different implementation of time_t

re,
 wh





More information about the xorg-devel mailing list