-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9450/
-----------------------------------------------------------
(Updated Feb. 15, 2013, 2:55 p.m.)
Review request for qpid, Cliff Jansen and Rafael Schloming.
Changes
-------
Modified new interface.
Description
-------
Proposed patch. It requires the work already done in PROTON-225, so this patch
must be layered on top of that.
1) messenger checks for buffered output bytes in transport
2) removes the buffering done in the driver (no longer needed due to PROTON-225)
3) transport reports the number of bytes it has buffered (along with any bytes
buffered by the SASL and SSL layers).
Cliff - I've synced the changes from the posix/driver.c file into
windows/driver.c. I haven't verified the windows/driver.c changes yet, but
will do so shortly.
This addresses bug proton-222.
https://issues.apache.org/jira/browse/proton-222
Diffs (updated)
-----
/proton/branches/kgiusti-proton-225/proton-c/include/proton/engine.h 1446334
/proton/branches/kgiusti-proton-225/proton-c/src/engine/engine-internal.h
1446334
/proton/branches/kgiusti-proton-225/proton-c/src/engine/engine.c 1446334
/proton/branches/kgiusti-proton-225/proton-c/src/messenger.c 1446334
/proton/branches/kgiusti-proton-225/proton-c/src/posix/driver.c 1446334
/proton/branches/kgiusti-proton-225/proton-c/src/ssl/openssl.c 1446334
/proton/branches/kgiusti-proton-225/proton-c/src/windows/driver.c 1446334
Diff: https://reviews.apache.org/r/9450/diff/
Testing
-------
Works against my reproducer.
Thanks,
Kenneth Giusti
