Attention is currently required from: Shreeyash .
4 comments:
Commit Message:
Patch Set #2, Line 7: Bug fix, Initialize uninitialized pointers
How about `fix false-positive compiler error`?
Patch Set #2, Line 9: on line 510
After rebasing, the line number is not correct. I suggest deleting it.
Patchset:
Please have a look at CB:67700 (especially on unresolved comments). This patch does the same, so it's a clone.
You should rewrite the commit message to point out that it's false-positive case.
Because then it's not a fix for anything in flashrom, certainly not a bug fix, but a workaround to assist limited path analysis. (C) Nico Huber
If you want, you can use the commit message from the patch I mentioned earlier.
File stlinkv3_spi.c:
Patch Set #1, Line 501: stlinkv3_handle = usb_dev_get_by_vid_pid_serial(usb_ctx,
I think both the issues have been 'fixed'. […]
Add the comment that Thomas suggests to you. But put it before the variable, otherwise 112-columns hard limit will be broken.
`/* Initialize stlinkv3_handle to NULL for suppressing scan-build false positive core.uninitialized.Branch */`
Thomas, can we get rid of `core.uninitialized.Branch`? It doesn't suppress the warning :D
To view, visit change 66225. To unsubscribe, or for help writing mail filters, visit settings.