Closed Bug 1289792 Opened 4 years ago Closed 4 years ago
[Static Analysis][Uninitialized scalar field] In constructor Dispatch
Wheel Input On Controller Thread
58 bytes, text/x-review-board-request
The Static Analysis tool Coverity detected that |mAPZResult| is not initialised in the constructor. But looking over the code i think we can remove this member variable and just use a local variable since it's not used in any other function nor it's passed as a reference nor pointer to DispatchWheelEventOnMainThread.
Review commit: https://reviewboard.mozilla.org/r/67424/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/67424/
Attachment #8775156 - Flags: review?(vladimir)
Attachment #8775156 - Flags: review?(vladimir) → review+
Comment on attachment 8775156 [details] Bug 1289792 - removed member variable mAPZResult and used a local one in Run(). https://reviewboard.mozilla.org/r/67424/#review64524
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/99cc6fc1ed3f removed member variable mAPZResult and used a local one in Run(). r=vlad
You need to log in before you can comment on or make changes to this bug.