#6108 closed defect (fixed)
WII: Zak FM-TOWNS mouse cursor encased in blue box
| Reported by: | (none) | Owned by: | lordhoto |
|---|---|---|---|
| Priority: | normal | Component: | Engine: SCUMM |
| Version: | Keywords: | ||
| Cc: | Game: | Zak McKracken |
Description
ScummVM ver: 1.5.0 and latest nightly build Game ver: FM-TOWNS English Platform: Wii
The mouse cursor for the game is encased in a blue box! Makes it difficult to see where you're clicking.
Ticket imported from: #3551156. Ticket imported from: bugs/6108.
Attachments (1)
Change History (19)
comment:1 by , 12 years ago
| Owner: | set to |
|---|
comment:2 by , 12 years ago
comment:3 by , 12 years ago
Abram: Thank you for the bug report. Please can you confirm the following: 1. Please confirm if this bug is present when playing Beneath a Steel Sky or Flight of the Amazon Queen? Both these games are freely downloadable from our website and different engines from ZAK. This is to confirm if this is an engine/game specific issue or occurs with all games on the Wii. 2. Please confirm if this bug was present in ealier versions of ScummVM on the Wii i.e. v1.4.1, 1.3.1 etc. This helps us to work out if this is a recent regression and approximately when this issue was introduced.
dhewg: This sounds like an issue with the mouse overlay. Can you replicate?
comment:4 by , 12 years ago
I tested both of the games you requested and found that the mouse cursor in each is displaying normally. Might this be a bug with FM-TOWNS games? I didn't find 1.4.1 or 1.4.0 Wii builds here on SourceForge, but I tested the 1.3.1 build and saw that the mouse cursor has no blue box in that version.
comment:5 by , 12 years ago
LoI have the same problem with Monkey Island 1 (FMTOWNS) on the Android port (v1.5.0). The same game on my debian desktop works fine using scummvm 1.4.1.
Samr problem indiana jones ñast crusade fmtowns on android
comment:7 by , 12 years ago
Also occurs in Android (4.2.1) version of ScummVM (1.5.0) when playing Loom FMTOWNS on a Nexus 7.
comment:8 by , 12 years ago
I have the same problem with the FM Towns versions of Zak McKracken on Samsung Galaxy S3.
comment:9 by , 11 years ago
Same problem with: Indiana last crusade FM-TOWNS and NEXUS 10, we need a fix :´(
comment:10 by , 11 years ago
Confirm the blue box bug (=BBB) in all FM- Towns Versions: Monkey Island I + II, Indiana Jones III, Look, Zak
Device: Nexus 10
Stell Sky + Amazon Queen work perfectly.
Would be great to See it fixed... Thanks!
comment:11 by , 11 years ago
@exitido: You may need a fix, but we are not able to provide one unless we can track down the cause of the bug. For that, debugging information is needed...
I have tried replicating this with the latest master on x86_64 with Blues Art Activities with no success. I don't own any FM-Towns versions. They are very rare and expensive to buy and thus only a handful of the developers own copies and can look at this bug.
There has been some discussion on this today and I am posting the following notes at the request of fuzzie and LordHoto:
comment:12 by , 11 years ago
Lordhoto said "that the OpenGL backend carefully does the inverse logic and that doesn't have any blue blox around the Loom FM-Towns cursor at least."
Unfortunately I don't think he has a Wii or Android phone to test with...
comment:13 by , 11 years ago
We would normally be able to use a demo to test this, but all the FM-TOWNS demos listed are non-interactive and thus have no cursors :/
Overall, this bug is a pain.
But to progress..
-
Can someone replicate this bug and take a screenshot or photo of the screen and attach it to this bug as a png or jpg file please? The exact color may give the developers a clue as to how the surface logic is failing.
-
Can someone with an Android device replicate the bug following the steps given here to get an Android Debug Log and see if there are any warnings, errors or other information messages associated with this problem: http://wiki.scummvm.org/index.php/Debugging_ScummVM#Android
-
Can someone test with the latest development v1.7.0git Android build from here and see if the problem is still occuring: http://buildbot.scummvm.org/builds.html
Instructions on installing daily builds are given here: http://wiki.scummvm.org/index.php/Android#Daily_Builds
-
Can someone do the same i.e. 3, but for the Wii v1.7.0git nightly build and see if the problem is still occurring there.
-
If someone with a Wii, an FM-Towns game and good C++ skills could set up the compiler toolchain as per: http://wiki.scummvm.org/index.php/Compiling_ScummVM/Wii and then try building v1.4.1 and v1.5.0 and confirm that the error was introduced between these two revisions (as we only currently have it localised between v1.3.1 and v1.5.0).
-
If they could then try doing a Git bisection and test and locate the regression commit... or at least reduce the range to a much smaller range of commits.
comment:14 by , 11 years ago
AHA... Had a brainwave and found a likely solution, by reversing the debugging problem.
If I can't fix the Android/Wii backend code... I'll break the OpenGL one instead!!! :)
To prove if Lordhoto's theory that the cause is the Android/Wii colorKey handling in setMouseCursor() function of OSystem, I have transplanted the core logic of the colorkey handling from backends/platform/android/gfx.cpp lines 758 to 763 as indicated to the relevant code in backends/graphics/opengl/opengl-graphics.cpp.
The resulting code can be found in a branch of my github repo: https://github.com/digitall/scummvm/tree/antiFixForBug6108
This should then provoke the same bug when using the OpenGL Graphics Mode when running Blue's Art Time Activities which I have a copy of, which it DOES!
I attach a screenshot to show the observed behaviour.
I have also asked Kirben who owns Zak FM-Towns to test with this and he has observed the Blue Box around the Cursor as described.
Hopefully, LordHoto can reverse my fix and apply the correct code from the OpenGL backend to the Android and Wii backend for testing.
by , 11 years ago
| Attachment: | antiFixBlueArt.jpg added |
|---|
comment:16 by , 11 years ago
| Owner: | changed from to |
|---|---|
| Resolution: | → fixed |
| Status: | new → pending |
comment:17 by , 11 years ago
Should be fixed with 4ca33e264802ec4c679c8b129ca461bb1e9c7f8d in master.
comment:18 by , 11 years ago
| Status: | pending → closed |
|---|
Assigning to Wii porter for visibility.