-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9450/
-----------------------------------------------------------
Review request for qpid, Cliff Jansen and Rafael Schloming.
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
-----
/proton/branches/kgiusti-proton-225/proton-c/include/proton/engine.h 1445894
/proton/branches/kgiusti-proton-225/proton-c/src/engine/engine-internal.h
1445894
/proton/branches/kgiusti-proton-225/proton-c/src/engine/engine.c 1445894
/proton/branches/kgiusti-proton-225/proton-c/src/messenger.c 1445894
/proton/branches/kgiusti-proton-225/proton-c/src/posix/driver.c 1445894
/proton/branches/kgiusti-proton-225/proton-c/src/ssl/openssl.c 1445894
/proton/branches/kgiusti-proton-225/proton-c/src/windows/driver.c 1445894
Diff: https://reviews.apache.org/r/9450/diff/
Testing
-------
Works against my reproducer.
Thanks,
Kenneth Giusti