Re: [Quickfix-developers] FieldConvertors.h for VC build

[Oren M. <or...@qu...>]

I meant to indicate the tests are in FieldConvertorsTestCase.  I didn't =20=

mean to specify the double conversion (which hasn't actually changed... =20=

yet)

--oren

On Aug 17, 2004, at 8:02 PM, Oren Miller wrote:

> Yeah, I fixed this earlier today.  The relevant changes are here:
>
> http://cvs.sourceforge.net/viewcvs.py/quickfix/quickfix/src/C%2B%2B/=20=

> FieldConvertors.h?r1=3D1.16&r2=3D1.17
>
> You run the unit tests as normal ( runut <port> ).  The same test case =
=20
> that tested the previous version tests the new version =20
> (FieldConvertorsTestCase::doubleConvertTo).
>
> You can always run the performance tests by going to the bin directory =
=20
> and running pt -c <number of iterations>.  50000 is a good number to =20=

> use.  To compare performance, just run the performance tests from the =20=

> previous release and the new release.
>
> --oren
>
> On Aug 17, 2004, at 12:33 PM, Yihu Fang wrote:
>
>> Hi,
>>
>> =A0
>>
>> I am not sure that anyone has similar problems of building the =20
>> current CVS with the new FieldConvertors.h using Microsoft VC6/VC7.
>>
>>  =A0
>>
>> It complains the std::numeric_limits and many more.
>>
>> =A0
>>
>> QuickFIX\quickfix\src\C++\FieldConvertors.h(138): error C2039: =20
>> 'numeric_limits' : is not a member of 'std'
>>
>> =A0
>>
>> The solution I have is to include <limits> and add check and undef =20=

>> the macro =93max=94 and =93min=94.
>>
>> =A0
>>
>> Is there any performance test as well as unit test of the new =20
>> FieldConvertors.h on Windows?
>>
>> =A0
>>
>> Thanks.
>>
>> =A0
>>
>> -Yihu
>>
>>
>>  -----------------------------------------------------------------
>>  Visit our Internet site at http://www.reuters.com
>>
>>  Get closer to the financial markets with Reuters Messaging - for =
more
>>  information and to register, visit http://www.reuters.com/messaging
>>
>>  Any views expressed in this message are those of the individual
>>  sender, except where the sender specifically states them to be
>>  the views of Reuters Ltd.
>> =20=