Ticket #559 (Fixed)Mon Oct 17 12:14:40 UTC 2022
Screen Setup allows selecting an MDF with no modes
| Reported by: | Cameron Cawley (3514) | Severity: | Normal |
| Part: | RISC OS: Application | Release: | |
| Milestone: | Status | Fixed |
Details by Cameron Cawley (3514):
The implementation of GV_IICOp returns a default EDID block that is modified slightly based on the display capabilities, but does not contain any video modes. As a result, the EDID option shows up in !Configure, but produces a “No mode definitions found in file” error when selecting it. Despite that, it’s still possible for this to be selected and set anyway.
Changelog:
Modified by Stuart Painting (5389) Fri, October 21 2022 - 11:50:42 GMT
- Attachment added: dud1.txt
The bug is slightly more wide-ranging, in that !Configure will allow an obviously-dud MDF (example MDF attached) to be chosen and set despite the error “No mode definitions found in file” appearing. Tested on RISC OS 3.71, RISC OS 4.39, RISC OS 5.28 (RPCEmu 0.9.4).
!Configure shouldn’t accept an MDF if this error occurs: the previously-selected MDF should still be current. As I see it, there are two options:
1. !Configure should reject an MDF on any error, or
2. The error box for “No mode definitions found in file” should have “Cancel” and “Use anyway” buttons instead of the existing “OK” button.
Modified by Sprow (202) Sun, October 23 2022 - 13:38:41 GMT
- Summary changed from VIDC20Video driver returns invalid EDID data to Screen Setup allows selecting an MDF with no modes
- Part changed from Unspecified to RISC OS: Application
VIDC20Video intentionally isn’t returning a complete set of EDID modes, because it can’t infer them from the information it has – the dummy EDID block is more so you can do an GraphicsV_IICOp on any RISC OS 5 driver and get /something/ back.
So really the issue is that Screen Setup allows you to select an MDF with no modes, which is a bit silly.
Modified by Sprow (202) Tue, February 21 2023 - 07:52:35 GMT
- Status changed from Open to Fixed
Fixed in Screen Setup 0.45 (now reverts to the Auto mode if an empty MDF is selected).
