randyrossi / bmc64

A bare metal Commodore 64 emulator for the Raspberry Pi with true 50hz/60hz smooth scrolling, low input latency and better audio/video sync.
GNU General Public License v3.0
474 stars 56 forks source link

40/80 column switch not honored on C128 autostart #226

Open rhester72 opened 1 year ago

rhester72 commented 1 year ago

Steps to reproduce:

Put 40/80 column switch in DOWN position Reset, confirm output is to VDC Autostart any disk

Expected result:

Autostart comes up with output to VDC (because 80 column switch is depressed)

Actual result:

Autostart comes up with output to VIC-II

Confirmed as a VICE upstream issue in the base release BMC64 uses but corrected by 3.6.1, looking to see if a backported patch is possible.

rhester72 commented 1 year ago

Not a bug per se - reason it's this way is because "screen reading" is required for autostart and it was not implemented in any 80-column displays in VICE 3.3. Massive rework over time was required to make it work (culminating in late 2019 with https://sourceforge.net/p/vice-emu/code/37166/) that is not practical to backport for such a minor issue/edge case.

Self-closing as WONTFIX.

randyrossi commented 1 year ago

Can you try the master version here: https://accentual.com/bmc64/downloads/master/

This sounds like something I fixed recently but haven't moved the file to 4.0 version yet.

On Fri, Oct 7, 2022 at 11:47 AM rhester72 @.***> wrote:

Steps to reproduce:

Put 40/80 column switch in DOWN position Reset, confirm output is to VDC Autostart any disk

Expected result:

Autostart comes up with output to VDC (because 80 column switch is depressed)

Actual result:

Autostart comes up with output to VIC-II

I originally thought this might somehow be related to 3ec5b07 https://github.com/randyrossi/bmc64/commit/3ec5b077d41aba92b96311b31ff70921a46edec5, but it is definitely not.

— Reply to this email directly, view it on GitHub https://github.com/randyrossi/bmc64/issues/226, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAI3HKA3BGYMGNQUGHMSVHDWCBAZFANCNFSM6AAAAAAQ7WYYBA . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- Randy Rossi

rhester72 commented 1 year ago

Sorry it took so long to test! This does not appear to be fixed in the master build as it appears on the server (which, incidentally, is from February 28 inside the files zip and image zip).

Am I testing the wrong pre-release or is it possible it wasn't fixed after all?

Exact steps I'm using to reproduce: