File path: /emf-2.4.1/src/org/eclipse/emf/ecore/xml/type/internal/DataValue.java | File path: /emf-2.4.1/src/org/eclipse/emf/ecore/xml/type/internal/DataValue.java | |||
Method name: boolean isValidServerBasedAuthority(String, int, String)
|
Method name: boolean isValidRegistryBasedAuthority(String)
|
|||
Number of AST nodes: 12 | Number of AST nodes: 13 | |||
1 | int index = 0;↵ | 1 | int index = 0;↵ | |
2 | int end = userinfo.length();↵ | 2 | int end = authority.length();↵ | |
3 | char testChar = '\0';↵ | 3 | char testChar;↵ | |
4 | ↵ | 4 | ↵ | |
5 | while (index < end) {↵ | 5 | while (index < end) {↵ | |
6 | testChar = userinfo.charAt(index);↵ | 6 | testChar = authority.charAt(index);↵ | |
7 | ↵ | 7 | ↵ | |
8 | // check for valid escape sequence↵ | |||
8 | if (testChar == '%') {↵ | 9 | if (testChar == '%') {↵ | |
9 | if (index+2 >= end ||↵ | 10 | if (index+2 >= end ||↵ | |
10 | !isHex(userinfo.charAt(index+1)) ||↵ | 11 | !isHex(authority.charAt(index+1)) ||↵ | |
11 | !isHex(userinfo.charAt(index+2))) {↵ | 12 | !isHex(authority.charAt(index+2))) {↵ | |
12 | return false;↵ | 13 | return false;↵ | |
13 | }↵ | 14 | ↵ | |
14 | ↵ | 15 | }↵ | |
15 | index += 2;↵ | 16 | index += 2;↵ | |
16 | }↵ | 17 | }↵ | |
17 | ↵ | 18 | // can check against path characters because the set↵ | |
19 | // is the same except for '/' which we've already excluded.↵ | |||
18 | else if (!isUserinfoCharacter(testChar)) {↵ | 20 | else if (!isPathCharacter(testChar)) {↵ | |
19 | return false;↵ | 21 | return false;↵ | |
20 | }↵ | 22 | }↵ | |
21 | ++index;↵ | 23 | ++index;↵ | |
22 | } | 24 | }↵ | |
25 |
| |||
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.7 |
Clones location | Clones are declared in the same class |
Number of node comparisons | 33 |
Number of mapped statements | 11 |
Number of unmapped statements in the first code fragment | 1 |
Number of unmapped statements in the second code fragment | 2 |
Time elapsed for statement mapping (ms) | 1.9 |
Clone type | Type 3 |
ID | Statement | ID | Statement | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
6 | int index = 0; | 1 | int index = 0; | ||||||||||||||
7 | int end = userinfo.length(); |
| 2 | int end = authority.length(); | |||||||||||||
| 3 | char testChar; | |||||||||||||||
8 | char testChar = '\0'; | | |||||||||||||||
9 | while (index < end) | 4 | while (index < end) | ||||||||||||||
10 | testChar = userinfo.charAt(index); |
| 5 | testChar = authority.charAt(index); | |||||||||||||
11 | if (testChar == '%') | 6 | if (testChar == '%') | ||||||||||||||
12 | if (index + 2 >= end || !isHex(userinfo.charAt(index + 1)) || !isHex(userinfo.charAt(index + 2))) |
| 7 | if (index + 2 >= end || !isHex(authority.charAt(index + 1)) || !isHex(authority.charAt(index + 2))) | |||||||||||||
13 | return false; | 8 | return false; | ||||||||||||||
14 | index += 2; | 9 | index += 2; | ||||||||||||||
15 | else if (!isUserinfoCharacter(testChar)) |
| 10 | else if (!isPathCharacter(testChar)) | |||||||||||||
16 | return false; | 11 | return false; | ||||||||||||||
17 | ++index; | 12 | ++index; | ||||||||||||||
|
| 13 | return true; |
Row | Violation |
---|---|
1 | Expression isUserinfoCharacter(testChar) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
2 | Expression isPathCharacter(testChar) cannot be parameterized, because it has dependencies to/from statements that will be extracted |
3 | Unmatched return true; |