Author: rajith
Date: Wed Jul 25 20:51:09 2012
New Revision: 1365737
URL: http://svn.apache.org/viewvc?rev=1365737&view=rev
Log:
NO-JIRA Removed the duplicate _next/_prev fields and their
accessors/modifiers as they seem extra bcos we have
_transportNext/_transportPrev.
Modified:
qpid/proton/branches/rajith_sandbox/proton-j/src/org/apache/qpid/proton/engine/impl/ConnectionImpl.java
qpid/proton/branches/rajith_sandbox/proton-j/src/org/apache/qpid/proton/engine/impl/EndpointImpl.java
qpid/proton/branches/rajith_sandbox/proton-j/src/org/apache/qpid/proton/engine/impl/TransportImpl.java
Modified:
qpid/proton/branches/rajith_sandbox/proton-j/src/org/apache/qpid/proton/engine/impl/ConnectionImpl.java
URL:
http://svn.apache.org/viewvc/qpid/proton/branches/rajith_sandbox/proton-j/src/org/apache/qpid/proton/engine/impl/ConnectionImpl.java?rev=1365737&r1=1365736&r2=1365737&view=diff
==============================================================================
---
qpid/proton/branches/rajith_sandbox/proton-j/src/org/apache/qpid/proton/engine/impl/ConnectionImpl.java
(original)
+++
qpid/proton/branches/rajith_sandbox/proton-j/src/org/apache/qpid/proton/engine/impl/ConnectionImpl.java
Wed Jul 25 20:51:09 2012
@@ -274,11 +274,6 @@ public class ConnectionImpl extends Endp
return _maxChannels;
}
- public EndpointImpl next()
- {
- return getNext();
- }
-
public String getLocalContainerId()
{
return _localContainerId;
Modified:
qpid/proton/branches/rajith_sandbox/proton-j/src/org/apache/qpid/proton/engine/impl/EndpointImpl.java
URL:
http://svn.apache.org/viewvc/qpid/proton/branches/rajith_sandbox/proton-j/src/org/apache/qpid/proton/engine/impl/EndpointImpl.java?rev=1365737&r1=1365736&r2=1365737&view=diff
==============================================================================
---
qpid/proton/branches/rajith_sandbox/proton-j/src/org/apache/qpid/proton/engine/impl/EndpointImpl.java
(original)
+++
qpid/proton/branches/rajith_sandbox/proton-j/src/org/apache/qpid/proton/engine/impl/EndpointImpl.java
Wed Jul 25 20:51:09 2012
@@ -32,8 +32,6 @@ public abstract class EndpointImpl imple
private EndpointError _localError;
private EndpointError _remoteError;
private boolean _modified;
- private EndpointImpl _next;
- private EndpointImpl _prev;
private EndpointImpl _transportNext;
private EndpointImpl _transportPrev;
@@ -141,26 +139,15 @@ public abstract class EndpointImpl imple
return _transportPrev;
}
- void setNext(EndpointImpl next)
- {
- _next = next;
- }
-
- void setPrev(EndpointImpl prev)
- {
- _prev = prev;
- }
-
-
public void free()
{
- if(_next != null)
+ if(_transportNext != null)
{
- _next._prev = _prev;
+ _transportNext.setTransportPrev(_transportPrev);
}
- if(_prev != null)
+ if(_transportPrev != null)
{
- _prev._next = _next;
+ _transportPrev.setTransportNext(_transportNext);
}
}
@@ -173,17 +160,4 @@ public abstract class EndpointImpl imple
{
_transportPrev = transportPrevious;
}
-
- EndpointImpl getPrev()
- {
- return _prev;
- }
-
- EndpointImpl getNext()
- {
- return _next;
- }
-
-
-
}
Modified:
qpid/proton/branches/rajith_sandbox/proton-j/src/org/apache/qpid/proton/engine/impl/TransportImpl.java
URL:
http://svn.apache.org/viewvc/qpid/proton/branches/rajith_sandbox/proton-j/src/org/apache/qpid/proton/engine/impl/TransportImpl.java?rev=1365737&r1=1365736&r2=1365737&view=diff
==============================================================================
---
qpid/proton/branches/rajith_sandbox/proton-j/src/org/apache/qpid/proton/engine/impl/TransportImpl.java
(original)
+++
qpid/proton/branches/rajith_sandbox/proton-j/src/org/apache/qpid/proton/engine/impl/TransportImpl.java
Wed Jul 25 20:51:09 2012
@@ -209,7 +209,7 @@ public class TransportImpl extends Endpo
}
}
- endpoint = endpoint.getNext();
+ endpoint = endpoint.transportNext();
}
return written;
}
@@ -251,7 +251,7 @@ public class TransportImpl extends Endpo
}
- endpoint = endpoint.getNext();
+ endpoint = endpoint.transportNext();
}
return written; //TODO - Implement
}
@@ -474,7 +474,7 @@ public class TransportImpl extends Endpo
}
}
}
- endpoint = endpoint.getNext();
+ endpoint = endpoint.transportNext();
}
return written;
}
@@ -676,7 +676,7 @@ public class TransportImpl extends Endpo
}
}
- endpoint = endpoint.getNext();
+ endpoint = endpoint.transportNext();
}
return written;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
