DigestInputStream dis = new DigestInputStream(fis,
messageDigest);
while (dis.read(buf, 0, readBufferSize) != -1) {
// Empty statement
}
dis.close();
fis.close();
fis = null;
byte[] fileDigest = messageDigest.digest ();
DigestInputStream dis = new DigestInputStream(fis,
messageDigest);
while (dis.read(buf, 0, readBufferSize) != -1) {
// do nothing
}
dis.close();
fis.close();
fis = null;
byte[] fileDigest = messageDigest.digest();
Clone fragments detected by clone detection tool
File path: /apache-ant-1.7.0/src/org/apache/tools/ant/taskdefs/Checksum.java
|
|
File path: /apache-ant-1.7.0/src/org/apache/tools/ant/types/selectors/modifiedselector/DigestAlgorithm.java
|
Method name: boolean generateChecksums()
|
|
Method name: String getValue(File)
|
Number of AST nodes: 6
|
|
Number of AST nodes: 6
|
|
1 | DigestInputStream dis = new DigestInputStream(fis,↵ | | 1 | DigestInputStream dis = new DigestInputStream(fis,↵
|
2 | messageDigest);↵ | | 2 | messageDigest);↵
|
3 | while (dis.read(buf, 0, readBufferSize) != -1) {↵ | | 3 | while (dis.read(buf, 0, readBufferSize) != -1) {↵
|
4 | // Empty statement↵ | | 4 | // do nothing↵
|
5 | }↵ | | 5 | }↵
|
6 | dis.close();↵ | | 6 | dis.close();↵
|
7 | fis.close();↵ | | 7 | fis.close();↵
|
8 | fis = null;↵ | | 8 | fis = null;↵
|
9 | byte[] fileDigest = messageDigest.digest (); | | 9 | byte[] fileDigest = messageDigest.digest();
|
See real code fragment |
|
See real code fragment |
Summary
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) | 1.1 |
Clones location | Clones are in different classes |
Number of node comparisons | 20 |
-
{Non-refactorable}
Mapping Summary
Number of mapped statements | 5 |
Number of unmapped statements in the first code fragment | 1 |
Number of unmapped statements in the second code fragment | 1 |
Time elapsed for statement mapping (ms) | 4.8 |
Clone type | Type 2 |
Mapped Statements
ID |
Statement |
|
ID |
Statement |
12 | DigestInputStream dis = new DigestInputStream(fis, messageDigest); | | 11 | DigestInputStream dis = new DigestInputStream(fis, messageDigest); |
13 | while (dis.read(buf, 0, readBufferSize) != -1) | | 12 | while (dis.read(buf, 0, readBufferSize) != -1) |
14 | dis.close(); | | 13 | dis.close(); |
15 | fis.close(); | | 14 | fis.close(); |
16 | fis = null; | | 15 | fis = null; |
| | | 16 | byte[] fileDigest = messageDigest.digest(); |
17 | byte[] fileDigest = messageDigest.digest(); | | | |
Precondition Violations (4)
Row |
Violation |
1 | Unmatched statement byte[] fileDigest=messageDigest.digest(); cannot be moved before the extracted code, because it has control dependencies from statements that will be extracted |
2 | Unmatched statement byte[] fileDigest=messageDigest.digest(); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
3 | Clone fragment #1 returns variables fis, fileDigest , while Clone fragment #2 returns variables fileDigest |
4 | The refactoring of the clones is infeasible, because classes org.apache.tools.ant.taskdefs.Checksum and org.apache.tools.ant.types.selectors.modifiedselector.DigestAlgorithm do not have a common superclass |