From 9eb47ada08d3c5cb35b7fe05b2bf9cf18f61e928 Mon Sep 17 00:00:00 2001 From: Craig Tiller Date: Tue, 5 May 2015 09:35:15 -0700 Subject: Further fixes --- include/grpc/grpc.h | 6 ++---- src/core/surface/completion_queue.h | 11 ----------- src/python/src/grpc/_adapter/_completion_queue.c | 1 - 3 files changed, 2 insertions(+), 16 deletions(-) diff --git a/include/grpc/grpc.h b/include/grpc/grpc.h index 1f5a8617df..63af727338 100644 --- a/include/grpc/grpc.h +++ b/include/grpc/grpc.h @@ -335,8 +335,7 @@ grpc_completion_queue *grpc_completion_queue_create(void); /* Blocks until an event is available, the completion queue is being shut down, or deadline is reached. Returns NULL on timeout, otherwise the event that - occurred. Callers should call grpc_event_finish once they have processed - the event. + occurred. Callers must not call grpc_completion_queue_next and grpc_completion_queue_pluck simultaneously on the same completion queue. */ @@ -345,8 +344,7 @@ grpc_event grpc_completion_queue_next(grpc_completion_queue *cq, /* Blocks until an event with tag 'tag' is available, the completion queue is being shutdown or deadline is reached. Returns NULL on timeout, or a pointer - to the event that occurred. Callers should call grpc_event_finish once they - have processed the event. + to the event that occurred. Callers must not call grpc_completion_queue_next and grpc_completion_queue_pluck simultaneously on the same completion queue. */ diff --git a/src/core/surface/completion_queue.h b/src/core/surface/completion_queue.h index 239bfe4aea..7b6fad98fd 100644 --- a/src/core/surface/completion_queue.h +++ b/src/core/surface/completion_queue.h @@ -46,17 +46,6 @@ void grpc_cq_internal_unref(grpc_completion_queue *cc); shutdown until a corrensponding grpc_cq_end_* call is made */ void grpc_cq_begin_op(grpc_completion_queue *cc, grpc_call *call); -/* grpc_cq_end_* functions pair with a grpc_cq_begin_op - - grpc_cq_end_* common arguments: - cc - the completion channel to queue on - tag - the user supplied operation tag - on_finish - grpc_event_finish_func that is called during grpc_event_finish - can be NULL to not get a callback - user_data - user_data parameter to be passed to on_finish - - Other parameters match the data member of grpc_event */ - /* Queue a GRPC_OP_COMPLETED operation */ void grpc_cq_end_op(grpc_completion_queue *cc, void *tag, grpc_call *call, int success); diff --git a/src/python/src/grpc/_adapter/_completion_queue.c b/src/python/src/grpc/_adapter/_completion_queue.c index 23e5b475e1..97828e67ad 100644 --- a/src/python/src/grpc/_adapter/_completion_queue.c +++ b/src/python/src/grpc/_adapter/_completion_queue.c @@ -440,7 +440,6 @@ static PyObject *pygrpc_completion_queue_get(CompletionQueue *self, if (tag) { pygrpc_tag_destroy(tag); } - grpc_event_finish(c_event); return event; } -- cgit v1.2.3