Fwd: FogBugz (Case [Issue]13154) Pharo VM - BitBlt bug in alphaSourceBlendBits8
Thank you Nicolas, I verified the fix, it works. I took nicolas' change and build a windows pharo vm. changing the bit depth to 8bit littleendian (-8) works now. But there are two more issues (not related to this fix). 1. Changing Display depth to 4/8/16 shows horrible rendering artifacts (those do not occur on squeak) 2. Since refactoring some parts out of DisplayScreen into MorphicUIManager you can not change the depth from -XX to XX Display newDepth:8 has no effect if current display depth is -8. (This happens on squeak too, verified in Squeak 4.4/Squeak 4.5). BTW found this change for alphaSourceBlendBits8 its way into squeaks vm? I took the lates 4.5 allinone and the bug is still there. regards Nicolai ---------- Forwarded message ---------- From: Pharo Issue Tracker <do-not-reply@pharo.fogbugz.com> Date: 2014-03-30 15:49 GMT+02:00 Subject: FogBugz (Case [Issue]13154) Pharo VM - BitBlt bug in alphaSourceBlendBits8 To: "nicolaihess@web.de" <nicolaihess@web.de> [image: avatar] *Ulysse The Galactic Monkey From Outer Space* edited *Case 13154*: BitBlt bug in alphaSourceBlendBits8 <https://pharo.fogbugz.com/default.asp?13154>: *Bug* in [image: Project:] Pharo VM: 2. External * You are subscribed to this case Issue Validation Failed: https://ci.inria.fr/pharo/job/Pharo-3.0-Issue-Validator/14726//artifact/vali... Errors: 1 | *Status* changed: Resolved (Monkey is checking) [image: changed to] Work Needed [image: Priority] *Priority*: 5 - Fix If Time [image: Status] *Status*: Work Needed [image: Assigned To] *Assigned to*: Nicolas Cellier [image: Milestone] *Milestone*: Pharo3.0 Go to Case <https://pharo.fogbugz.com/default.asp?13154> No longer need updates? Unsubscribe from this case.<https://pharo.fogbugz.com/default.asp?pre=preUnsubscribe&pg=pgEditBug&comman...> Don't want FogBugz notifications anymore? Update your preferences.<https://pharo.fogbugz.com/default.asp?pg=pgPrefs> [image: FogBugz] A FogBugz case was edited by Ulysse The Galactic Monkey
From Outer Space. Case ID: 13154 Title: BitBlt bug in alphaSourceBlendBits8 Status: Work Needed Category: Bug Project: Pharo VM Area: 2. External Priority: 5 - Fix If Time Milestone: Pharo3.0: 30/03/2014 (Today) Assigned To: Nicolas Cellier URL: https://pharo.fogbugz.com/default.asp?13154 Changes: Status changed from 'Resolved (Monkey is checking)' to 'Work Needed'. Issue Validation Failed: https://ci.inria.fr/pharo/job/Pharo-3.0-Issue-Validator/14726//artifact/vali... Errors: 1 | You are subscribed to this case. If you do not want to receive automatic notifications in the future, unsubscribe (https://pharo.fogbugz.com/default.asp?pre=preUnsubscribe&pg=pgEditBug&comman...) from this case.
2014-03-31 23:58 GMT+02:00 Nicolai Hess <nicolaihess@web.de>:
Thank you Nicolas, I verified the fix, it works.
I took nicolas' change and build a windows pharo vm. changing the bit depth to 8bit littleendian (-8) works now.
hey, you made the major contribution : find the bug and suggest a correction! http://bugs.squeak.org/view.php?id=7247
But there are two more issues (not related to this fix). 1. Changing Display depth to 4/8/16 shows horrible rendering artifacts (those do not occur on squeak)
could it be related to issue 13057?
2. Since refactoring some parts out of DisplayScreen into MorphicUIManager you can not change the depth from -XX to XX Display newDepth:8 has no effect if current display depth is -8. (This happens on squeak too, verified in Squeak 4.4/Squeak 4.5).
Hmm, bad, but certainly not an operation tested everyday :)
BTW found this change for alphaSourceBlendBits8 its way into squeaks vm? I took the lates 4.5 allinone and the bug is still there.
regards
It's in both VMMaker and VMMaker.oscog head revisions for sure. But I don't know which VM was included in the distribution. Squeak VM production is not as engineered as the Pharo one, I think it's hand-crafted, and the MacOSX version lacks hands now, so it might be behind... Nicolai
---------- Forwarded message ---------- From: Pharo Issue Tracker <do-not-reply@pharo.fogbugz.com> Date: 2014-03-30 15:49 GMT+02:00 Subject: FogBugz (Case [Issue]13154) Pharo VM - BitBlt bug in alphaSourceBlendBits8 To: "nicolaihess@web.de" <nicolaihess@web.de>
[image: avatar] *Ulysse The Galactic Monkey From Outer Space* edited *Case 13154*: BitBlt bug in alphaSourceBlendBits8 <https://pharo.fogbugz.com/default.asp?13154>:
*Bug* in [image: Project:] Pharo VM: 2. External * You are subscribed to this case Issue Validation Failed: https://ci.inria.fr/pharo/job/Pharo-3.0-Issue-Validator/14726//artifact/vali... Errors: 1 | *Status* changed: Resolved (Monkey is checking) [image: changed to] Work Needed
[image: Priority] *Priority*: 5 - Fix If Time [image: Status] *Status*: Work Needed [image: Assigned To] *Assigned to*: Nicolas Cellier [image: Milestone] *Milestone*: Pharo3.0
Go to Case <https://pharo.fogbugz.com/default.asp?13154> No longer need updates? Unsubscribe from this case.<https://pharo.fogbugz.com/default.asp?pre=preUnsubscribe&pg=pgEditBug&comman...>
Don't want FogBugz notifications anymore? Update your preferences.<https://pharo.fogbugz.com/default.asp?pg=pgPrefs>
[image: FogBugz] A FogBugz case was edited by Ulysse The Galactic Monkey From Outer Space. Case ID: 13154 Title: BitBlt bug in alphaSourceBlendBits8 Status: Work Needed Category: Bug Project: Pharo VM Area: 2. External Priority: 5 - Fix If Time Milestone: Pharo3.0: 30/03/2014 (Today) Assigned To: Nicolas Cellier URL: https://pharo.fogbugz.com/default.asp?13154 Changes: Status changed from 'Resolved (Monkey is checking)' to 'Work Needed'. Issue Validation Failed: https://ci.inria.fr/pharo/job/Pharo-3.0-Issue-Validator/14726//artifact/vali... Errors: 1 | You are subscribed to this case. If you do not want to receive automatic notifications in the future, unsubscribe (https://pharo.fogbugz.com/default.asp?pre=preUnsubscribe&pg=pgEditBug&comman...) from this case.
2014-04-01 0:31 GMT+02:00 Nicolas Cellier < nicolas.cellier.aka.nice@gmail.com>:
2014-03-31 23:58 GMT+02:00 Nicolai Hess <nicolaihess@web.de>:
Thank you Nicolas,
I verified the fix, it works.
I took nicolas' change and build a windows pharo vm. changing the bit depth to 8bit littleendian (-8) works now.
hey, you made the major contribution : find the bug and suggest a correction! http://bugs.squeak.org/view.php?id=7247
But there are two more issues (not related to this fix). 1. Changing Display depth to 4/8/16 shows horrible rendering artifacts (those do not occur on squeak)
could it be related to issue 13057?
No. At least it is not solved by fixing that. Maybe it is related to the fast bitblitting, but I don't think so, and it is in pharo only.
2. Since refactoring some parts out of DisplayScreen into MorphicUIManager you can not change the depth from -XX to XX Display newDepth:8 has no effect if current display depth is -8. (This happens on squeak too, verified in Squeak 4.4/Squeak 4.5).
Hmm, bad, but certainly not an operation tested everyday :)
BTW found this change for alphaSourceBlendBits8 its way into squeaks vm? I took the lates 4.5 allinone and the bug is still there.
regards
It's in both VMMaker and VMMaker.oscog head revisions for sure. But I don't know which VM was included in the distribution. Squeak VM production is not as engineered as the Pharo one, I think it's hand-crafted, and the MacOSX version lacks hands now, so it might be
participants (2)
-
Nicolai Hess -
Nicolas Cellier