[Xapian-tickets] [Xapian] #311: set_sort_by_value has incorrect sense for 'ascending'

Xapian nobody at xapian.org
Thu Apr 23 14:51:20 BST 2009


#311: set_sort_by_value has incorrect sense for 'ascending'
-------------------------+--------------------------------------------------
 Reporter:  tommortimer  |        Owner:  olly     
     Type:  defect       |       Status:  closed   
 Priority:  normal       |    Milestone:  1.0.11   
Component:  Library API  |      Version:  SVN trunk
 Severity:  minor        |   Resolution:  fixed    
 Keywords:               |    Blockedby:           
 Platform:  All          |     Blocking:           
-------------------------+--------------------------------------------------

Comment(by olly):

 Hmm, well spotted.

 The default direction is good for !MultiValueSorter::add(), but the
 parameter name is annoyingly inconsistent.

 Not sure I see a good fix other than renaming the class and/or method.

 I think we probably do want a new ticket for this - I'll open one.

-- 
Ticket URL: <http://trac.xapian.org/ticket/311#comment:16>
Xapian <http://xapian.org/>
Xapian



More information about the Xapian-tickets mailing list