Something like this can't really be a macro configurable between such wildly differing purposes, especially when callers almost certainly expect the traditional default behavior anyway. I can't see anybody missing this.
Besides nuking an apparently unnecessary feature and weird logic (how many levels of default do you want?), it eliminates an unwanted static initializer relying on a non-enum value of an enum variable. You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/2954 -- Commit Summary -- * Kick out an "experimental" %_query_selector_match from 2001 -- File Changes -- M lib/rpmdb.c (20) M macros.in (11) -- Patch Links -- https://github.com/rpm-software-management/rpm/pull/2954.patch https://github.com/rpm-software-management/rpm/pull/2954.diff -- Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/2954 You are receiving this because you are subscribed to this thread. Message ID: <rpm-software-management/rpm/pull/2...@github.com>
_______________________________________________ Rpm-maint mailing list Rpm-maint@lists.rpm.org http://lists.rpm.org/mailman/listinfo/rpm-maint
