Code review comment for lp://qastaging/~larsu/unity-control-center/add-sound-indicator-visible-checkbox

Revision history for this message
Lars Karlitski (larsu) wrote :

> Small question, "Rename indicator_settings to sound_settings" ... should you
> clean out "GSettings *indicator_settings;"?

I do, don't I? It's not an added line in this diff because it shows the two commits together, but you can see it in the context:

66 g_clear_object (&dialog->priv->indicator_settings);

« Back to merge proposal