Been working through some Microchip TechBriefs. In TB3217 (introduction to TCA in Mega0, Tinyo/1, and DA chips) came across this line of code and several like it:
TCA0.SPLIT.CTRLA = TCA_SPLIT_CLKSEL_DIV16_gc /* set clock source (sys_clk/16) */ | TCA_SPLIT_ENABLE_bm; /* start timer */
Now, I have relatively little doubt that it is legal. But, this is the first time I have encountered code like this and I do not like it. I understand why it is used, but that does not make it "good". It is so easy to loose where the true end of the code line is!
Might as well write:
TCA0.SPLIT.CTRLA = /* what is this? */ TCA_SPLIT_CLKSEL_DIV16_gc | TCA_SPLIT_ENABLE_bm;
Why not? It seems to be allowed and that must make it "A Good Thing".
 Typo correct