On Sat, Feb 01, 2020 at 07:33:43AM +0100, Michal Privoznik wrote:
Instead of calling virAtomicIntAdd(&var, 1); we can call virAtomicIntInc(&var) directly.Signed-off-by: Michal Privoznik <mpriv...@redhat.com> --- src/test/test_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/test/test_driver.c b/src/test/test_driver.c index 47c1fc588f..6f617592f3 100644 --- a/src/test/test_driver.c +++ b/src/test/test_driver.c @@ -695,7 +695,7 @@ testDomainStartState(testDriverPtr privconn, int ret = -1; virDomainObjSetState(dom, VIR_DOMAIN_RUNNING, reason); - dom->def->id = virAtomicIntAdd(&privconn->nextDomID, 1); + dom->def->id = virAtomicIntInc(&privconn->nextDomID);
g_atomic_int_add(&privconn->nextDomID, 1); I don't see the value of changing it to a function that is being phased out. With that fixed: Reviewed-by: Ján Tomko <jto...@redhat.com> Jano
signature.asc
Description: PGP signature
