[PATCH 2/5] msi/tests: Test deferral of MsiPublishAssemblies and MsiUnpublishAssemblies.
Hans Leidekker
hans at codeweavers.com
Wed May 23 03:30:15 CDT 2018
On Tue, 2018-05-22 at 18:39 -0500, Zebediah Figura wrote:
> Signed-off-by: Zebediah Figura <z.figura12 at gmail.com>
> ---
> dlls/msi/tests/action.c | 14 ++++++++++++
> dlls/msi/tests/custom.c | 53 ++++++++++++++++++++++++++++++++++++++++++++++
> dlls/msi/tests/custom.spec | 2 ++
> 3 files changed, 69 insertions(+)
This produces warnings here (gcc 6.3.0):
custom.c: In function ‘pa_present’:
custom.c:75:31: warning: this ‘for’ clause does not guard... [-Wmisleading-indentation]
#define todo_wine_if(is_todo) for (winetest_start_todo(is_todo); \
^
custom.c:75:31: note: in definition of macro ‘todo_wine_if’
#define todo_wine_if(is_todo) for (winetest_start_todo(is_todo); \
^~~
custom.c:1415:5: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘for’
RegCloseKey(key);
^~~~~~~~~~~
custom.c: In function ‘pa_absent’:
custom.c:75:31: warning: this ‘for’ clause does not guard... [-Wmisleading-indentation]
#define todo_wine_if(is_todo) for (winetest_start_todo(is_todo); \
^
custom.c:75:31: note: in definition of macro ‘todo_wine_if’
#define todo_wine_if(is_todo) for (winetest_start_todo(is_todo); \
^~~
custom.c:1431:9: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘for’
RegCloseKey(key);
^~~~~~~~~~~
Adding braces as in the other functions gets rid of the warnings.
More information about the wine-devel
mailing list