Commit 2bd4b5e2 authored by Marko Mäkelä's avatar Marko Mäkelä

Make UNIV_DEBUG Valgrind friendly in the built-in InnoDB.

Use | instead of +, and mask out the dont-care bits in debug assertions.
parent 973c59be
...@@ -19,7 +19,7 @@ mach_write_to_1( ...@@ -19,7 +19,7 @@ mach_write_to_1(
ulint n) /* in: ulint integer to be stored, >= 0, < 256 */ ulint n) /* in: ulint integer to be stored, >= 0, < 256 */
{ {
ut_ad(b); ut_ad(b);
ut_ad(n <= 0xFFUL); ut_ad((n | 0xFFUL) <= 0xFFUL);
b[0] = (byte)n; b[0] = (byte)n;
} }
...@@ -48,7 +48,7 @@ mach_write_to_2( ...@@ -48,7 +48,7 @@ mach_write_to_2(
ulint n) /* in: ulint integer to be stored */ ulint n) /* in: ulint integer to be stored */
{ {
ut_ad(b); ut_ad(b);
ut_ad(n <= 0xFFFFUL); ut_ad((n | 0xFFFFUL) <= 0xFFFFUL);
b[0] = (byte)(n >> 8); b[0] = (byte)(n >> 8);
b[1] = (byte)(n); b[1] = (byte)(n);
...@@ -64,10 +64,7 @@ mach_read_from_2( ...@@ -64,10 +64,7 @@ mach_read_from_2(
/* out: ulint integer */ /* out: ulint integer */
byte* b) /* in: pointer to 2 bytes */ byte* b) /* in: pointer to 2 bytes */
{ {
ut_ad(b); return(((ulint)(b[0]) << 8) | (ulint)(b[1]));
return( ((ulint)(b[0]) << 8)
+ (ulint)(b[1])
);
} }
/************************************************************ /************************************************************
...@@ -112,7 +109,7 @@ mach_write_to_3( ...@@ -112,7 +109,7 @@ mach_write_to_3(
ulint n) /* in: ulint integer to be stored */ ulint n) /* in: ulint integer to be stored */
{ {
ut_ad(b); ut_ad(b);
ut_ad(n <= 0xFFFFFFUL); ut_ad((n | 0xFFFFFFUL) <= 0xFFFFFFUL);
b[0] = (byte)(n >> 16); b[0] = (byte)(n >> 16);
b[1] = (byte)(n >> 8); b[1] = (byte)(n >> 8);
...@@ -131,8 +128,8 @@ mach_read_from_3( ...@@ -131,8 +128,8 @@ mach_read_from_3(
{ {
ut_ad(b); ut_ad(b);
return( ((ulint)(b[0]) << 16) return( ((ulint)(b[0]) << 16)
+ ((ulint)(b[1]) << 8) | ((ulint)(b[1]) << 8)
+ (ulint)(b[2]) | (ulint)(b[2])
); );
} }
...@@ -166,9 +163,9 @@ mach_read_from_4( ...@@ -166,9 +163,9 @@ mach_read_from_4(
{ {
ut_ad(b); ut_ad(b);
return( ((ulint)(b[0]) << 24) return( ((ulint)(b[0]) << 24)
+ ((ulint)(b[1]) << 16) | ((ulint)(b[1]) << 16)
+ ((ulint)(b[2]) << 8) | ((ulint)(b[2]) << 8)
+ (ulint)(b[3]) | (ulint)(b[3])
); );
} }
...@@ -670,7 +667,7 @@ mach_read_from_2_little_endian( ...@@ -670,7 +667,7 @@ mach_read_from_2_little_endian(
/* out: unsigned long int */ /* out: unsigned long int */
byte* buf) /* in: from where to read */ byte* buf) /* in: from where to read */
{ {
return((ulint)(*buf) + ((ulint)(*(buf + 1))) * 256); return((ulint)(buf[0]) | ((ulint)(buf[1]) << 8));
} }
/************************************************************* /*************************************************************
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment