[libvirt] [PATCH 1/3] Add simple bitmap operations to utils

[Eric Blake]

On 05/21/2010 09:29 AM, Jim Fehlig wrote:
> 
> This and your other comments are addressed in V3.
> 
> V3:
>   - Check for overflow in virBitmapAlloc
>   - Fix copy and paste bug in virBitmapAlloc
>   - Use size_t in prototypes
>   - Add ATTRIBUTE_NONNULL in prototypes where appropriate
>     and remove NULL check form impl

If you keep that as part of the commit message, s/form/from/

(Personally, I put my comments about intra-version diffs...

> ---

...here, after the --- (by using git send-email --annotate), so that
they are part of the email trail but not the actual commit message.
After all, at the end of the day, the email trail shows how we got the
the end result, but the commit history need only show the end result if
we aren't committing the intermediate results.

But aside from my nitpicking on the commit message, the patch itself
looks good!

ACK.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100521/fba40905/attachment-0001.sig>