[Rpm-maint] [rpm] Rename expandMacrosU to rpmExpandMacros (#34)

Panu Matilainen notifications at github.com
Wed Nov 25 13:17:13 UTC 2015


Looks sane to me to.

Just a note on error returns: NULL is a blunt tool for reporting errors, when adding new interfaces its worth considering whether you might want more nyanced feedback in some cases / in the future. That rpm returns NULL instead of error code in many cases does not necessarily make it a good design :)

---
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/34#issuecomment-159604687
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20151125/be545c31/attachment.html>


More information about the Rpm-maint mailing list