screen_device::create_composited_bitmap() - missing 'break'?

[fasteddo]

\src\emu\screen.cpp
screen_device::create_composited_bitmap()

I'm assuming there's a break missing in the first case statement. If this is intentional, please ignore, I'm still trying to learn the source.

void screen_device::create_composited_bitmap()
{
	screen_bitmap &curbitmap = m_bitmap[m_curtexture];
	if (!curbitmap.valid())
		return;

	s32 dstwidth = std::max(m_max_width, m_visarea.right() + 1);
	int dstheight = curbitmap.height();

	switch (curbitmap.format())
	{
		default:
		case BITMAP_FORMAT_IND16:
		{
			for (int y = 0; y < dstheight; y++)
			{
				bitmap_ind16 &srcbitmap = *(bitmap_ind16 *)m_scan_bitmaps[m_curbitmap][y];
				u16 *dst = &curbitmap.as_ind16().pix16(y);
				const u16 *src = &srcbitmap.pix16(0);
				const int dx = (m_scan_widths[y] << 15) / dstwidth;
				for (int x = 0; x < m_scan_widths[y]; x += dx)
				{
					*dst++ = src[x >> 15];
				}
			}
			//  missing break;   ????
		}

		case BITMAP_FORMAT_RGB32:
		{
			for (int y = 0; y < dstheight; y++)
			{
				bitmap_rgb32 &srcbitmap = *(bitmap_rgb32 *)m_scan_bitmaps[m_curbitmap][y];
				u32 *dst = &curbitmap.as_rgb32().pix32(y);
				const u32 *src = &srcbitmap.pix32(0);
				const int dx = (m_scan_widths[y] << 15) / dstwidth;
				for (int x = 0; x < dstwidth << 15; x += dx)
				{
					*dst++ = src[x >> 15];
				}
			}
			break;
		}
	}
}

[MooglyGuy]

You're absolutely right! Nice catch! 👍

I'm a bit busy installing the latest version of Visual Studio 2019, but as soon as that's done, I'll commit the fix. Many thanks!

[MooglyGuy]

Thanks again for the fix. This has been applied and credited to you.