proton-j/src/main/java/org/apache/qpid/proton/engine/impl/TransportImpl.java (13 lines): - line 282: // TODO - check if already bound - line 1211: // TODO - check channel < max_channel - line 1215: // TODO - fail due to begin on begun session - line 1229: // TODO handle failure rather than just throwing a nicer NPE - line 1255: // TODO - fail due to attach on non-begun session - line 1285: // TODO - fail - attempt attach on a handle which is in use - line 1338: // TODO - fail due to attach on non-begun session - line 1350: // TODO - check channel < max_channel - line 1358: // TODO - fail due to begin on begun session - line 1368: // TODO - fail due to attach on non-begun session - line 1382: // TODO - fail due to attach on non-begun session - line 1407: // TODO - fail - attempt attach on a handle which is in use - line 1418: // TODO - fail due to attach on non-begun session proton-j/src/main/java/org/apache/qpid/proton/engine/Link.java (4 lines): - line 41: * TODO describe the application's responsibility to honour settlement. - line 57: * TODO to clarify - this adds the delivery to the connection list. It is not yet - line 97: * TODO document the meaning of the return value more fully. Currently Senderimpl only returns false if there is no current delivery - line 203: * TODO should this be part of the interface? proton-j/src/main/java/org/apache/qpid/proton/engine/Sender.java (3 lines): - line 37: //TODO is this absolute or cumulative? - line 50: * TODO Proton-j current copies all the bytes it has been given so the return value will always be - line 97: * TODO fully state the rules regarding when you have to call this method, what happens if you don't call it proton-j/src/main/java/org/apache/qpid/proton/engine/Delivery.java (3 lines): - line 62: * TODO proton-j and proton-c return the local and remote statuses respectively. Resolve this ambiguity. - line 71: * TODO When does an application call this method? Do we really need this? - line 98: * TODO what is the main intended use case for calling this method? proton-j/src/main/java/org/apache/qpid/proton/engine/impl/LinkImpl.java (2 lines): - line 236: // TODO - should be an error if local state is ACTIVE - line 249: // TODO - should be an error if local state is ACTIVE proton-j/src/main/java/org/apache/qpid/proton/engine/Receiver.java (2 lines): - line 51: * TODO might the flags other than IO_WORK in DeliveryImpl also prevent the work list being pruned? - line 89: * TODO document what this method conceptually does and when you should use it. proton-j/src/main/java/org/apache/qpid/proton/message/Message.java (2 lines): - line 167: * TODO describe what happens if the data does not represent a complete message. - line 188: * TODO describe what happens if length is smaller than the encoded form, Currently proton-j/src/main/java/org/apache/qpid/proton/engine/impl/TransportSession.java (2 lines): - line 45: private final UnsignedInteger _handleMax = UnsignedInteger.valueOf(HANDLE_MAX); //TODO: should this be configurable? - line 463: //TODO - Implement. proton-j/src/main/java/org/apache/qpid/proton/engine/Connection.java (2 lines): - line 53: * TODO does the Connection's channel-max property limit how many sessions can be created, - line 64: * TODO what ordering guarantees on the returned "linked list" are provided? proton-j/src/main/java/org/apache/qpid/proton/codec/transport/FastPathDispositionType.java (1 line): - line 105: // TODO - Technically invalid however old decoder also allowed this. proton-j/src/main/java/org/apache/qpid/proton/engine/impl/ssl/SslEngineFacadeFactory.java (1 line): - line 73: * TODO allow the protocol name to be overridden somehow proton-j/src/main/java/org/apache/qpid/proton/engine/impl/ConnectionImpl.java (1 line): - line 228: // TODO - store state proton-j/src/main/java/org/apache/qpid/proton/codec/transport/FastPathFlowType.java (1 line): - line 93: // TODO - Technically invalid however old decoder also allowed this. proton-j/src/main/java/org/apache/qpid/proton/engine/impl/DeliveryImpl.java (1 line): - line 245: return (_complete && consumed == 0) ? Transport.END_OF_STREAM : consumed; //TODO - Implement proton-j/src/main/java/org/apache/qpid/proton/engine/impl/HandshakeSniffingTransportWrapper.java (1 line): - line 111: // TODO what if the selected transport has insufficient capacity?? Maybe use pour, and then try to finish pouring next time round. proton-j/src/main/java/org/apache/qpid/proton/engine/impl/SaslImpl.java (1 line): - line 600: * TODO rationalise this method with respect to the other similar checks of _role/_initReceived etc proton-j/src/main/java/org/apache/qpid/proton/engine/impl/ssl/SimpleSslTransportWrapper.java (1 line): - line 41: * TODO close the SSLEngine when told to, and modify {@link #wrapOutput()} and {@link #unwrapInput()} proton-j/src/main/java/org/apache/qpid/proton/engine/impl/EndpointImpl.java (1 line): - line 130: // TODO - check state change legal proton-j/src/main/java/org/apache/qpid/proton/amqp/messaging/Accepted.java (1 line): - line 37: * TODO should this (and other DeliveryStates) have a private constructor?? proton-j/src/main/java/org/apache/qpid/proton/amqp/messaging/Terminus.java (1 line): - line 50: // TODO: Do we need to copy or can we make a simple reference? proton-j/src/main/java/org/apache/qpid/proton/engine/impl/ssl/SslHandshakeSniffingTransportWrapper.java (1 line): - line 90: // TODO perhaps the sniffer should save up the bytes from each proton-j/src/main/java/org/apache/qpid/proton/codec/EncoderImpl.java (1 line): - line 477: // TODO - java character may be half of a pair, should probably throw exception then proton-j/src/main/java/org/apache/qpid/proton/codec/transport/FastPathTransferType.java (1 line): - line 74: // TODO - Technically invalid however old decoder also allowed this. proton-j/src/main/java/org/apache/qpid/proton/amqp/transport/DeliveryState.java (1 line): - line 28: * TODO clarify the concept of ownership? how is link recovery involved? proton-j/src/main/java/org/apache/qpid/proton/engine/impl/ReceiverImpl.java (1 line): - line 143: return false; //TODO - Implement