Re: [Mingw-w64-public] [PATCH] headers: Add missing names in winsvc.h

2021-07-02 Thread LIU Hao

在 2021-06-30 12:29, Biswapriyo Nath 写道:

Same mistake as previous. How did you catch those warnings? I copied
the modified header to /mingw64/x86_64-w43-mingw32/include folder then
compile a simple file with `#include `. But gcc and clang
shows nothing. I did something wrong but can't figure it out.



GCC does not warn about code in system paths. I usually use this command to 
examine headers:

  /mingw32/bin/gcc -xc++ -fsyntax-only -ansi -Wall -Wextra  \
-D_WIN32_WINNT=0x -Wno-{long-long,expansion-to-defined}  \
-Imingw-w64-headers/include/  -include windows.h  \
mingw-w64-headers/include/winsvc.h

Hope it helps.


I pushed this patch, following a commit that removed those two erroneous macros.



--
Best regards,
LIU Hao



OpenPGP_signature
Description: OpenPGP digital signature
___
Mingw-w64-public mailing list
Mingw-w64-public@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public


Re: [Mingw-w64-public] [PATCH] headers: Add missing names in winsvc.h

2021-06-30 Thread LIU Hao

在 6/30/21 12:29 PM, Biswapriyo Nath 写道:

Same mistake as previous. How did you catch those warnings? I copied
the modified header to /mingw64/x86_64-w43-mingw32/include folder then
compile a simple file with `#include `. But gcc and clang
shows nothing. I did something wrong but can't figure it out.




I did that by hand; not sure whether you would like it each time:

  ```
  gcc -E -D_WIN32_WINNT=0x 'C:\Program Files (x86)\Windows 
Kits\10\Include\10.0.19041.0\um\winsvc.h' -Wp,-dM -include minwindef.h | sort -u > ~/Desktop/winsdk.i
  gcc -E -D_WIN32_WINNT=0x 'mingw-w64-headers/include/winsvc.h'  -Wp,-dM -include minwindef.h | 
sort -u > ~/Desktop/mingw-w64.i

  git diff --no-index ~/Desktop/{winsdk,mingw-w64}.i
  ```

Our headers are left behind Windows SDK a bit, so it's fine for there to be a few deletions. OTOH 
additions usually indicate mismatches, which are sometimes mistakes.



--
Best regards,
LIU Hao



OpenPGP_signature
Description: OpenPGP digital signature
___
Mingw-w64-public mailing list
Mingw-w64-public@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public


Re: [Mingw-w64-public] [PATCH] headers: Add missing names in winsvc.h

2021-06-29 Thread Biswapriyo Nath
Same mistake as previous. How did you catch those warnings? I copied
the modified header to /mingw64/x86_64-w43-mingw32/include folder then
compile a simple file with `#include `. But gcc and clang
shows nothing. I did something wrong but can't figure it out.


___
Mingw-w64-public mailing list
Mingw-w64-public@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public


Re: [Mingw-w64-public] [PATCH] headers: Add missing names in winsvc.h

2021-06-29 Thread LIU Hao

在 2021-06-29 18:13, Biswapriyo Nath 写道:

 From 152155ec38dd68f1a59880dd4f7b410756325eb0 Mon Sep 17 00:00:00 2001
From: Biswapriyo Nath
Date: Tue, 29 Jun 2021 15:41:41 +0530
Subject: [PATCH] headers: Add missing names in winsvc.h

Signed-off-by: Biswapriyo Nath
---
  mingw-w64-headers/include/winsvc.h | 113 +
  1 file changed, 113 insertions(+)


This patch looks good itself, but I have discovered some old, incorrect definitions. Should they be 
removed at the same time?


  ```
  In file included from mingw-w64-headers/include/windows.h:122,
   from :
  mingw-w64-headers/include/winsvc.h:482: warning: 
"SERVICE_STOP_REASON_MINOR_NONE" redefined
482 | #define SERVICE_STOP_REASON_MINOR_NONE 0x0006
|
  mingw-w64-headers/include/winsvc.h:171: note: this is the location of the 
previous definition
171 | #define SERVICE_STOP_REASON_MINOR_NONE 0x0017
|
  mingw-w64-headers/include/winsvc.h:492: warning: 
"SERVICE_STOP_REASON_MINOR_SOFTWARE_UPDATE_UNINSTALL" redefined

492 | #define SERVICE_STOP_REASON_MINOR_SOFTWARE_UPDATE_UNINSTALL 0x000e
|
  mingw-w64-headers/include/winsvc.h:168: note: this is the location of the 
previous definition
168 | #define SERVICE_STOP_REASON_MINOR_SOFTWARE_UPDATE_UNINSTALL 0x0014
|
  ```


--
Best regards,
LIU Hao



OpenPGP_signature
Description: OpenPGP digital signature
___
Mingw-w64-public mailing list
Mingw-w64-public@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public