You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by tr...@apache.org on 2009/02/28 18:40:11 UTC

svn commit: r748892 - /apr/apr/trunk/include/apr_poll.h

Author: trawick
Date: Sat Feb 28 17:40:10 2009
New Revision: 748892

URL: http://svn.apache.org/viewvc?rev=748892&view=rev
Log:
Document the restriction that you can't (successfully) add the same socket or
file to the same pollset.

(At present, two of the n implementations will let you do it, but the subsequent
results from a poll operation can differ from when add the socket/file once
with all necessary request events.)

Modified:
    apr/apr/trunk/include/apr_poll.h

Modified: apr/apr/trunk/include/apr_poll.h
URL: http://svn.apache.org/viewvc/apr/apr/trunk/include/apr_poll.h?rev=748892&r1=748891&r2=748892&view=diff
==============================================================================
--- apr/apr/trunk/include/apr_poll.h (original)
+++ apr/apr/trunk/include/apr_poll.h Sat Feb 28 17:40:10 2009
@@ -200,6 +200,12 @@
  * @remark If the pollset has been created with APR_POLLSET_NOCOPY, the 
  *         apr_pollfd_t structure referenced by descriptor will not be copied
  *         and must have a lifetime at least as long as the pollset.
+ * @remark Do not add the same socket or file descriptor to the same pollset
+ *         multiple times, even if the requested events differ for the 
+ *         different calls to apr_pollset_add().  If the events of interest
+ *         for a descriptor change, you must first remove the descriptor 
+ *         from the pollset with apr_pollset_remove(), then add it again 
+ *         specifying all requested events.
  */
 APR_DECLARE(apr_status_t) apr_pollset_add(apr_pollset_t *pollset,
                                           const apr_pollfd_t *descriptor);