File path: /emf-2.4.1/src/org/eclipse/emf/ecore/plugin/DynamicPackageRegistryReader.java | File path: /emf-2.4.1/src/org/eclipse/emf/ecore/plugin/ExtensionParserRegistryReader.java | |||
Method name: boolean readElement(IConfigurationElement, boolean)
|
Method name: boolean readElement(IConfigurationElement, boolean)
|
|||
Number of AST nodes: 13 | Number of AST nodes: 13 | |||
1 | String packageURI = element.getAttribute(ATT_URI);↵ | 1 | String type = element.getAttribute(ATT_TYPE);↵ | |
2 | if (packageURI == null)↵ | 2 | if (type == null)↵ | |
3 | {↵ | 3 | {↵ | |
4 | logMissingAttribute(element, ATT_URI);↵ | 4 | logMissingAttribute(element, ATT_TYPE);↵ | |
5 | }↵ | 5 | }↵ | |
6 | else if (element.getAttribute(ATT_LOCATION) == null)↵ | 6 | else if (element.getAttribute(ATT_CLASS) == null)↵ | |
7 | {↵ | 7 | {↵ | |
8 | logMissingAttribute(element, ATT_LOCATION);↵ | 8 | logMissingAttribute(element, ATT_CLASS);↵ | |
9 | }↵ | 9 | }↵ | |
10 | else if (add)↵ | 10 | else if (add)↵ | |
11 | {↵ | 11 | {↵ | |
12 | Object previous = EPackage.Registry.INSTANCE.put(packageURI, new EPackageDescriptor.Dynamic(element, ATT_LOCATION));↵ | 12 | Object previous = Resource.Factory.Registry.INSTANCE.getExtensionToFactoryMap().put(type, new ResourceFactoryDescriptor(element, ATT_CLASS));↵ | |
13 | if (previous instanceof PluginClassDescriptor)↵ | 13 | if (previous instanceof ResourceFactoryDescriptor)↵ | |
14 | {↵ | 14 | {↵ | |
15 | PluginClassDescriptor descriptor = (PluginClassDescriptor)previous;↵ | 15 | ResourceFactoryDescriptor descriptor = (ResourceFactoryDescriptor)previous;↵ | |
16 | EcorePlugin.INSTANCE.log↵ | 16 | EcorePlugin.INSTANCE.log↵ | |
17 | ("Both '" + descriptor.element.getContributor().getName() + "' and '" + element.getContributor().getName() + "' register a package for '" + packageURI + "'");↵ | 17 | ("Both '" + descriptor.element.getContributor().getName() + "' and '" + element.getContributor().getName() + "' register an extension parser for '" + type + "'");↵ | |
18 | }↵ | 18 | }↵ | |
19 | ↵ | |||
20 | return true;↵ | 19 | return true;↵ | |
21 | }↵ | 20 | }↵ | |
22 | else↵ | 21 | else↵ | |
23 | {↵ | 22 | {↵ | |
24 | EPackage.Registry.INSTANCE.remove(packageURI);↵ | 23 | Resource.Factory.Registry.INSTANCE.getExtensionToFactoryMap().remove(type);↵ | |
25 | return true;↵ | 24 | return true;↵ | |
26 | } | 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) | 1.0 |
Clones location | Clones are in different classes having the same super class |
Number of node comparisons | 49 |
Number of mapped statements | 11 |
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) | 3.5 |
Clone type | Type 2 |
ID | Statement | ID | Statement | |||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
2 | String packageURI = element.getAttribute(ATT_URI); |
| 2 | String type = element.getAttribute(ATT_TYPE); | ||||||||||||||||
3 | if (packageURI == null) |
| 3 | if (type == null) | ||||||||||||||||
4 | logMissingAttribute(element, ATT_URI); |
| 4 | logMissingAttribute(element, ATT_TYPE); | ||||||||||||||||
5 | else if (element.getAttribute(ATT_LOCATION) == null) |
| 5 | else if (element.getAttribute(ATT_CLASS) == null) | ||||||||||||||||
6 | logMissingAttribute(element, ATT_LOCATION); |
| 6 | logMissingAttribute(element, ATT_CLASS); | ||||||||||||||||
7 | else if (add) | 7 | else if (add) | |||||||||||||||||
|
| 8 | Object previous = Resource.Factory.Registry.INSTANCE.getExtensionToFactoryMap().put(type, new ResourceFactoryDescriptor(element, ATT_CLASS)); | |||||||||||||||||
8 | Object previous = EPackage.Registry.INSTANCE.put(packageURI, new EPackageDescriptor.Dynamic(element, ATT_LOCATION)); |
| | |||||||||||||||||
9 | if (previous instanceof PluginClassDescriptor) |
| 9 | if (previous instanceof ResourceFactoryDescriptor) | ||||||||||||||||
10 | PluginClassDescriptor descriptor = (PluginClassDescriptor)previous; |
| 10 | ResourceFactoryDescriptor descriptor = (ResourceFactoryDescriptor)previous; | ||||||||||||||||
11 | EcorePlugin.INSTANCE.log("Both '" + descriptor.element.getContributor().getName() + "' and '" + element.getContributor().getName() + "' register a package for '" + packageURI + "'"); |
| 11 | EcorePlugin.INSTANCE.log("Both '" + descriptor.element.getContributor().getName() + "' and '" + element.getContributor().getName() + "' register an extension parser for '" + type + "'"); | ||||||||||||||||
12 | return true; | 12 | return true; | |||||||||||||||||
else | else | |||||||||||||||||||
| 13 | Resource.Factory.Registry.INSTANCE.getExtensionToFactoryMap().remove(type); | ||||||||||||||||||
13 | EPackage.Registry.INSTANCE.remove(packageURI); | | ||||||||||||||||||
14 | return true; | 14 | return true; |
Row | Violation |
---|---|
1 | Unmatched statement Object previous=Resource.Factory.Registry.INSTANCE.getExtensionToFactoryMap().put(type,new ResourceFactoryDescriptor(element,ATT_CLASS)); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |
2 | Unmatched statement Object previous=EPackage.Registry.INSTANCE.put(packageURI,new EPackageDescriptor.Dynamic(element,ATT_LOCATION)); cannot be moved before or after the extracted code, because it has dependencies to/from statements that will be extracted |