05625: msx2: Astro Star Demo hangs when dipswitch Render Resolution is set to "auto"

ID	Category [?]	Severity [?]	Reproducibility	Date Submitted	Last Update
05625	Graphics	Minor	Always	Jul 10, 2014, 09:58	Aug 3, 2015, 07:23

Tester	Dr.Venom	View Status	Public	Platform	MESS (Official Binary)
Assigned To		Resolution	Open	OS	Windows Vista/7/8 (64-bit)
Status [?]	Acknowledged			Driver	msx/msx.cpp
Version	0.153	Fixed in Version		Build
		Fixed in Git Commit		Github Pull Request #

Summary	05625: msx2: Astro Star Demo hangs when dipswitch Render Resolution is set to "auto"
Description	When the dipswitch "Render Resolution" is set to "auto" the Astro Star demo will hang upon loading the main demo part.
Steps To Reproduce	Set MESS Render Resolution dipswitch to "Auto" Start the Astro Star demo The demo will hang after completing the intro picture, i.e. upon starting the main demo part.
Additional Information	The demo hangs between intro picture and main demo part, which is also where the Auto resolution switch tries to switch between 272x240 intro picture screen and a (linedoubled) 544x480 screen. As such this issue seems part of the whole resolution, borders and refresh rate problems in the MSX driver, for which separate bug reports have been filed earlier. Demo can be downloaded here: http://www.passionmsx.org/modules/mydownloads/singlefile.php?cid=67&lid=688 The screen where it hangs only shows the first character of the screen it's supposed to show, see attached pic. Correct picture is shown here: http://www.passionmsx.org/modules/mydownloads/images/shots/astrost.png When setting the Render Resolution dipswitch to "High" instead of "Auto" the problem is circumvented.
Github Commit

Flags	Verified with Original
Regression Version
Affected Sets / Systems	msx2

Attached Files	0001.png (1,952 bytes) Jul 10, 2014, 09:58 Uploaded by Dr.Venom

No.11935 wilbert Developer Aug 3, 2015, 07:23	I have removed the high/low/auto setting from the v9938 code entirely. The code draws everything according to the old high settings now. This keeps the code simpler and prevents bugs like this one.