[Rpm-maint] [rpm-software-management/rpm] Make macros::expand_numeric return int64_t (PR #3454)

[Panu Matilainen]

@pmatilai commented on this pull request.



>      if (arg == NULL)
 	return 0;
     auto [ ign, val ] = macros().expand_numeric(arg);
-    return val;
+    res = std::clamp(val, (int64_t) INT_MIN, (int64_t) INT_MAX);

Right... Any time one gets delusioned to think something simple like this would "just work", c++ is merry to prove you wrong :laughing: 



-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3454#discussion_r1848473227
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/3454/review/2445654163 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20241119/4ebaa420/attachment.htm>