#4403 closed defect (fixed)
GUI: "Help" dialog causes crash @ 320x200
| Reported by: | sunmax | Owned by: | sev- |
|---|---|---|---|
| Priority: | high | Component: | GUI |
| Version: | Keywords: | ||
| Cc: | Game: |
Description
When you choose "help" from GMM on a scumm based game (eg. indy4, samnmax, etc.) running @ 320x200, it will crash:
Widget <ScummHelp.Close> has y > 200! Error: assertion `isValidRect()' failed in ../../../common/rect.h:
Max (Horn) was able to reproduce it on a SDL backend, please get in touch with him if you need details on that.
Ticket imported from: #2820514. Ticket imported from: bugs/4403.
Attachments (3)
Change History (10)
comment:1 by , 15 years ago
| Summary: | help crash on backend @ 320x200 → GUI: "Help" dialog causes crash @ 320x200 |
|---|
comment:2 by , 15 years ago
comment:3 by , 15 years ago
Just FYI I also dropped tanoku a line, maybe he knows how to fix all of the renderer properly, the VectorRendererSpec code could need some cleanup / comments :-/.
by , 15 years ago
| Attachment: | classic_320x200.png added |
|---|
screenshot of the missing 'outlines' with scummclassic at 320x200
comment:4 by , 15 years ago
Hello gentlemen. This bug is caused because the beveled and rounded square outlines were being drawn on the outside of their shapes, instead of their inside. Revision 42529 changes this and adds a few changes to the STX files to the themes look mostly the same. However, I'm quite sure there have been some regressions (i.e. minor graphical glitches) because of the change.
If you were so kind to report them, I'll attempt to fix them. Feel free to open a new ticket and assign it to me, or just comment here.
Cheers.
comment:5 by , 15 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:6 by , 15 years ago
Actually, disregard that. That commit has been reverted and the bug is now fully fixed in revision 42537 with supposedly no regressions. Cheers.
comment:7 by , 5 years ago
| Component: | --Unset-- → GUI |
|---|
I wasn't able to reproduce the "isValidRect" assertion, but I noticed that the dialog is both in 320x240 and in 320x200 unusable for me. I can't click the buttons. That happens in both modern and classic theme for me. It seems that this is caused by invalid memory writes. I will attach valgrind logs for both modern and classic theme in 320x200.
Somehow the render does not only draw within the given bounds (320x200), but also draws outside of it. Easily noticeable in classic theme, where the right and bottom gray line of the dialog is missing.