Code review comment for lp:~hui.wang/unity-settings-daemon/unity-settings-daemon

Revision history for this message
David Henningsson (diwic) wrote :

> David, could you comment on whether "GVC_IS_MIXER_SINK (stream)" is the right way to check if the right way to tell a mic appart from an output?

I suppose so, since it is already used that way a few lines down.

In short, assuming Hui has properly tested his patch, I see no problems with it.

Nitpick: perhaps get_icon_name_for_volume can now be refactored to use the global definitions of "icon_names" and "mic_icon_names" instead of copy-pasted local ones.

review: Approve

« Back to merge proposal