Opened 16 years ago

Closed 16 years ago

Last modified 14 years ago

#965 closed defect (fixed)

music volume slider in F5 menu doesn't work

Reported by: SF/khalek Owned by: lavosspawn
Priority: normal Component: Engine: Sky
Keywords: Cc:
Game: Beneath a Steel Sky


latest cvs with Linux
The music volume slider moves but doesn't affect the
music volume.

Tested with both disk and cd version.

Ticket imported from: #768253. Ticket imported from: bugs/965.

Change History (7)

comment:1 Changed 16 years ago by lavosspawn

Resolution: fixed
Status: newclosed

comment:2 Changed 16 years ago by joostp

Owner: set to lavosspawn
Status: closednew

comment:3 Changed 16 years ago by joostp

Re-opened this because someone on irc (Wishbone) reported
that it was broken in the daily build of 11th of July and
because both the daily win32 build as well as current CVS
segfault for me at startup. (most likely same cause for the

Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 8192 (LWP 2708)]
0x08054908 in SkyAdlibMusic::setVolume(unsigned char)
volume=0 '\0') at sky/music/adlibmusic.cpp:55
55 _channels[cnt]->updateVolume(volume
| 128);

When commenting out this offending line (+ the corresponding
for-loop) the game runs again, but the slider obviously
doesn't work anymore either. :)

comment:4 Changed 16 years ago by lavosspawn

No, those two bugs don't have enything in common. It
crashed on your system, joost, because of a bug I newly
introduced (volume was set before any section was loaded,
so _numberOfChannels wasn't initialized).
For wishbone, I bet that he used an old CVS build.
I'll leave this bug open to have him verify it, but I'm sure that
it works with current CVS.

comment:5 Changed 16 years ago by lavosspawn

Status: newclosed

comment:6 Changed 16 years ago by SF/wishbone

No, I did NOT use an old CVS build, not unless noone had
updated the daily win32 snapshot in a looong time. I used the
snapshot from July 11th, off the download page. If it's
possible to get the program to tell me its version/build#, and
you can tell me how to do that, I'll be glad to supply it.


comment:7 Changed 14 years ago by fingolfin

Component: Engine: Sky
Game: Beneath a Steel Sky
Note: See TracTickets for help on using tickets.