Simplify bswap_16 implementation

danra · web-flow · commit e40fa987e40c · 2017-08-25T19:23:02.000+03:00
Simplify bswap_16 implementation on platforms which don't already have it defined.
This has no effect on the generated assembly; it just simplifies the source code.
diff --git a/src/compat/byteswap.h b/src/compat/byteswap.h
@@ -35,7 +35,7 @@
 #if HAVE_DECL_BSWAP_16 == 0
 inline uint16_t bswap_16(uint16_t x)
 {
-    return (x >> 8) | ((x & 0x00ff) << 8);
+    return (x >> 8) | (x << 8);
 }
 #endif // HAVE_DECL_BSWAP16
 

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@`
`35`	`35`	`#if HAVE_DECL_BSWAP_16 == 0`
`36`	`36`	`inline uint16_t bswap_16(uint16_t x)`
`37`	`37`	`{`
`38`		`- return (x >> 8) \| ((x & 0x00ff) << 8);`
	`38`	`+ return (x >> 8) \| (x << 8);`
`39`	`39`	`}`
`40`	`40`	`#endif // HAVE_DECL_BSWAP16`
`41`	`41`