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

Always initialize upd3301 attr_extend_info, as the loop isn't guaranteed to set all entries. I don't know what real hardware would do, but it triggered a compiler warning. [smf]

Browse Source
This commit is contained in:
smf- 2022-05-18 19:15:39 +01:00
parent 9863cbfcfd
commit e6e2a137e9
1 changed files with 1 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/devices/video/upd3301.cpp
View File

@ -604,15 +604,11 @@ int upd3301_device::vrtc_r()
UPD3301_FETCH_ATTRIBUTE( upd3301_device::default_attr_fetch )
{
const u8 attr_max_size = 80;
std::array<u16, attr_max_size> attr_extend_info;
std::array<u16, attr_max_size> attr_extend_info = {0};
// elthlead (pc8801) uses b&w no attributes/no special control mode
// 0-fill buffer seems enough
if (m_gfx_mode == 1)
{
std::fill(attr_extend_info.begin(), attr_extend_info.end(), 0x00);
return attr_extend_info;
}
// TODO: may actually fetch in LIFO order
// Some edge cases in pc8801 N88 Basic (status on bottom), jettermi and play6lim backs up this theory.