callisto-debian/gtk3
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fontchooser: Remove useless code

Browse Source 
gtk_adjustment_set_value() clamps itself, no need to do the clamping.

But I was impressed at the amount of comments.
This commit is contained in:
Benjamin Otte
2011-09-19 20:37:38 +02:00
parent e465023051
commit a4c3271854
1 changed files with 1 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
gtk/gtkfontchooserwidget.c
View File

@ -329,22 +329,6 @@ spin_change_cb (GtkAdjustment *adjustment,
else else
pango_font_description_set_size (priv->font_desc, ((gint)size) * PANGO_SCALE); pango_font_description_set_size (priv->font_desc, ((gint)size) * PANGO_SCALE);
/* If the new value is lower than the lower bound of the slider, we set
* the slider adjustment to the lower bound value if it is not already set
*/
if (size < gtk_adjustment_get_lower (slider_adj) &&
gtk_adjustment_get_value (slider_adj) != gtk_adjustment_get_lower (slider_adj))
gtk_adjustment_set_value (slider_adj, gtk_adjustment_get_lower (slider_adj));
/* If the new value is upper than the upper bound of the slider, we set
* the slider adjustment to the upper bound value if it is not already set
*/
else if (size > gtk_adjustment_get_upper (slider_adj) &&
gtk_adjustment_get_value (slider_adj) != gtk_adjustment_get_upper (slider_adj))
gtk_adjustment_set_value (slider_adj, gtk_adjustment_get_upper (slider_adj));
/* If the new value is not already set on the slider we set it */
else if (size != gtk_adjustment_get_value (slider_adj))
gtk_adjustment_set_value (slider_adj, size); gtk_adjustment_set_value (slider_adj, size);
gtk_widget_queue_draw (priv->preview); gtk_widget_queue_draw (priv->preview);