[Xapian-tickets] [Xapian] #300: Test failure : apitest, bigoaddvalue

Xapian nobody at xapian.org
Fri Oct 10 22:31:19 BST 2008


#300: Test failure : apitest, bigoaddvalue
-------------------------------+--------------------------------------------
 Reporter:  charliejuggler     |        Owner:  richard 
     Type:  defect             |       Status:  assigned
 Priority:  normal             |    Milestone:  1.0.9   
Component:  Test Suite         |      Version:  1.0.8   
 Severity:  normal             |   Resolution:          
 Keywords:                     |    Blockedby:          
 Platform:  Microsoft Windows  |     Blocking:          
-------------------------------+--------------------------------------------

Comment(by olly):

 I suggest that for now we skip instead of fail if we get both times being
 equal and exact integers.

 There is a better function to use - I noted it on IRC a while ago:

 @1214350526 <ojwb!i=olly at atreus.tartarus.org> just that windows will be
 using time() to read the current time
 @1214350533 <ojwb!i=olly at atreus.tartarus.org> so will have second
 granularity
 @1214350812 <ojwb!i=olly at atreus.tartarus.org> http://msdn.microsoft.com
 /en-us/library/ms724397(VS.85).aspx
 @1214350817 <ojwb!i=olly at atreus.tartarus.org> that seems the function to
 use
 @1214350848 <ojwb!i=olly at atreus.tartarus.org> no support for NT or the 9x
 family though
 @1214350869 <ojwb!i=olly at atreus.tartarus.org> which isn't a big deal, but
 it would be nice to at least fall back to using time()

 But as it's not available on NT or win 9x, and other platforms might be
 missing a better time function (or we may not detect it), skipping as
 described above would be good anyway.

 I'm not sure it's worthwhile adding support for above to the existing
 OmTime - we're testing
 for O() behaviour of portable code here, which I can't see varying by
 platform.  And as you say it's worth rewriting OmTime at some point.

 Hmm, in fact I think I might have a partial rewrite somewhere already - I
 certainly remember thinking about a better API myself.  It's not anywhere
 obvious on atreus or the laptop, but perhaps it's on one of the drives
 from the dead dev box.  I'll look later.

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



More information about the Xapian-tickets mailing list