diff options
author | 2009-01-22 10:21:52 +0000 | |
---|---|---|
committer | 2009-01-22 10:21:52 +0000 | |
commit | 325dadfdc0d0cd2afcad8d04e8c973793f4866fc (patch) | |
tree | bc88745544df69ba4eb523b8ac9f511168f9b193 /src/emu/machine/8237dma.c | |
parent | 58af175636d898f6bda49c9e119c9f4bce373ed0 (diff) |
On Mon, Jan 19, 2009 at 02:48:05PM +0100, Olivier Galibert wrote:
> On Mon, Jan 19, 2009 at 05:37:35AM -0800, R. Belmont wrote:
> > My mistake. I thought you were suggesting that we should actually
> > somehow handle malloc failures. Given that aborting is an OK way to
> > express failure, I'd suggest the return values be changed to DEFER
> > and DONT_DEFER to eliminate the conceptual imbalance of OK/DEFER.
>
> That's where comes the fact that we have 130 OK/DONT_DEFER and 1
> DEFER. It makes me think that the exceptional DEFER case should be
> handled by an exceptional function call.
>
> I know, code talks, but I'm at work right now :-)
Here we go.
OG.
Diffstat (limited to 'src/emu/machine/8237dma.c')
-rw-r--r-- | src/emu/machine/8237dma.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/emu/machine/8237dma.c b/src/emu/machine/8237dma.c index ebb659aa6f5..2a2aefcc764 100644 --- a/src/emu/machine/8237dma.c +++ b/src/emu/machine/8237dma.c @@ -395,8 +395,6 @@ static DEVICE_START( dma8237 ) { dma8237_t *dma8237 = get_safe_token(device); dma8237->intf = device->static_config; - - return DEVICE_START_OK; } |