[Rpm-maint] [rpm-software-management/rpm] Refactor rpmds internals to use STL vectors and strings instead of C dynamic allocations (PR #3953)
Michal Domonkos
notifications at github.com
Mon Sep 29 12:25:48 UTC 2025
@dmnks commented on this pull request.
> }
if (N)
- t = stpcpy(t, N);
+ s.append(N);
Maybe use `s += N` here, or use `s.append(EVR)` below? :smile: It's a bit inconsistent in this function (although not a big deal of course, just cosmetic).
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3953#pullrequestreview-3279501123
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/3953/review/3279501123 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20250929/ee122090/attachment.htm>
More information about the Rpm-maint
mailing list