[Xapian-tickets] [Xapian] #303: UUID functions missing on Windows

Xapian nobody at xapian.org
Tue Oct 14 00:38:12 BST 2008


#303: UUID functions missing on Windows
-------------------------------+--------------------------------------------
 Reporter:  charliejuggler     |        Owner:  richard 
     Type:  defect             |       Status:  assigned
 Priority:  normal             |    Milestone:  1.0.9   
Component:  Backend-Flint      |      Version:  SVN HEAD
 Severity:  normal             |   Resolution:          
 Keywords:                     |    Blockedby:          
 Platform:  Microsoft Windows  |     Blocking:          
-------------------------------+--------------------------------------------

Comment(by olly):

 Some other comments:

  * I think msvc_uuid.{h,cc} is the wrong name choice - it's not specific
 to the compiler MSVC, but to the platform.  I suggest win32_uuid.{h,cc}
 since "win32" is arguably the relevant aspect of the platform (and this is
 a wrapper around the win32 UUID API functions).

  * I think msvc_uuid.h should error out if {{{__WIN32__}}} isn't defined -
 there's no reason to include it on other platforms, so it's better to
 catch if we do (c.f. safewindows.h and safewinsock2.h).

  * One of the files lacks a '\n' at the end, which some compilers don't
 handle (I'll add a note about that to HACKING).

  * The code formatting doesn't follow the rest of Xapian in several
 regards (especially noticeable is the indentation of "}" and spacing
 around "(" and ")").

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



More information about the Xapian-tickets mailing list