>> --- include/libssh2.h 4 Nov 2006 19:30:31 -0000 1.60
>> +++ include/libssh2.h 23 Nov 2006 17:06:42 -0000
>> @@ -172,8 +172,8 @@ typedef struct _LIBSSH2_POLLFD {
>> LIBSSH2_LISTENER *listener; /* Read polls only -- are inbound connections waiting to be accepted? */
>> } fd;
>>
>> - unsigned long events; /* Requested Events */
>> - unsigned long revents; /* Returned Events */
>> + unsigned long ssh2events; /* Requested Events */
>> + unsigned long ssh2revents; /* Returned Events */
>> } LIBSSH2_POLLFD;
>
> As this is in include/libssh2.h, are applications expected to access
> those fields? If so the patch would create API problems obviously, but
> it would be tricky to avoid that.
>
> -Tor (aka tor2)
>
Yep, calling apps are expected to set events and review revents similar
to syscalls to poll(). This will definately cause a problem for
existing applications using the libssh2_poll() API, but it's not a major
one and can be fixed in these few applications through the use of an
ifdef/define.
I'm inclined to say this break is okay while we're in beta versions so
long as it's noted in large type both in the release notes and the
manual page on the wiki.
-Sara
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
libssh2-devel mailing list
libssh2-devel_at_lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libssh2-devel
Received on 2006-11-23