[coreboot-gerrit] Change in coreboot[master]: src: Add '-Wduplicated-branches' warning option

[Nico Huber (Code Review)]

Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39561 )

Change subject: src: Add '-Wduplicated-branches' warning option
......................................................................

Patch Set 22:

(1 comment)

Such warning options can produce a lot of noise, e.g. false
positives. Unless there was a serious bug in the past that
would have been discovered by this, I'd prefer _not_ to use
this option.

https://review.coreboot.org/c/coreboot/+/39561/20/src/vendorcode/amd/agesa/f... 
File src/vendorcode/amd/agesa/f15tn/Proc/GNB/Modules/GnbInitTN/GfxIntegratedInfoTableTN.c:

https://review.coreboot.org/c/coreboot/+/39561/20/src/vendorcode/amd/agesa/f... 
PS20, Line 188:     ((Channel == 0) ? D18F2x94_dct0_ADDRESS : D18F2x94_dct1_ADDRESS),
> What is the warning here? The code looks correct?

Maybe both types are just the same?