summary refs log tree commit diff stats
path: root/results/scraper/launchpad/1658506
blob: 54f7efbfe5e539c0401d5e21ef8f37ae7f82ddd4 (plain) (blame)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
qemu/hw/intc/arm_gicv3_cpuif.c:2433: bad expression ?

qemu/hw/intc/arm_gicv3_cpuif.c:2433]: (style) Expression '(X & 0x2000000000000000) == 0x1' is always false.

Source code is

           ((lr & ICH_LR_EL2_HW) == 1 || (lr & ICH_LR_EL2_EOI) == 0)) {

Maybe better code

           ((lr & ICH_LR_EL2_HW) != 0 || (lr & ICH_LR_EL2_EOI) == 0)) {

Thanks for this bug report -- I have sent a patch making the proposed change: http://lists.nongnu.org/archive/html/qemu-devel/2017-01/msg05030.html


Now fixed in git master, commit d87576e38df7.