Bug 138626

Summary:	master channel config value ignored
Product:	[Applications] kmix	Reporter:	Daniel Reichhard <drei>
Component:	general	Assignee:	Christian Esken <esken>
Status:	RESOLVED WORKSFORME
Severity:	normal	CC:	ovit.debian
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Ubuntu
OS:	Linux
Latest Commit:		Version Fixed In:
Attachments:	fixes problem with loading right config for more than one sound card fixes problem with loading right config for more than one sound card

Description Daniel Reichhard 2006-12-10 14:36:54 UTC

Version:            (using KDE KDE 3.5.5)
Installed from:    Ubuntu Packages
OS:                Linux

i have two sound cards, and whenever i set the master channel to the second one, the value is stored to kmixrc correctly, but kmix doesn't honor the settings on the next start. the master channel is always set to the default (first) sound card it can find.

Comment 1 Daniel Reichhard 2006-12-10 17:57:03 UTC

Created attachment 18877 [details]
fixes problem with loading right config for more than one sound card

this works for me; the problem was that setMasterCard() tried to use the
"mixer" list before it has been entirely filled

Comment 2 Christian Esken 2006-12-10 21:41:53 UTC

Already fixed. Please see https://bugs.kde.org/show_bug.cgi?id=136378#c33 . The patch I did is pretty much identical, but instead of using the defaultPrimaryKey thing, I am just using the first mixer via Mixer::mixers().first()->id() .

Comment 3 Daniel Reichhard 2006-12-10 22:33:12 UTC

Created attachment 18884 [details]
fixes problem with loading right config for more than one sound card

the patch from https://bugs.kde.org/show_bug.cgi?id=136378#c33 doesn't solve my
problem, because it still calls setMasterCard before the mixer structure is
filled completely. setMasterCard has to be called after the loop over the sound
cards, i believe.

this patch now uses Mixer::mixers().first()->id() in setMasterCard, similar to
the patch mentioned above

Comment 4 Daniel Reichhard 2006-12-10 22:34:40 UTC

reopening - reason is in #3

Comment 5 Christian Esken 2006-12-11 21:03:09 UTC

Hi, why do you claim that "it still calls setMasterCard before the mixer structure is filled completely". It is not. Have you read and understood the patch?!?

Comment 6 Daniel Reichhard 2006-12-12 02:45:27 UTC

Sorry for any inconvenience, i was a bit to hasty...
of course it works.

regards, daniel

Comment 7 Christian Esken 2007-03-02 15:51:06 UTC

*** Bug 142409 has been marked as a duplicate of this bug. ***