You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ws.apache.org by ve...@apache.org on 2015/06/18 22:08:22 UTC
svn commit: r1686313 - in
/webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom:
Attribute.java NamespaceDeclaration.java OMAttributeImpl.java
Author: veithen
Date: Thu Jun 18 20:08:22 2015
New Revision: 1686313
URL: http://svn.apache.org/r1686313
Log:
Eliminate unused value attribute in LLOM's NamespaceDeclaration.
Modified:
webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/Attribute.java
webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/NamespaceDeclaration.java
webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMAttributeImpl.java
Modified: webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/Attribute.java
URL: http://svn.apache.org/viewvc/webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/Attribute.java?rev=1686313&r1=1686312&r2=1686313&view=diff
==============================================================================
--- webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/Attribute.java (original)
+++ webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/Attribute.java Thu Jun 18 20:08:22 2015
@@ -23,7 +23,6 @@ import org.apache.axiom.om.OMFactory;
public abstract class Attribute extends OMInformationItemImpl implements CoreAttribute {
private OMFactory factory;
- private String value;
public Attribute(OMFactory factory) {
this.factory = factory;
@@ -32,12 +31,4 @@ public abstract class Attribute extends
public final OMFactory getOMFactory() {
return factory;
}
-
- public final String coreGetValue() {
- return value;
- }
-
- public final void coreSetValue(String value) {
- this.value = value;
- }
}
Modified: webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/NamespaceDeclaration.java
URL: http://svn.apache.org/viewvc/webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/NamespaceDeclaration.java?rev=1686313&r1=1686312&r2=1686313&view=diff
==============================================================================
--- webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/NamespaceDeclaration.java (original)
+++ webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/NamespaceDeclaration.java Thu Jun 18 20:08:22 2015
@@ -28,6 +28,16 @@ public final class NamespaceDeclaration
super(factory);
}
+ public String coreGetValue() {
+ // Method not used in LLOM
+ throw new UnsupportedOperationException();
+ }
+
+ public void coreSetValue(String value) {
+ // Method not used in LLOM
+ throw new UnsupportedOperationException();
+ }
+
public OMInformationItem clone(OMCloneOptions options) {
NamespaceDeclaration decl = new NamespaceDeclaration(getOMFactory());
decl.setDeclaredNamespace(getDeclaredNamespace());
Modified: webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMAttributeImpl.java
URL: http://svn.apache.org/viewvc/webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMAttributeImpl.java?rev=1686313&r1=1686312&r2=1686313&view=diff
==============================================================================
--- webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMAttributeImpl.java (original)
+++ webservices/axiom/branches/attrs-aspects/implementations/axiom-impl/src/main/java/org/apache/axiom/om/impl/llom/OMAttributeImpl.java Thu Jun 18 20:08:22 2015
@@ -28,6 +28,8 @@ import org.apache.axiom.om.impl.common.A
/** Class OMAttributeImpl */
public class OMAttributeImpl extends Attribute implements AxiomAttribute {
+ private String value;
+
/**
* Constructor OMAttributeImpl.
*
@@ -58,6 +60,14 @@ public class OMAttributeImpl extends Att
internalSetNamespace(ns);
coreSetType(OMConstants.XMLATTRTYPE_CDATA);
}
+
+ public final String coreGetValue() {
+ return value;
+ }
+
+ public final void coreSetValue(String value) {
+ this.value = value;
+ }
public OMInformationItem clone(OMCloneOptions options) {
return new OMAttributeImpl(getLocalName(), getNamespace(), getAttributeValue(), getOMFactory());