Re: [ros-cvs] CVS Update: reactos

[James Tabor]

Hi!

Ge van Geldorp wrote:
>   CVSROOT:	/CVS/ReactOS
>  Module name:	reactos
>  Repository:	reactos/lib/shlwapi/
>  Changes by:	gvg(a)mok.osexperts.com	04/10/20 09:20:23
>  
>  Modified files:
>  	reactos/include/wine/: shlwapi.h 
>  	reactos/w32api/include/: shlwapi.h 
>  	reactos/lib/shlwapi/: Makefile.ros-template 
>  
>  Log message:
>  	Simplify wine/shlwapi.h
>   
Not to sure how this started time wise,

i386-mingw32-gcc -DWINSHLWAPI= -D_SHLWAPI_ -D__USE_W32API -D_WIN32_IE=0x600
-D_WIN32_WINNT=0x501 
-DWINVER=0x501 -D_STDDEF_H -I../../include/wine -I../../include -I../../w32api/include
-pipe 
-march=i486 -D_M_IX86 -c msgbox.c -o msgbox.o
In file included from msgbox.c:23:
../../include/wine/port.h:248: error: parse error before "pread"
../../include/wine/port.h:248: warning: data definition has no type or storage class
../../include/wine/port.h:252: error: parse error before "pwrite"
../../include/wine/port.h:252: warning: data definition has no type or storage class
make: *** [msgbox.o] Error 1


Thanks,
James

[Ge van Geldorp]

>   From: James Tabor
>  
>  Hi!
>  
>  Ge van Geldorp wrote:
> >   Modified files:
> >  	reactos/lib/shlwapi/: Makefile.ros-template
> >  
> >  Log message:
> >  	Simplify wine/shlwapi.h
> >   
>  
>  Not to sure how this started time wise,
>  
>  i386-mingw32-gcc -DWINSHLWAPI= -D_SHLWAPI_ -D__USE_W32API 
>  -D_WIN32_IE=0x600 -D_WIN32_WINNT=0x501 
>  -DWINVER=0x501 -D_STDDEF_H -I../../include/wine 
>  -I../../include -I../../w32api/include -pipe 
>  -march=i486 -D_M_IX86 -c msgbox.c -o msgbox.o
>  In file included from msgbox.c:23:
>  ../../include/wine/port.h:248: error: parse error before "pread"
>  ../../include/wine/port.h:248: warning: data definition has 
>  no type or storage class
>  ../../include/wine/port.h:252: error: parse error before "pwrite"
>  ../../include/wine/port.h:252: warning: data definition has 
>  no type or storage class 
No problem with GCC 3.4.1 on Windows, which is why I didn't notice it.
http://reactos.com:8080/archives/public/ros-cvs/2004-October/005776.html
should fix it.

I don't see the point of the use of the __REACTOS__ preprocessor symbol in
our tree. I think we should just remove "#ifdef __REACTOS__" lines and
remove the code inactivated by "#ifndef __REACTOS__". After all, we are
ALWAYS compiling for ReactOS in our tree.

If noone objects, I'll change it this weekend.

Ge van Geldorp.

[art yerkes]

>   I don't see the point of the use of the
> __REACTOS__ preprocessor symbol in
>  our tree. I think we should just remove "#ifdef __REACTOS__" lines and
>  remove the code inactivated by "#ifndef __REACTOS__". After all, we are
>  ALWAYS compiling for ReactOS in our tree. 
Please preserve it in drivers/lib/oskittcp.  I'm trying to make diff output
small against freebsd in there.
-- 
Hey, Adam Smith, keep your invisible hands to yourself!