Hi

The functions libssh2_sftp_read() and libssh2_sftp_write() are declared to 
return size_t.

First, I don't like that the code then proceeds and return -1 when size_t is 
in fact unsigned on most platforms.

I would much rather see them resember read() and write() more closely (as 
that's what their names somewhat imply at least in my head) and they return 
ssize_t.

Would anyone object if I modified this?

Should we somehow write down API/ABI changes somewhere for the upcoming 
release notes?

Should we perhaps start noting our changes in a ChangeLog somewhere (I mean 
generally, not only API/ABI changes)? I think we all would benefit from 
that...

-------------------------------------------------------------------------
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
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/libssh2-devel

Reply via email to