On Thu, Oct 23, 2008 at 07:53:34PM +0000, Adam Litke wrote:
> Oops.  The greater-than operator (>) is not commutative.  While trying to test
> for a > b, we are accidentally re-testing a < b.  Fix.
> 
> Signed-off-by: Adam Litke <[EMAIL PROTECTED]>

Acked-by: Mel Gorman ,[EMAIL PROTECTED]>

> ---
> 
>  kernel-features.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> 
> diff --git a/kernel-features.c b/kernel-features.c
> index 8a80117..c6a9987 100644
> --- a/kernel-features.c
> +++ b/kernel-features.c
> @@ -104,7 +104,7 @@ static int int_cmp(int a, int b)
>  {
>       if (a < b)
>               return -1;
> -     if (b > a)
> +     if (a > b)
>               return 1;
>       else
>               return 0;
> 

-- 
Mel Gorman
Part-time Phd Student                          Linux Technology Center
University of Limerick                         IBM Dublin Software Lab

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Libhugetlbfs-devel mailing list
Libhugetlbfs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libhugetlbfs-devel

Reply via email to