|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
58 bytes, text/x-review-board-request
|Details | Review|
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.
Created attachment 8752734 [details] MozReview Request: Bug 1273075 - initialize |mPixelsPerFrame| and |mPixelsFilled|. r?jrmuizel 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+
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox49: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.