You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Reported in version: 1.2.15 Reported for operating system, platform: Linux, x86
Comments on the original bug report:
On 2013-07-19 17:48:15 +0000, Sven Hesse wrote:
Created attachment 1243
Trivial patch to check the XStoreColors return value
Here on my laptop with an ATI Mobility Radeon 9600 PRO Turbo (RV350), the free radeon drivers and X.org 1.12.4, SDL_SetGamma() fails because it tries to use SDL_SetGammaRamp() if available. SDL_SetGammaRamp() uses XStoreColors(), and that call always fails with BadRequest (for whatever reason, I don't know). The SDL_Visual class check goes through, though.
Now, XStoreColors failing is maybe another bug in SDL, or in Xorg, or in the radeon drivers or anywhere else, but I think SDL should still check that the XStoreColors() doesn't error as well.
For extra clarity and convinence, I'll attach a patch, but it's quite trivial.
Hi,
This patch (3665bc284271), tested vi hg SDL-1.2,
breaks brightness with Quakespasm on my Nvidia 260.19.29, fedora 14 amd64 box.
To reproduce, run quakespasm in windowed mode, set in-game brightness around 80%, and change video resolution via hitting Alt-Enter, or the in-game menus.
The game and X display straight away get very bright, and remain that way till app is closed.
cheers, Steven
On 2014-05-01 12:25:18 +0000, Sven Hesse wrote:
Interesting.
For the record: where I noticed this bug originally was with Neverwinter Nights. Without the patch, the brightness controls in NWN do nothing on the RV350 laptop. With the patch, they work.
On 2014-05-01 12:31:51 +0000, Steven wrote:
This bug escaped my attention for a while because at low gamma settings, it is not very evident. You have to make the gamma around 80% for it to be really obvious.
It also seems to make our gamma slider not linear. Move the brightness slider up and down, and brightness is kindof all over the place.
On 2014-05-01 13:30:40 +0000, Ozkan Sezer wrote:
The thing is, quakespasm is calling SDL_SetColors, not SDL_SetGammaRamp.
If SetGammaRamp function of SDL's driver (in this case X11_SetGammaRamp)
fails, SDL_SetColors calls the SetColors function of its driver (in this
case X11_SetColors): I guess that's where the problem is.
On 2014-05-01 15:21:43 +0000, Ozkan Sezer wrote:
OK, here's the thing: I noticed that with SDL-1.2.15 and older, my
brightness does actually work, and with SDL-1.2-hg my display gets
almost twice as bright.
... and I saw on my terminal "XStoreColors returned 1", i.e. BadRequest.
On the other hand, my man page for XStoreColors() says that only
BadAccess (10), BadColor (12), BadName (15), or BadValue (2) would be
generated.
Unless the above information is changed/wrong, this is possibly a driver
error (mine is an AMD Radeon HD 7700 series, firegl 13.4 driver, I think)
and SDL is ending up appling the gamma value twice.
On 2014-05-01 15:26:17 +0000, Sven Hesse wrote:
And my laptop generates a BadRequest while /not/ applying the gamma value?
On 2014-05-01 15:34:48 +0000, Ozkan Sezer wrote:
(In reply to Sven Hesse from comment # 7)
And my laptop generates a BadRequest while /not/ applying the gamma value?
Oh, I noticed that you also noticed the returned error code. (Damn bugzilla
not wrapping the long lines..) I surely cannot speak for your setup, but I
described what mine does. The brightness does actually work, even though I
may want to set my gamma value to rather small numbers such as 0.7 or 0.6.
I don't have a NWN setup ready to test what it does. Besides the man page
clearly documents what error codes can be returned and 1 (RadRequest) is not
one of them. So I don't know what the good solution would be.
And my laptop generates a BadRequest while /not/ applying the gamma value?
[...] I surely cannot speak for your setup, but I
described what mine does. [...]
Clarification: What I described was with X server 1.5.2. Went onto a laptop
with X server 1.11.4 + x.org radeon drivers, brightness changing indeed does
not work without a patched SDL.
On 2015-08-25 09:38:22 +0000, Ryan C. Gordon wrote:
Hello, and sorry if you're getting several copies of this message by email, since we are closing many bugs at once here.
We have decided to mark all SDL 1.2-related bugs as RESOLVED ENDOFLIFE, as we don't intend to work on SDL 1.2 any further, but didn't want to mark a large quantity of bugs as RESOLVED WONTFIX, to clearly show what was left unattended to and make it easily searchable.
Our current focus is on SDL 2.0.
If you are still having problems with an ENDOFLIFE bug, your absolute best option is to move your program to SDL2, as it will likely fix the problem by default, and give you access to modern platforms and tons of super-cool new features.
Failing that, we will accept small patches to fix these issues, and put them in revision control, although we do not intend to do any further official 1.2 releases.
Failing that, please feel free to contact me directly by email (icculus@icculus.org) and we'll try to find some way to help you out of your situation.
Thank you,
--ryan.
The text was updated successfully, but these errors were encountered:
This bug report was migrated from our old Bugzilla tracker.
These attachments are available in the static archive:
Reported in version: 1.2.15
Reported for operating system, platform: Linux, x86
Comments on the original bug report:
On 2013-07-19 17:48:15 +0000, Sven Hesse wrote:
On 2013-07-21 14:10:28 +0000, Sam Lantinga wrote:
On 2014-05-01 12:17:02 +0000, Steven wrote:
On 2014-05-01 12:25:18 +0000, Sven Hesse wrote:
On 2014-05-01 12:31:51 +0000, Steven wrote:
On 2014-05-01 13:30:40 +0000, Ozkan Sezer wrote:
On 2014-05-01 15:21:43 +0000, Ozkan Sezer wrote:
On 2014-05-01 15:26:17 +0000, Sven Hesse wrote:
On 2014-05-01 15:34:48 +0000, Ozkan Sezer wrote:
On 2014-05-01 15:58:53 +0000, Ozkan Sezer wrote:
On 2014-05-01 16:42:41 +0000, Ozkan Sezer wrote:
On 2015-08-25 09:38:22 +0000, Ryan C. Gordon wrote:
The text was updated successfully, but these errors were encountered: