From 5438036077f87ff353267763c1a9606b9490c8f9 Mon Sep 17 00:00:00 2001
From: Pras Velagapudi <psigen@gmail.com>
Date: Wed, 3 Aug 2016 13:44:51 -0400
Subject: [PATCH] Fix for compilation warnings in bitwise operations

The ADXL362 driver embedded in this header throws compiler warnings about bitwise arithmetic.  This is because there are one-liners containing multiple bitwise operators with no parens.  These are generally not recommended because they depend on operator precedence to evaluate in the correct order, and that can be quite tricky to get right.

This commit simply adds some parens that follow `C` operator precedence (doing what the statement was already doing, but explicitly).
---
 firmware/InternetButton.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/firmware/InternetButton.cpp b/firmware/InternetButton.cpp
index 76846cd..dd094c3 100644
--- a/firmware/InternetButton.cpp
+++ b/firmware/InternetButton.cpp
@@ -617,7 +617,7 @@ void ADXL362::setRange(uint8_t Range){
     break;
   }
 
-  temp = temp & 0b00111111 | Range;
+  temp = (temp & 0b00111111) | Range;
   SPIwriteOneRegister(XL362_FILTER_CTL, temp); // Write to XL362_FILTER_CTL
   delay(10);
 
@@ -637,7 +637,7 @@ void ADXL362::setBandwidth(uint8_t BandWidth){
   Serial.print(temp);
 #endif
 
-  temp = temp & 0b11101111 | BandWidth;
+  temp = (temp & 0b11101111) | BandWidth;
   SPIwriteOneRegister(XL362_FILTER_CTL, temp); // Write to XL362_FILTER_CTL
   delay(10);
 
@@ -657,7 +657,7 @@ void ADXL362::setOutputDatarate(uint8_t ODR){
   Serial.print(temp);
 #endif
 
-  temp = temp & 0b11111000 | ODR;
+  temp = (temp & 0b11111000) | ODR;
   SPIwriteOneRegister(XL362_FILTER_CTL, temp); // Write to XL362_FILTER_CTL
   delay(10);
 
@@ -677,7 +677,7 @@ void ADXL362::setNoiseLevel(uint8_t NoiseLevel){
   Serial.print(temp);
 #endif
 
-  temp = temp & 0b11001111  | NoiseLevel;
+  temp = (temp & 0b11001111)  | NoiseLevel;
   SPIwriteOneRegister(XL362_POWER_CTL, temp); // Write to XL362_FILTER_CTL
   delay(10);