Sprinter/mame
2
0
Fork 0
mirror of https://github.com/holub/mame synced 2025-04-24 09:20:02 +03:00
Code Issues Actions Packages Projects Releases Wiki Activity

viewgfx.cpp: Allocate bitmap in place rather than using global_alloc; explicitly initialize each member of ui_gfx_state (nw)

Browse Source 
This code honestly deserves more modernization than this, but this is enough for now.
This commit is contained in:
AJR 2019-11-16 13:35:28 -05:00
parent 464f464dd0
commit 653fd83da5
1 changed files with 48 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

74
src/frontend/mame/ui/viewgfx.cpp
View File

@ -59,7 +59,7 @@ struct ui_gfx_state
// intermediate bitmaps
bool bitmap_dirty; // is the bitmap dirty?
bitmap_rgb32 * bitmap; // bitmap for drawing gfx and tilemaps
bitmap_rgb32 bitmap; // bitmap for drawing gfx and tilemaps
render_texture *texture; // texture for rendering the above bitmap
// palette-specific data
@ -141,29 +141,52 @@ static void tilemap_handler(mame_ui_manager &mui, render_container &container, u
void ui_gfx_init(running_machine &machine)
{
ui_gfx_state *state = &ui_gfx;
ui_gfx_state &state = ui_gfx;
uint8_t rotate = machine.system().flags & machine_flags::MASK_ORIENTATION;
// make sure we clean up after ourselves
machine.add_notifier(MACHINE_NOTIFY_EXIT, machine_notify_delegate(&ui_gfx_exit, &machine));
// initialize our global state
memset(state, 0, sizeof(*state));
state.started = false;
state.mode = 0;
state.bitmap_dirty = false;
state.bitmap.reset();
state.texture = nullptr;
state.gfxset.devcount = 0;
state.gfxset.devindex = 0;
state.gfxset.set = 0;
// set up the palette state
state->palette.columns = 16;
state.palette.interface = nullptr;
state.palette.devcount = 0;
state.palette.devindex = 0;
state.palette.which = 0;
state.palette.columns = 16;
// set up the graphics state
for (uint8_t i = 0; i < MAX_GFX_DECODERS; i++)
{
state.gfxdev[i].interface = nullptr;
state.gfxdev[i].setcount = 0;
for (uint8_t j = 0; j < MAX_GFX_ELEMENTS; j++)
{
state->gfxdev[i].rotate[j] = rotate;
state->gfxdev[i].columns[j] = 16;
state.gfxdev[i].rotate[j] = rotate;
state.gfxdev[i].columns[j] = 16;
state.gfxdev[i].offset[j] = 0;
state.gfxdev[i].color[j] = 0;
state.gfxdev[i].palette[j] = nullptr;
state.gfxdev[i].color_count[j] = 0;
}
}
// set up the tilemap state
state->tilemap.rotate = rotate;
state->tilemap.flags = TILEMAP_DRAW_ALL_CATEGORIES;
state.tilemap.which = 0;
state.tilemap.xoffs = 0;
state.tilemap.yoffs = 0;
state.tilemap.zoom = 0;
state.tilemap.rotate = rotate;
state.tilemap.flags = TILEMAP_DRAW_ALL_CATEGORIES;
}
@ -230,8 +253,7 @@ static void ui_gfx_exit(running_machine &machine)
ui_gfx.texture = nullptr;
// free the bitmap
global_free(ui_gfx.bitmap);
ui_gfx.bitmap = nullptr;
ui_gfx.bitmap.reset();
}
@ -906,16 +928,16 @@ static void gfxset_update_bitmap(running_machine &machine, ui_gfx_state &state,
cellypix = 1 + ((info.rotate[set] & ORIENTATION_SWAP_XY) ? gfx.width() : gfx.height());
// realloc the bitmap if it is too small
if (state.bitmap == nullptr || state.texture == nullptr || state.bitmap->bpp() != 32 || state.bitmap->width() != cellxpix * xcells || state.bitmap->height() != cellypix * ycells)
if (!state.bitmap.valid() || state.texture == nullptr || state.bitmap.width() != cellxpix * xcells || state.bitmap.height() != cellypix * ycells)
{
// free the old stuff
machine.render().texture_free(state.texture);
global_free(state.bitmap);
state.bitmap.reset();
// allocate new stuff
state.bitmap = global_alloc(bitmap_rgb32(cellxpix * xcells, cellypix * ycells));
state.bitmap.allocate(cellxpix * xcells, cellypix * ycells);
state.texture = machine.render().texture_alloc();
state.texture->set_bitmap(*state.bitmap, state.bitmap->cliprect(), TEXFORMAT_ARGB32);
state.texture->set_bitmap(state.bitmap, state.bitmap.cliprect(), TEXFORMAT_ARGB32);
// force a redraw
state.bitmap_dirty = true;
@ -930,7 +952,7 @@ static void gfxset_update_bitmap(running_machine &machine, ui_gfx_state &state,
rectangle cellbounds;
// make a rect that covers this row
cellbounds.set(0, state.bitmap->width() - 1, y * cellypix, (y + 1) * cellypix - 1);
cellbounds.set(0, state.bitmap.width() - 1, y * cellypix, (y + 1) * cellypix - 1);
// only display if there is data to show
if (info.offset[set] + y * xcells < gfx.elements())
@ -946,21 +968,21 @@ static void gfxset_update_bitmap(running_machine &machine, ui_gfx_state &state,
// only render if there is data
if (index < gfx.elements())
gfxset_draw_item(machine, gfx, index, *state.bitmap, cellbounds.min_x, cellbounds.min_y, info.color[set], info.rotate[set], info.palette[set]);
gfxset_draw_item(machine, gfx, index, state.bitmap, cellbounds.min_x, cellbounds.min_y, info.color[set], info.rotate[set], info.palette[set]);
// otherwise, fill with transparency
else
state.bitmap->fill(0, cellbounds);
state.bitmap.fill(0, cellbounds);
}
}
// otherwise, fill with transparency
else
state.bitmap->fill(0, cellbounds);
state.bitmap.fill(0, cellbounds);
}
// reset the texture to force an update
state.texture->set_bitmap(*state.bitmap, state.bitmap->cliprect(), TEXFORMAT_ARGB32);
state.texture->set_bitmap(state.bitmap, state.bitmap.cliprect(), TEXFORMAT_ARGB32);
state.bitmap_dirty = false;
}
}
@ -1302,16 +1324,16 @@ static void tilemap_update_bitmap(running_machine &machine, ui_gfx_state &state,
std::swap(width, height);
// realloc the bitmap if it is too small
if (state.bitmap == nullptr || state.texture == nullptr || state.bitmap->width() != width || state.bitmap->height() != height)
if (!state.bitmap.valid() || state.texture == nullptr || state.bitmap.width() != width || state.bitmap.height() != height)
{
// free the old stuff
machine.render().texture_free(state.texture);
global_free(state.bitmap);
state.bitmap.reset();
// allocate new stuff
state.bitmap = global_alloc(bitmap_rgb32(width, height));
state.bitmap.allocate(width, height);
state.texture = machine.render().texture_alloc();
state.texture->set_bitmap(*state.bitmap, state.bitmap->cliprect(), TEXFORMAT_RGB32);
state.texture->set_bitmap(state.bitmap, state.bitmap.cliprect(), TEXFORMAT_RGB32);
// force a redraw
state.bitmap_dirty = true;
@ -1320,16 +1342,16 @@ static void tilemap_update_bitmap(running_machine &machine, ui_gfx_state &state,
// handle the redraw
if (state.bitmap_dirty)
{
state.bitmap->fill(0);
state.bitmap.fill(0);
tilemap_t *tilemap = machine.tilemap().find(state.tilemap.which);
screen_device *first_screen = screen_device_iterator(machine.root_device()).first();
if (first_screen)
{
tilemap->draw_debug(*first_screen, *state.bitmap, state.tilemap.xoffs, state.tilemap.yoffs, state.tilemap.flags);
tilemap->draw_debug(*first_screen, state.bitmap, state.tilemap.xoffs, state.tilemap.yoffs, state.tilemap.flags);
}
// reset the texture to force an update
state.texture->set_bitmap(*state.bitmap, state.bitmap->cliprect(), TEXFORMAT_RGB32);
state.texture->set_bitmap(state.bitmap, state.bitmap.cliprect(), TEXFORMAT_RGB32);
state.bitmap_dirty = false;
}
}