[Rpm-maint] [rpm-software-management/rpm] Turn nrefs into atomic_int (PR #3370)

Panu Matilainen notifications at github.com
Fri Oct 11 06:50:34 UTC 2024


Whoa. Unfortunately it's not that simple, the increment is atomic but the comparison is not.

It's a step in the right direction though and we can take that step, but slice this up a bit: the places where the change is just a straightforward and obvious Unlink -> --nrefs change can go in one bigger commit, but do each of the special cases as a separate commit, each with their own explanations. Makes it easier to bisect it if something goes wrong.

Mind, this is not a review, just took a quick overview of it.

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

Message ID: <rpm-software-management/rpm/pull/3370/c2406666366 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20241010/18c32bd5/attachment.html>


More information about the Rpm-maint mailing list