[Rpm-maint] [rpm-software-management/rpm] rpmio.c: In Fread(buf, size, nmemb, FD), require size be 1 (#400)
Panu Matilainen
notifications at github.com
Mon Feb 26 10:10:26 UTC 2018
I'd rather just fix the documentation to match behavior instead of introducing arbitrary assert()'s in the code. Something like
```
- * fread(3) clone.
+ * fread(3) clone, but return number of bytes instead of items.
```
...and ditto for Fwrite(). Of course there's the option of actually making the functions behave like fread()/fwrite(), but since this is long-standing behavior it seems more like gratuitous breakage than fixing anything.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/400#issuecomment-368451503
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20180226/1f46ddc8/attachment.html>
More information about the Rpm-maint
mailing list