diff options
author | Aaron Giles <aaron@aarongiles.com> | 2009-10-01 17:27:29 +0000 |
---|---|---|
committer | Aaron Giles <aaron@aarongiles.com> | 2009-10-01 17:27:29 +0000 |
commit | fe289e67f5e9d1967762588f29b3d9730256cdf5 (patch) | |
tree | bc62503bcc8e7836fa014af34975b3413ecc6e97 /src/osd | |
parent | 61eb81209630e89f8c9b90141c4a5e6ee95fda0a (diff) |
> -----Original Message-----
> From: Atari Ace [mailto:atari_ace@verizon.net]
> Sent: Sunday, September 27, 2009 7:58 AM
> To: submit@mamedev.org
> Cc: atariace@hotmail.com
> Subject: [patch] More _NAME macros
>
> Hi mamedev,
>
> MAME's idiom for function/data macros is to first implement
> <name>_NAME, then implement the other macros in terms of the _NAME
> macro. Then in principle only a single line needs editing to change
> the naming convention.
>
> This patchset implements this idiom more completely. The first patch
> adds some missing _NAME macros and fixes cases in source files that
> should be using the macros. The second patch then changes header
> files where the macros should have been used, but weren't. This
> required changing the idiom for removing a machine driver function
> pointer from MDRV_<FUNCTION>(NULL) to MDRV_<FUNCTION>(0), to avoid
> problems with NULL being macro expanded. This actually unifies the
> handling of all such cases, as we already had ipt_0 and driver_init_0.
> It also required reworking the devtempl.h implementation in a way that
> triggered a warning on MSVC about using empty macros, so vconv.c
> needed to be updated. The third patch then renames all the _NAME and
> _0 macros to verify that all the cases have been covered, so it isn't
> intended to be applied.
>
> ~aa
Diffstat (limited to 'src/osd')
-rw-r--r-- | src/osd/windows/vconv.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/osd/windows/vconv.c b/src/osd/windows/vconv.c index 266457fac43..4b380b4039e 100644 --- a/src/osd/windows/vconv.c +++ b/src/osd/windows/vconv.c @@ -68,7 +68,7 @@ static const translation_info gcc_translate[] = { 0, "-fno-strict-aliasing", "/Oa" }, { 0, "-fno-omit-frame-pointer", "" }, { 0, "-Werror", "/WX" }, - { VS7, "-Wall", "/Wall /W3 /wd4018 /wd4146 /wd4242 /wd4244 /wd4619 /wd4702 /wd4706 /wd4710 /wd4711 /wd4738 /wd4826" }, + { VS7, "-Wall", "/Wall /W3 /wd4003 /wd4018 /wd4146 /wd4242 /wd4244 /wd4619 /wd4702 /wd4706 /wd4710 /wd4711 /wd4738 /wd4826" }, { 0, "-Wall", "/W0" }, { VS7, "-Wno-unused", "/wd4100 /wd4101 /wd4102" }, { 0, "-W*", "" }, |