From fcedd75ab00195e6ab4eff556609d14004db3d8c Mon Sep 17 00:00:00 2001 From: Arun Thondapu Date: Wed, 12 Sep 2012 23:51:00 +0530 Subject: Bug 388369 Remove redundant else check in Label and helper function in Theme --- .../gtk/org/eclipse/swt/internal/theme/Theme.java | 20 +++++--------------- .../gtk/org/eclipse/swt/widgets/Label.java | 4 ++-- 2 files changed, 7 insertions(+), 17 deletions(-) diff --git a/bundles/org.eclipse.swt/Eclipse SWT Theme/gtk/org/eclipse/swt/internal/theme/Theme.java b/bundles/org.eclipse.swt/Eclipse SWT Theme/gtk/org/eclipse/swt/internal/theme/Theme.java index 569bd5b822..dddb83a991 100644 --- a/bundles/org.eclipse.swt/Eclipse SWT Theme/gtk/org/eclipse/swt/internal/theme/Theme.java +++ b/bundles/org.eclipse.swt/Eclipse SWT Theme/gtk/org/eclipse/swt/internal/theme/Theme.java @@ -37,7 +37,11 @@ public Theme(Device device) { progressHandle = OS.gtk_progress_bar_new(); toolbarHandle = OS.gtk_toolbar_new(); treeHandle = OS.gtk_tree_view_new_with_model(0); - separatorHandle = gtk_separator_new (OS.GTK_ORIENTATION_VERTICAL); + if (OS.GTK_VERSION >= OS.VERSION (3, 0, 0)) { + separatorHandle = OS.gtk_separator_new (OS.GTK_ORIENTATION_VERTICAL); + } else { + separatorHandle = OS.gtk_vseparator_new(); + } labelHandle = OS.gtk_label_new(null); OS.gtk_container_add (fixedHandle, labelHandle); OS.gtk_container_add (fixedHandle, frameHandle); @@ -215,18 +219,4 @@ void transferClipping(GC gc, int /*long*/ style) { OS.gdk_region_destroy(clipping); } } - -int /*long*/ gtk_separator_new (int orientation) { - int /*long*/ separator = 0; - if (OS.GTK_VERSION >= OS.VERSION(3, 0, 0)) { - separator = OS.gtk_separator_new (orientation); - } else { - if (orientation == OS.GTK_ORIENTATION_HORIZONTAL) { - separator = OS.gtk_hseparator_new (); - } else if (orientation == OS.GTK_ORIENTATION_VERTICAL) { - separator = OS.gtk_vseparator_new (); - } - } - return separator; -} } diff --git a/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/Label.java b/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/Label.java index 345295fae7..d7f8ab56ee 100644 --- a/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/Label.java +++ b/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/Label.java @@ -587,12 +587,12 @@ void showWidget () { int /*long*/ gtk_separator_new (int orientation) { int /*long*/ separator = 0; - if (OS.GTK_VERSION >= OS.VERSION(3, 0, 0)) { + if (OS.GTK_VERSION >= OS.VERSION (3, 0, 0)) { separator = OS.gtk_separator_new (orientation); } else { if (orientation == OS.GTK_ORIENTATION_HORIZONTAL) { separator = OS.gtk_hseparator_new (); - } else if (orientation == OS.GTK_ORIENTATION_VERTICAL) { + } else { separator = OS.gtk_vseparator_new (); } } -- cgit