virBitmapUnion: Remove return value

The function can't fail at this point. Remove the return value.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
This commit is contained in:
Peter Krempa 2021-12-06 15:55:46 +01:00
parent ab88ace58d
commit 58fec2cef4
4 changed files with 5 additions and 11 deletions

View File

@ -1164,10 +1164,8 @@ virBitmapIntersect(virBitmap *a,
* @b: other bitmap
*
* Performs union of two bitmaps: a = union(a, b)
*
* Returns 0 on success, <0 on failure.
*/
int
void
virBitmapUnion(virBitmap *a,
const virBitmap *b)
{
@ -1178,8 +1176,6 @@ virBitmapUnion(virBitmap *a,
for (i = 0; i < b->map_len; i++)
a->map[i] |= b->map[i];
return 0;
}

View File

@ -133,8 +133,8 @@ bool virBitmapOverlaps(virBitmap *b1,
void virBitmapIntersect(virBitmap *a, virBitmap *b)
ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2);
int virBitmapUnion(virBitmap *a,
const virBitmap *b)
void virBitmapUnion(virBitmap *a,
const virBitmap *b)
ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2);
void virBitmapSubtract(virBitmap *a, virBitmap *b)

View File

@ -333,8 +333,7 @@ virNumaNodesetToCPUset(virBitmap *nodeset,
return -1;
}
if (virBitmapUnion(allNodesCPUs, nodeCPUs) < 0)
return -1;
virBitmapUnion(allNodesCPUs, nodeCPUs);
}
*cpuset = g_steal_pointer(&allNodesCPUs);

View File

@ -665,8 +665,7 @@ test15(const void *opaque)
return -1;
}
if (virBitmapUnion(amap, bmap) < 0)
return -1;
virBitmapUnion(amap, bmap);
if (!virBitmapEqual(amap, resmap)) {
fprintf(stderr,