File path: /hibernate-distribution-3.3.2.GA/project/core/src/main/java/org/hibernate/cache/ReadWriteCache.java | File path: /hibernate-distribution-3.3.2.GA/project/core/src/main/java/org/hibernate/cache/ReadWriteCache.java | |||
Method name: boolean afterUpdate(Object, Object, Object, SoftLock)
|
Method name: boolean put(Object, Object, long, Object, Comparator, boolean)
|
|||
Number of AST nodes: 4 | Number of AST nodes: 4 | |||
1 | cache.update( key, new Item( value, version, cache.nextTimestamp() ) );↵ | 1 | cache.put( key, new Item( value, version, cache.nextTimestamp() ) );↵ | |
2 | if ( log.isTraceEnabled() ) log.trace("Updated: " + key);↵ | 2 | if ( log.isTraceEnabled() ) log.trace("Cached: " + key);↵ | |
3 | return true; | 3 | return true; | |
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.1 |
Clones location | Clones are declared in the same class |
Number of node comparisons | 8 |
Number of mapped statements | 4 |
Number of unmapped statements in the first code fragment | 0 |
Number of unmapped statements in the second code fragment | 0 |
Time elapsed for statement mapping (ms) | 1.7 |
Clone type | Type 2 |
ID | Statement | ID | Statement | |||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
11 | cache.update(key, new Item(value, version, cache.nextTimestamp())); |
| 8 | cache.put(key, new Item(value, version, cache.nextTimestamp())); | ||||||||||||
12 | if (log.isTraceEnabled()) | 9 | if (log.isTraceEnabled()) | |||||||||||||
13 | log.trace("Updated: " + key); |
| 10 | log.trace("Cached: " + key); | ||||||||||||
14 | return true; |
| 11 | return true; |
Row | Violation |
---|---|
1 | Expression cache.update(key,new Item(value,version,cache.nextTimestamp())) is a void method call, and thus it cannot be parameterized |
2 | Expression cache.put(key,new Item(value,version,cache.nextTimestamp())) is a void method call, and thus it cannot be parameterized |
3 | Conditional return true; |
4 | Conditional return true; |