php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Request #59235 Missed event_buffer_setcb()
Submitted: 2010-05-27 06:13 UTC Modified: 2010-05-27 10:05 UTC
From: kak dot serpom dot po dot yaitsam at gmail dot com Assigned:
Status: Closed Package: libevent (PECL)
PHP Version: 5_3 SVN-2010-05-27 (dev) OS: Fedora 13
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: kak dot serpom dot po dot yaitsam at gmail dot com
New email:
PHP Version: OS:

 

 [2010-05-27 06:13 UTC] kak dot serpom dot po dot yaitsam at gmail dot com
Description:
------------
I've noticed that libevent has the following function:

bufferevent_setcb (struct bufferevent *bufev, evbuffercb 
readcb, evbuffercb writecb, everrorcb errorcb, void *cbarg)
 	Changes the callbacks for a bufferevent. 


Please, make wrapper to this function. It's very useful for me 
and many other people. Thanks.


Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2010-05-27 06:46 UTC] tony at daylessday dot org
http://dev.daylessday.org/diff/event_buffer_set_callback.diff
check this patch out.

I'm not sure if error callback should be still a requirement, though it is in libevent itself.
The point is that we actually always specify C callbacks for libevent, but PHP callbacks can be missing - that won't result in any error.
What do you think?
 [2010-05-27 10:05 UTC] kak dot serpom dot po dot yaitsam at gmail dot com
It works! Excuse me for the late answer and thank you very 
much for quick patch.

I'm not sure too, but I always define errorcb. I think that it 
should be a requirement, because it's requirement at C-level.
 
PHP Copyright © 2001-2017 The PHP Group
All rights reserved.
Last updated: Sun Nov 19 01:31:42 2017 UTC