mingw build gives warning about 64 bit om_tostring method.

[richard]

The cygwin build reports:

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

This warning looks bogus, but we should have a test to check that it works properly. This probably requires someone with a cygwin environment on windows to test it (currently, the tinderbox cygwin environment is a cross compile).

[richard]

Patch to add required test to internaltest.cc

[olly]

The tinderbox build is a linux -> mingw cross (on ixion). We no longer have any cygwin builds in the tindexbox.

Test looks fine, though checking conversions of a negative value, and an unsigned value with the top bit set would improve the coverage.

[richard]

I've committed the test - feel free to enhance, but it should give a basic coverage. We should probably leave this bug open for now in the hope that someone will report whether the test passes on mingw. (I'll try and look at it at some point, but we're focusing on MSVC builds for now.)

[olly]

I've fixed this warning (I put the string into a static const array initialised character by character!)

And I've run the cross-built mingw test executables on XP too and all passed, so closing this bug now.

[olly]

Fixed in 1.0.0 release.