Bug 71153 Is this a mistake in SourceActionDialog.java
org.eclipse.jdt.ui/ui/org/eclipse/jdt/internal/ui/dialogs/SourceActionDialog.java CVS Revision 1.24 from line 364 to 379 commentButton.setLayoutData(new GridData(GridData.HORIZONTAL_ALIGN_FILL)); ... ... GridData gd= new GridData(GridData.HORIZONTAL_ALIGN_FILL); gd.horizontalSpan= 2; commentButton.setLayoutData(gd); which constructs two GridData objects for commentButton.
org.eclipse.jdt.ui/ui/org/eclipse/jdt/internal/ui/dialogs/SourceActionDialog.java
}
	
	private List convertToIntegerList(int[] array) {
		List result= new ArrayList(array.length);
		for (int i= 0; i < array.length; i++) {
			result.add(new Integer(array[i]));
		}
		return result;
	}	
	
	protected Composite createVisibilityControl(Composite parent, final IVisibilityChangeListener visibilityChangeListener, int[] availableVisibilities, int correctVisibility) {
		List allowedVisibilities= convertToIntegerList(availableVisibilities);
		if (allowedVisibilities.size() == 1)
			return null;
		
		Group group= new Group(parent, SWT.NONE);
		group.setText(ActionMessages.getString("SourceActionDialog.modifier_group")); 
		GridData gd= new GridData(GridData.FILL_BOTH);
		group.setLayoutData(gd);
		GridLayout layout= new GridLayout();
		layout.makeColumnsEqualWidth= true;
		layout.numColumns= 4; 
		group.setLayout(layout);
		
		String[] labels= new String[] {
			ActionMessages.getString("SourceActionDialog.modifier_public"), 
			ActionMessages.getString("SourceActionDialog.modifier_protected"), 
			ActionMessages.getString("SourceActionDialog.modifier_default"), 
			ActionMessages.getString("SourceActionDialog.modifier_private"), 
		};