Skip to content

fix(backfill): restore MUI 3.x equality guard, gated on lib_Version#3

Merged
midwan merged 1 commit intomasterfrom
fix/mui-3x-backfill-guard
May 1, 2026
Merged

fix(backfill): restore MUI 3.x equality guard, gated on lib_Version#3
midwan merged 1 commit intomasterfrom
fix/mui-3x-backfill-guard

Conversation

@midwan
Copy link
Copy Markdown
Owner

@midwan midwan commented May 1, 2026

Summary

  • Commit 6e678e4 removed the original "stuntzi" MUIM_Backfill early-return guard so MUI 4/5 would stop rendering as a blue checkerboard. That fix regressed MUI 3.8/3.9, which still need the guard to avoid painting outside the gadget bounds.
  • Restore the strict-equality guard in mcc/Dispatcher.cpp, but gate it on MUIMasterBase->lib_Version <= 20 (MUI 3.9 = v20, MUI 3.8 = v19). MUI 4+ (v21+) keeps the unguarded path that fixed the checkerboard.
  • Verified working on real MUI 3.8 and MUI 5; OS3 cross-build (sacredbanana/amiga-compiler:m68k-amigaos) compiles clean.

Test plan

  • make OS=os3 rebuilds HTMLview.mcc cleanly
  • Renders correctly under MUI 3.8 (no out-of-box drawing)
  • Renders correctly under MUI 5 (no blue checkerboard)
  • CI green for OS3 / OS4 / MorphOS

Commit 6e678e4 removed the long-standing "stuntzi" early-return guard in
the MUIM_Backfill handler so MUI 4/5 would stop rendering as a blue
checkerboard. That regressed MUI 3.8 / 3.9, which need the guard to
avoid painting outside the gadget bounds.

Restore the guard but only apply it when MUIMasterBase->lib_Version <= 20
(MUI 3.9 = v20, MUI 3.8 = v19). MUI 4+ (v21+) keeps the unguarded path
introduced for the checkerboard fix. Verified working on both MUI 3.8
and MUI 5.
@midwan midwan self-assigned this May 1, 2026
@midwan midwan merged commit fc86e16 into master May 1, 2026
4 checks passed
@midwan midwan deleted the fix/mui-3x-backfill-guard branch May 1, 2026 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant