Xapian devel - Mar 2007 - Re: [Xapian-commits] 7817: trunk/xapian-core/ trunk/xapian-core/common/

On Thu, Mar 01, 2007 at 03:29:26PM +0000, richard wrote:
> * common/utils.c,common/utils.h: For windows, add an om_tostring()function
for
>   64 bit integers: time() and GetProcessId() return these, and without
this,
>   backends/quartz/quartz_log.cc reports an error.
This change generates a new warning in the mingw cross-build:

common/utils.cc:82: warning: int format, different type arg (arg 3)

Presumably this warning is bogus, since mingw uses the MSVC C runtime
DLL?

Cheers,
    Olly

Olly Betts wrote:
> On Thu, Mar 01, 2007 at 03:29:26PM +0000, richard wrote:
>> * common/utils.c,common/utils.h: For windows, add an
om_tostring()function for
>>   64 bit integers: time() and GetProcessId() return these, and without
this,
>>   backends/quartz/quartz_log.cc reports an error.
> 
> This change generates a new warning in the mingw cross-build:
> 
> common/utils.cc:82: warning: int format, different type arg (arg 3)
> 
> Presumably this warning is bogus, since mingw uses the MSVC C runtime
> DLL?
It sounds pretty bogus - perhaps the best plan would be to put together 
a test (presumably in internaltest.cc) which checked that it was working 
correctly.  I'll look into that shortly.

-- 
Richard