bitWrite macro incorrectly expands ternary operator

_From @svatoun on December 13, 2016 8:34_

let's have a bitWrite(data, bitIndex, computedValue == 5 ? 1: 0);

bitWrite macro definition lacks parenthesis around the `bitvalue' parameter, so it expands as follows

(computedValue == 5 ? 1 : 0 ? bitSet(value, bit) : bitClear(value, bit))

and the `0 ? bitSet...' is then compiled as the false branch of the ternary operator in the passed expression.
Please correct the definition as follows:

#define bitWrite(value, bit, bitvalue) ((bitvalue) ? bitSet(value, bit) : bitClear(value, bit))

(parenthesis added)

_Copied from original issue: arduino/Arduino#5714_

Locate the bitWrite macro definition in ArduinoCore-sam and inspect how the bitvalue parameter is expanded in a ternary call. Confirm that parenthesizing that parameter prevents the passed expression from changing the macro's conditional grouping, then validate the affected compilation case.

Written by the indexing model from the issue text.