if (action == null) { throw new IllegalArgumentException("Action == null"); } return _textPopupMenu.add(action);
if (tab == null) { throw new IllegalArgumentException("Null IMainPanelTab passed"); } return _mainTabPane.removeMainPanelTab(tab);
Clone fragments detected by clone detection tool
File path: /sql12/app/src/net/sourceforge/squirrel_sql/client/session/BaseSQLEntryPanel.java File path: /sql12/app/src/net/sourceforge/squirrel_sql/client/gui/session/SessionPanel.java
Method name: JMenuItem addToSQLEntryAreaMenu(Action) Method name: int removeMainTab(IMainPanelTab)
Number of AST nodes: 3 Number of AST nodes: 3
1
if (action == null)
1
if (tab == null)
2
		{
2
		{
3
			throw new IllegalArgumentException("Action == null");
3
			throw new IllegalArgumentException("Null IMainPanelTab passed");
4
		}
4
		}
5
		return _textPopupMenu.add(action);
5
		return _mainTabPane.removeMainPanelTab(tab);
Summary
Number of common nesting structure subtrees1
Number of refactorable cases0
Number of non-refactorable cases1
Time elapsed for finding largest common nesting structure subtrees (ms)0.1
Clones locationClones are in different classes
Number of node comparisons5
  1. {Non-refactorable}
    Mapping Summary
    Number of mapped statements2
    Number of unmapped statements in the first code fragment1
    Number of unmapped statements in the second code fragment1
    Time elapsed for statement mapping (ms)1.0
    Clone typeType 2
    Mapped Statements
    ID Statement ID Statement
    1
    if (action == null)
    1
    if (action == null)
    1
    if (tab == null)
    Differences
    Expression1Expression2Difference
    actiontabVARIABLE_NAME_MISMATCH
    javax.swing.Actionnet.sourceforge.squirrel_sql.client.session.mainpanel.IMainPanelTabVARIABLE_TYPE_MISMATCH
    Preondition Violations
    Type javax.swing.Action of variable action does not match with type net.sourceforge.squirrel_sql.client.session.mainpanel.IMainPanelTab of variable tab
    • Make classes javax.swing.Action and net.sourceforge.squirrel_sql.client.session.mainpanel.IMainPanelTab extend a common superclass
    1
    if (tab == null)
    2
    throw new IllegalArgumentException("Action == null");
    2
    throw new IllegalArgumentException("Action == null");
    2
    throw new IllegalArgumentException("Null IMainPanelTab passed");
    Differences
    Expression1Expression2Difference
    "Action == null""Null IMainPanelTab passed"LITERAL_VALUE_MISMATCH
    2
    throw new IllegalArgumentException("Null IMainPanelTab passed");
                                                                                                  
    3
    return _mainTabPane.removeMainPanelTab(tab);
    Preondition Violations
    Unmatched statement return _mainTabPane.removeMainPanelTab(tab); cannot be moved before the extracted code, because it has control dependencies from statements that will be extracted
    Unmatched return _mainTabPane.removeMainPanelTab(tab);
    3
    return _mainTabPane.removeMainPanelTab(tab);
    3
    return _textPopupMenu.add(action);
    3
    return _textPopupMenu.add(action);
    Preondition Violations
    Unmatched statement return _textPopupMenu.add(action); cannot be moved before the extracted code, because it has control dependencies from statements that will be extracted
    Unmatched return _textPopupMenu.add(action);
                                                                              
    Precondition Violations (6)
    Row Violation
    1Type javax.swing.Action of variable action does not match with type net.sourceforge.squirrel_sql.client.session.mainpanel.IMainPanelTab of variable tab
    2Unmatched statement return _mainTabPane.removeMainPanelTab(tab); cannot be moved before the extracted code, because it has control dependencies from statements that will be extracted
    3Unmatched return _mainTabPane.removeMainPanelTab(tab);
    4Unmatched statement return _textPopupMenu.add(action); cannot be moved before the extracted code, because it has control dependencies from statements that will be extracted
    5Unmatched return _textPopupMenu.add(action);
    6The refactoring of the clones is infeasible, because classes net.sourceforge.squirrel_sql.client.session.BaseSQLEntryPanel and net.sourceforge.squirrel_sql.client.gui.session.SessionPanel do not have a common superclass