Closed Bug 1273075 Opened 5 years ago Closed 5 years ago
[Static Analysis][Uninitialized scalar field] In constructor explicit Compositor
58 bytes, text/x-review-board-request
The Static Analysis tool Coverity added that variables |mPixelsPerFrame|, |mPixelsFilled| are uninitialized. As we are trying to push for a more RAII architecture and we want to prepare our code for this clang-plugin - 525063. We should initialize these member variables.
Review commit: https://reviewboard.mozilla.org/r/52754/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/52754/
Attachment #8752734 - Flags: review?(jmuizelaar)
Comment on attachment 8752734 [details] MozReview Request: Bug 1273075 - initialize |mPixelsPerFrame| and |mPixelsFilled|. r?jrmuizel https://reviewboard.mozilla.org/r/52754/#review49714
Attachment #8752734 - Flags: review?(jmuizelaar) → review+
You need to log in before you can comment on or make changes to this bug.