- --
Viewing Issue Advanced Details
[ Jump to Notes ]
ID | Category [?] | Severity [?] | Reproducibility | Date Submitted | Last Update |
---|---|---|---|---|---|
07411 | Interface | Minor | Always | Sep 8, 2019, 12:46 | Sep 9, 2019, 18:22 |
Tester | hap | View Status | Public | Platform | MAME (Official Binary) |
Assigned To | Resolution | Open | OS | Windows 10 (64-bit) | |
Status [?] | Acknowledged | Driver | |||
Version | 0.213 | Fixed in Version | Build | 64-bit | |
Fixed in Git Commit | Github Pull Request # | ||||
Summary | 07411: INP header size is too small | ||||
Description |
INP header size is too small. emu/ioport.h(see line 756) reserves 12 bytes for sysname, 32 bytes for appdesc. It is outdated: MAME increased maximum sysname size (AFAIK to 16 bytes?). Appdesc max size is 33 bytes, 35 bytes if we ever go from MAME 0.999 to 0.1000. I suppose it's an easy fix, simply increase the header size. But I expect this to break every INP. |
||||
Steps To Reproduce |
mame64 vendetta2peba -record a mame64 vendetta2peba -playback a reports warning: Input file is for machine 'vendetta2peb', not for current machine 'vendetta2peba' |
||||
Additional Information | Bug reported by Haze on shoutbox, I'm reporting it here in case it's forgotten about. | ||||
Github Commit | |||||
Flags | |||||
Regression Version | |||||
Affected Sets / Systems | |||||
Attached Files
|
|||||
Relationships
There are no relationship linked to this issue. |
Notes
0
There are no notes attached to this issue. |