txnamedpipes

Merge lp:~mattia.barbon/txnamedpipes/initial-import into lp:txnamedpipes

  1. initial-import
  2. Merge into trunk
Proposed by Mattia Barbon
Status: Merged
Merged at revision: 1
Proposed branch: lp:~mattia.barbon/txnamedpipes/initial-import
Merge into: lp:txnamedpipes
To merge this branch: bzr merge lp:~mattia.barbon/txnamedpipes/initial-import
Reviewer Review Type Date Requested Status
Manuel de la Peña (community) Approve
Review via email: mp+60925@code.launchpad.net

Description of the change

Initial implementation. The tests can be run using trial txnamedpipes.test.

To post a comment you must log in.
Revision history for this message
Manuel de la Peña (mandel) wrote :

All tests run and code is clean. Will approve and merge to trunk since it is the initial code.

Revision history for this message
Manuel de la Peña (mandel) wrote :

Ups, forgot the approve

review: Approve
lp:~mattia.barbon/txnamedpipes/initial-import updated
2. By Mattia Barbon

Force all writes to complete asynchronously even on Twisted 10.

The Twisted 10 IOCP reactor has some logic to handle I/O operations that
complete synchronously (rc = 1) and only handles asynchronously the ones
that fail with ERROR_IO_PENDING. Twisted 11 aways waits for the async
completion notification. In theory, the number of bytes written could
be obtained using GetOverlappedResult(), but it's probably not worth the
trouble of a different code path.

Subscribers

People subscribed via source and target branches