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
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to