File path: /jruby-1.4.0/src/org/jruby/compiler/ASTCompiler.java | File path: /jruby-1.4.0/src/org/jruby/compiler/ASTCompiler.java | |||
Method name: void compilePreExe(Node, BodyCompiler, boolean)
|
Method name: void compileBreak(Node, BodyCompiler, boolean)
|
|||
Number of AST nodes: 5 | Number of AST nodes: 5 | |||
1 | final PreExeNode preExeNode = (PreExeNode) node;↵ | 1 | final BreakNode breakNode = (BreakNode) node;↵ | |
2 | // create the closure class and instantiate it↵ | 2 | ↵ | |
3 | final CompilerCallback closureBody = new CompilerCallback() {↵ | 3 | CompilerCallback valueCallback = new CompilerCallback() {↵ | |
4 | public void call(BodyCompiler context) {↵ | 4 | public void call(BodyCompiler context) {↵ | |
5 | if (preExeNode.getBodyNode() != null) {↵ | 5 | if (breakNode.getValueNode() != null) {↵ | |
6 | compile(preExeNode.getBodyNode(), context,true);↵ | 6 | compile(breakNode.getValueNode(), context, true);↵ | |
7 | } else {↵ | 7 | } else {↵ | |
8 | context.loadNil();↵ | 8 | context.loadNil();↵ | |
9 | }↵ | 9 | }↵ | |
10 | }↵ | 10 | }↵ | |
11 | };↵ | 11 | };↵ | |
12 | context.runBeginBlock(preExeNode.getScope(), closureBody);↵ | 12 | context.issueBreakEvent(valueCallback);↵ | |
13 | // TODO: don't require pop↵ | 13 | // TODO: don't require pop↵ | |
14 | if (!expr) context.consumeCurrentValue(); | 14 |
| |
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 | 13 |
Number of mapped statements | 4 |
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.3 |
Clone type | Type 2 |
ID | Statement | ID | Statement | ||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
1 | final PreExeNode preExeNode = (PreExeNode)node; |
| 1 | final BreakNode breakNode = (BreakNode)node; | |||||||||||||||||||
|
| 2 | CompilerCallback valueCallback = new CompilerCallback() {...}; | ||||||||||||||||||||
2 | final CompilerCallback closureBody = new CompilerCallback() {...}; |
| | ||||||||||||||||||||
3 | context.runBeginBlock(preExeNode.getScope(), closureBody); |
| 3 | context.issueBreakEvent(valueCallback); | |||||||||||||||||||
4 | if (!expr) | 4 | if (!expr) | ||||||||||||||||||||
5 | context.consumeCurrentValue(); | 5 | context.consumeCurrentValue(); |
Row | Violation |
---|---|
1 | Unmatched statement CompilerCallback valueCallback=new CompilerCallback(){ public void call( BodyCompiler context){ if (breakNode.getValueNode() != null) { compile(breakNode.getValueNode(),context,true); } else { context.loadNil(); } } } ; cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
2 | Unmatched statement final CompilerCallback closureBody=new CompilerCallback(){ public void call( BodyCompiler context){ if (preExeNode.getBodyNode() != null) { compile(preExeNode.getBodyNode(),context,true); } else { context.loadNil(); } } } ; cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
3 | Expression context.runBeginBlock(preExeNode.getScope(),closureBody) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
4 | Expression context.runBeginBlock(preExeNode.getScope(),closureBody) is a void method call, and thus it cannot be parameterized |
5 | Expression context.issueBreakEvent(valueCallback) is a void method call, and thus it cannot be parameterized |
6 | Expression context.runBeginBlock(preExeNode.getScope(),closureBody) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
7 | Expression context.runBeginBlock(preExeNode.getScope(),closureBody) is a void method call, and thus it cannot be parameterized |
8 | Expression context.issueBreakEvent(valueCallback) is a void method call, and thus it cannot be parameterized |
9 | Clone fragment #1 returns variable preExeNode with type org.jruby.ast.PreExeNode , while Clone fragment #2 returns variable breakNode with type org.jruby.ast.BreakNode |