bitWrite macro does not enclose argument in ()

[positron96]

ArduinoCore-API/api/Common.h

Line 68 in 173e8ea

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

Current version does not wrap arguments in "()", so it works incorrectly when used like this:

bitWrite(var, bit, cond?1:0);

Some cores probably copied this header, as seen in stm32 and esp32

[facchinm]

Hi @positron96 ,
thanks for spotting this! would you mind submitting a PR with the fix so we can give the proper attribution?