From: Olav Sandstaa
Date: April 1 2011 9:51am
Subject: Re: bzr commit into mysql-5.5 branch (jorgen.loland:3404) Bug#11882131
List-Archive: http://lists.mysql.com/commits/134438
Message-Id: <4D95A01F.2030708@oracle.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

On 03/28/2011 02:52 PM, Tor Didriksen wrote:
>
>>> -  bool     asc;                /* true if ascending */
>>> +  enum enum_order {
>>> +    ORDER_NOT_RELEVANT,
>>> +    ORDER_ASC,
>>> +    ORDER_DESC
>>> +  };
>>> +
>>> +  enum_order order;                     /* Requested direction of
> ordering */
>
> Please align comments.
> What happens if we have NOT_RELEVANT here?
> Maybe asc_or_desc is a better name, compare:
>    order->order= ...;
> with
>    order->asc_or_desc= ....;
>

I have no strong opinions about the name but I think keeping "order" as 
the name is fine. A third alternative could be "sort_order"?

Olav