Bug 171027 Emtpy space created when calling setEnabled on encoding field editor
I20070116-1510 FieldEditor.setEnabled(boolean) is implemented in a pretty strange way: it creates the label if not yet created. This has side effects for clients that do not need/create that label but later call setEnabled(...). An example for such a subclass is AbstractEncodingFieldEditor. Now, fixing this in FieldEditor.setEnabled(...) might be problematic because currently for some clients the label might only appear as a side-effect of calling setEnabled(...) during initialization. A safer fix might be to simply not call super.setEnabled(...) in AbstractEncodingFieldEditor.setEnabled(...).
bundles/org.eclipse.ui.ide/src/org/eclipse/ui/ide/dialogs/AbstractEncodingFieldEditor.java
return current == null || current.length() == 0;
		}
		return encodingSetting.equals(current);
	}
	/**
	 * Return whether or not the default has been selected.
	 * 
	 * @return <code>true</code> if the default button has been selected.
	 */
	boolean isDefaultSelected() {
		return defaultEncodingButton.getSelection();
	}
	/**
	 * Set the title of the group to groupTitle. If this is not called a default
	 * title is used. If groupTitle is <code>null</code> the control will be
	 * unlabelled (by default a {@link Composite} instead of a {@link Group}.
	 * 
	 * <strong>NOTE</strong> this value must be set before
	 * {@link #createControl(Composite)} is called or it will be ignored.
	 * 
	 * @param groupTitle
	 *            The groupTitle to set.
     * @since 3.3
	 */
	public void setGroupTitle(String groupTitle) {
		this.groupTitle = groupTitle;
	}
}