<div dir="ltr"><div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Mon, Sep 17, 2018 at 6:03 PM Richard W.M. Jones <<a href="mailto:rjones@redhat.com">rjones@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">This should have no effect.  However it's probably better to pass the<br>
full type explicitly rather than using an implicit truncation.<br></blockquote><div><br></div><div>Why is it better?</div><div><br></div><div>We care only about the least significant bits; current code is clear about that.</div><div><br></div><div>This is also specified behavior, although the only reference I can find now is this:</div><div><a href="https://docs.microsoft.com/en-us/cpp/c-language/conversions-from-unsigned-integral-types?view=vs-2017">https://docs.microsoft.com/en-us/cpp/c-language/conversions-from-unsigned-integral-types?view=vs-2017</a><br></div><div><br></div><div>Nir</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
---<br>
 common/include/isaligned.h | 3 ++-<br>
 1 file changed, 2 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/common/include/isaligned.h b/common/include/isaligned.h<br>
index e693820..9ed635e 100644<br>
--- a/common/include/isaligned.h<br>
+++ b/common/include/isaligned.h<br>
@@ -36,13 +36,14 @@<br>
<br>
 #include <assert.h><br>
 #include <stdbool.h><br>
+#include <stdint.h><br>
<br>
 #include "ispowerof2.h"<br>
<br>
 /* Return true if size is a multiple of align. align must be power of 2.<br>
  */<br>
 static inline bool<br>
-is_aligned (unsigned int size, unsigned int align)<br>
+is_aligned (uint64_t size, unsigned int align)<br>
 {<br>
   assert (is_power_of_2 (align));<br>
   return !(size & (align - 1));<br>
-- <br>
2.19.0.rc0<br>
<br>
_______________________________________________<br>
Libguestfs mailing list<br>
<a href="mailto:Libguestfs@redhat.com" target="_blank">Libguestfs@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/libguestfs" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/libguestfs</a><br>
</blockquote></div></div></div>