Re: [PATCH v9 0/1] MR4672: include: Add msdelta.h header file.

[Alexandre Julliard (＠julliard)]

Alexandre Julliard (@julliard) commented about include/msdelta.h:
> +#define DELTA_FILE_TYPE_CLI_I386             (DELTA_FILE_TYPE)0x00000010
> +#define DELTA_FILE_TYPE_CLI_AMD64            (DELTA_FILE_TYPE)0x00000020
> +#define DELTA_FILE_TYPE_CLI_ARM              (DELTA_FILE_TYPE)0x00000040
> +#define DELTA_FILE_TYPE_CLI_ARM64            (DELTA_FILE_TYPE)0x00000080
> -
>
> +#define DELTA_FILE_TYPE_SET_RAW_ONLY         DELTA_FILE_TYPE_RAW
> +#define DELTA_FILE_TYPE_SET_EXECUTABLES_1    DELTA_FILE_TYPE_RAW|DELTA_FILE_TYPE_I386| \
> -
>                                         DELTA_FILE_TYPE_IA64|DELTA_FILE_TYPE_AMD64
>
> +#define DELTA_FILE_TYPE_SET_EXECUTABLES      DELTA_FILE_TYPE_SET_EXECUTABLES_1
> +#define DELTA_FILE_TYPE_SET_EXECUTABLES_2    DELTA_FILE_TYPE_RAW|DELTA_FILE_TYPE_CLI_I386| \
> -
>                                         DELTA_FILE_TYPE_IA64|DELTA_FILE_TYPE_CLI_AMD64| \
> -
>                                         DELTA_FILE_TYPE_CLI_ARM
>
> +#define DELTA_FILE_TYPE_SET_EXECUTABLES_3    DELTA_FILE_TYPE_RAW|DELTA_FILE_TYPE_CLI_I386| \
> -
>                                         DELTA_FILE_TYPE_IA64|DELTA_FILE_TYPE_CLI_AMD64| \
> -
>                                         DELTA_FILE_TYPE_CLI_ARM|DELTA_FILE_TYPE_CLI_ARM64
>
> +#define DELTA_FILE_TYPE_SET_EXECUTABLES_LATEST DELTA_FILE_TYPE_SET_EXECUTABLES_3

Most of these macros will need an extra pair of parentheses.