Code review comment for lp://qastaging/~desrt/indicator-datetime/timedated

Revision history for this message
Charles Kerr (charlesk) wrote :

Thanks for doing the upstream patch!

It looks like the updated prefs patch here has an accidental copy-paste on line 261, we probably don't want to call toggle_ntp() twice each time priv->auto_radio is toggled.

« Back to merge proposal