astitcher commented on a change in pull request #180: PROTON-2030 Use
CLOCK_MONOTONIC in proactors for pn_transport_tick
URL: https://github.com/apache/qpid-proton/pull/180#discussion_r340205929
##########
File path: c/src/proactor/win_iocp.c
##########
@@ -2145,8 +2132,8 @@ static void pconnection_tick(pconnection_t *pc) {
if(!stop_timer(pc->context.proactor->timer_queue, &pc->tick_timer)) {
// TODO: handle error
}
- uint64_t now = pn_i_now2();
- uint64_t next = pn_transport_tick(t, now);
+ int64_t now = pn_proactor_now_64();
+ int64_t next = pn_transport_tick(t, now);
if (next) {
if (!start_timer(pc->context.proactor->timer_queue, &pc->tick_timer,
tick_timer_cb, pc, next - now)) {
Review comment:
Comment as above in the epoll case - but note that here there may actually
be a bug as the subtraction is done using signed int64_t arithmetic which
doesn't have defined wraparound semantics.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org
