quickfix-developers

[Quickfix-developers] Visual Studio 2005 (VC8) compilation

[Alexey Z. <ale...@gm...>]

Hello,

I compiled one of my quickfix projects with the new Microsoft compiler 
(VC8) today.
So, as usual, M$ has its own way.

To compile quickfix you need to add a define into the project: 
_CRT_SECURE_NO_DEPRECATE.
Plus there is an issue with time_t structure which is 64 bit now (even 
if you compile on a 32 machine).
I added two fixes in QUICKFIX/src/C++/FieldTypes.h to solve it:

inline long operator-( const UtcTimeStamp& lhs, const UtcTimeStamp& rhs )
{
  tm l_copy = lhs; tm r_copy = rhs;
  time_t l_time = mktime( &l_copy );
  time_t r_time = mktime( &r_copy );
  return *(long)*(l_time - r_time);
}

inline long operator-( const UtcDate& lhs, const UtcDate& rhs )
{
  tm l_copy = lhs; tm r_copy = rhs;
  time_t l_time = mktime( &l_copy );
  time_t r_time = mktime( &r_copy );
  return *(long)*(( l_time - r_time ) / UTC_DAY);
}

I don't know if it's stable and faster yet, I'll try the new version Monday.

-- 
Regards,
		Alexey Zubko

Re: [Quickfix-developers] Visual Studio 2005 (VC8) compilation

[Caleb E. <cal...@gm...>]

On 11/25/05, Alexey Zubko <ale...@gm...> wrote:
>
> To compile quickfix you need to add a define into the project:
> _CRT_SECURE_NO_DEPRECATE.


That macro isn't strictly necessary, but does quiet a lot of compiler
warnings about POSIX functions which MS now considers to be deprecated.
This is the biggest headache everyone seems to run into with this new
compiler.

The internal implementation of the UTC* classes was changed after the last
release and so the patches you submitted for struct tm/time_t related
manipulations are no longer necessary with the latest code in CVS.

I don't know if it's stable and faster yet, I'll try the new version Monday=
.


Everything seems stable, and the code passes all regression tests.


--
Caleb Epstein
caleb dot epstein at gmail dot com