----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20122/#review39818 -----------------------------------------------------------
Ship it! This looks good, assuming it tests cleanly then I say ship it! - Andrew Stitcher On April 8, 2014, 2:35 p.m., Cliff Jansen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20122/ > ----------------------------------------------------------- > > (Updated April 8, 2014, 2:35 p.m.) > > > Review request for qpid. > > > Bugs: QPID-5669 > https://issues.apache.org/jira/browse/QPID-5669 > > > Repository: qpid > > > Description > ------- > > This fix does the following: > > It moves all testing for self-delete to the IO thread by means of a reaper > callback. > > It frees the SChannel encrypt/decrypt resources in the queueForDeletion() > method which guarantees that the resources are freed at the last point they > can safely be done without the transport layer's own destructor cleaning up > associated resources out of order. > > If forces a single "winner" between close and disconnect (see QPID-5668). > > This results in a small performance penalty on connection teardown, while > allowing a tiny performance win in ordinary read processing. > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/qpid/trunk/qpid/cpp/src/qpid/sys/windows/AsynchIO.cpp > 1585645 > > http://svn.apache.org/repos/asf/qpid/trunk/qpid/cpp/src/qpid/sys/windows/SslAsynchIO.h > 1585645 > > http://svn.apache.org/repos/asf/qpid/trunk/qpid/cpp/src/qpid/sys/windows/SslAsynchIO.cpp > 1585645 > > Diff: https://reviews.apache.org/r/20122/diff/ > > > Testing > ------- > > Windows 7, 32/64 bit clients, amqp1.0 and amqp0-10, C++ and C#, wifi/wired > connections. > > > Thanks, > > Cliff Jansen > >