Can't compile with vc7: oleauto.h(229) : error C2719: 'function-parameter': formal parameter with __declspec(align('8')) won't be aligned

[Dan Kegel]

A UCLA student trying to compile wine conformance tests on Windows
ran into this:
> [Compiling wine tests with VC7 or
> the server 2003 sdk gets] this error when compiling.
>
> ..\..\..\include\oleauto.h(229) : error C2719: 'function-parameter':
> formal parameter with __declspec(align('8')) won't be aligned

(See http://msdn2.microsoft.com/en-us/library/373ak2y1.aspx )

Examining oleauto.h line 229, I saw the bad parameter was
of type DOUBLE.
I then went looking for the typedef for DOUBLE:
~/wine/include$ grep 'typedef.*DOUBLE' *.h
The interesting hit was:
wtypes.h:typedef double DECLSPEC_ALIGN(8) DOUBLE;

The patch that introduced the change was
 http://www.winehq.com/hypermail/wine-cvs/2004/05/0027.html

I told him to try replacing DOUBLE and DATE with double
and date in the formal parameters in oleauto.h
for now.  Is that generally the right thing to do?
- Dan

--
Wine for Windows ISVs: http://kegel.com/wine/isv

[Alexandre Julliard]

Dan Kegel <dank(a)kegel.com> writes:
> A UCLA student trying to compile wine conformance tests on Windows
> ran into this:
> > [Compiling wine tests with VC7 or
> > the server 2003 sdk gets] this error when compiling.
> >
> > ..\..\..\include\oleauto.h(229) : error C2719: 'function-parameter':
> > formal parameter with __declspec(align('8')) won't be aligned
>
> (See http://msdn2.microsoft.com/en-us/library/373ak2y1.aspx )
>
> Examining oleauto.h line 229, I saw the bad parameter was
> of type DOUBLE.
> I then went looking for the typedef for DOUBLE:
> ~/wine/include$ grep 'typedef.*DOUBLE' *.h
> The interesting hit was:
> wtypes.h:typedef double DECLSPEC_ALIGN(8) DOUBLE;
>
> The patch that introduced the change was
>  http://www.winehq.com/hypermail/wine-cvs/2004/05/0027.html
>
> I told him to try replacing DOUBLE and DATE with double
> and date in the formal parameters in oleauto.h
> for now.  Is that generally the right thing to do?

I committed a change to add a #ifdef MSVC around that typedef,
hopefully that will work for you.

-- 
Alexandre Julliard
julliard(a)winehq.org