Review please!

Thanks,
 Thomas

On Wed, Mar 19, 2014 at 08:52:46AM +0100, Thomas Klausner wrote:
> From: Jörg Sonnenberger <[email protected]>
> 
> For long arguments, use labs().
> 
> Signed-off-by: Thomas Klausner <[email protected]>
> ---
>  src/Text.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/Text.c b/src/Text.c
> index a1ae74a..8100122 100644
> --- a/src/Text.c
> +++ b/src/Text.c
> @@ -2837,7 +2837,7 @@ DoSelection(TextWidget ctx, XawTextPosition pos, Time 
> time, Bool motion)
>      if (motion)
>       newType = ctx->text.s.type;
>      else {
> -     if ((abs((long) time - (long) ctx->text.lasttime) < MULTI_CLICK_TIME)
> +     if ((labs((long) time - (long) ctx->text.lasttime) < MULTI_CLICK_TIME)
>           && (pos >= ctx->text.s.left && pos <= ctx->text.s.right)) {
>           sarray = ctx->text.sarray;
>           for (; *sarray != XawselectNull && *sarray != ctx->text.s.type;
> -- 
> 1.9.0
> 
> _______________________________________________
> [email protected]: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to