File path: /apache-ant-1.7.0/src/org/apache/tools/ant/types/selectors/SelectorUtils.java | File path: /apache-ant-1.7.0/src/org/apache/tools/ant/types/selectors/SelectorUtils.java | |||
Method name: boolean matchPath(String, String, boolean)
|
Method name: boolean matchPath(String, String, boolean)
|
|||
Number of AST nodes: 9 | Number of AST nodes: 9 | |||
1 | String patDir = patDirs[patIdxStart];↵ | 1 | String patDir = patDirs[patIdxEnd];↵ | |
2 | if (patDir.equals("**")) {↵ | 2 | if (patDir.equals("**")) {↵ | |
3 | break;↵ | 3 | break;↵ | |
4 | }↵ | 4 | }↵ | |
5 | if (!match(patDir, strDirs[strIdxStart], isCaseSensitive)) {↵ | 5 | if (!match(patDir, strDirs[strIdxEnd], isCaseSensitive)) {↵ | |
6 | patDirs = null;↵ | 6 | patDirs = null;↵ | |
7 | strDirs = null;↵ | 7 | strDirs = null;↵ | |
8 | return false;↵ | 8 | return false;↵ | |
9 | }↵ | 9 | }↵ | |
10 | patIdxStart++;↵ | 10 | patIdxEnd--;↵ | |
11 | strIdxStart++; | 11 | strIdxEnd--; | |
See real code fragment | See real code fragment |
Number of common nesting structure subtrees | 1 |
Number of refactorable cases | 0 |
Number of non-refactorable cases | 1 |
Time elapsed for finding largest common nesting structure subtrees (ms) | 0.4 |
Clones location | Clones are in the same method |
Number of node comparisons | 41 |
Number of mapped statements | 7 |
Number of unmapped statements in the first code fragment | 2 |
Number of unmapped statements in the second code fragment | 2 |
Time elapsed for statement mapping (ms) | 1.8 |
Clone type | Type 2 |
ID | Statement | ID | Statement | |||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
8 | String patDir = patDirs[patIdxStart]; |
| 29 | String patDir = patDirs[patIdxEnd]; | ||||||||||
9 | if (patDir.equals("**")) | 30 | if (patDir.equals("**")) | |||||||||||
10 | break; |
| 31 | break; | ||||||||||
11 | if (!match(patDir, strDirs[strIdxStart], isCaseSensitive)) |
| 32 | if (!match(patDir, strDirs[strIdxEnd], isCaseSensitive)) | ||||||||||
12 | patDirs = null; | 33 | patDirs = null; | |||||||||||
13 | strDirs = null; | 34 | strDirs = null; | |||||||||||
14 | return false; |
| 35 | return false; | ||||||||||
15 | patIdxStart++; |
| | |||||||||||
16 | strIdxStart++; |
| | |||||||||||
|
| 36 | patIdxEnd--; | |||||||||||
|
| 37 | strIdxEnd--; |
Row | Violation |
---|---|
1 | Statement break; without innermost loop |
2 | Statement break; without innermost loop |
3 | Unmatched statement patIdxStart++; cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
4 | Unmatched statement strIdxStart++; cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
5 | Unmatched statement patIdxEnd--; cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
6 | Unmatched statement strIdxEnd--; cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
7 | Clone fragment #1 returns variables patDirs, strDirs , while Clone fragment #2 returns variables patDirs, strDirs |
8 | Conditional return false; |
9 | Conditional return false; |