Clarifications in response to merge req. comments

diff --git a/bufferevent-internal.h b/bufferevent-internal.h
index 12ae142baa6f067f58b7d6af2e0c46260b3a3eda..70b25cd90464006aab8a4b8a9ce8c082bbbc1940 100644 (file)
--- a/bufferevent-internal.h
+++ b/bufferevent-internal.h
@@ -340,10 +340,10 @@ int bufferevent_decref_(struct bufferevent *bufev);
 int bufferevent_decref_and_unlock_(struct bufferevent *bufev);
 
 /** Internal: If callbacks are deferred and we have a read callback, schedule
- * a readcb.  Otherwise just run the readcb. */
+ * a readcb.  Otherwise just run the readcb. Ignores watermarks. */
 void bufferevent_run_readcb_(struct bufferevent *bufev, int options);
 /** Internal: If callbacks are deferred and we have a write callback, schedule
- * a writecb.  Otherwise just run the writecb. */
+ * a writecb.  Otherwise just run the writecb. Ignores watermarks. */
 void bufferevent_run_writecb_(struct bufferevent *bufev, int options);
 /** Internal: If callbacks are deferred and we have an eventcb, schedule
  * it to run with events "what".  Otherwise just run the eventcb.
@@ -352,7 +352,8 @@ void bufferevent_run_eventcb_(struct bufferevent *bufev, short what, int options
 
 /** Internal: Run or schedule (if deferred or options contain
  * BEV_TRIG_DEFER_CALLBACKS) I/O callbacks specified in iotype.
- * Must already hold the bufev lock. */
+ * Must already hold the bufev lock. Honors watermarks unless
+ * BEV_TRIG_IGNORE_WATERMARKS is in options. */
 void bufferevent_trigger_nolock_(struct bufferevent *bufev, short iotype, int options);
 
 

diff --git a/bufferevent.c b/bufferevent.c
index 5f424d7e72723e33b0c5a43cbd11656348303d36..d8c84da4caf4f9e85705900ce8ca46d6235b74db 100644 (file)
--- a/bufferevent.c
+++ b/bufferevent.c
@@ -629,14 +629,14 @@ bufferevent_getwatermark(struct bufferevent *bufev, short events,
     size_t *lowmark, size_t *highmark)
 {
        BEV_LOCK(bufev);
-       if (events & EV_WRITE) {
+       if (events == EV_WRITE) {
                if (lowmark)
                        *lowmark = bufev->wm_write.low;
                if (highmark)
                        *highmark = bufev->wm_write.high;
        }
 
-       if (events & EV_READ) {
+       if (events == EV_READ) {
                if (lowmark)
                        *lowmark = bufev->wm_read.low;
                if (highmark)