1273075 - [Static Analysis][Uninitialized scalar field] In constructor explicit Compositor

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Andi [:andi]]

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.

Comment 1

[Andi [:andi]]

Attachment: 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/

Comment 2

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 8752734 [details]
MozReview Request: Bug 1273075 - initialize |mPixelsPerFrame| and |mPixelsFilled|. r?jrmuizel

https://reviewboard.mozilla.org/r/52754/#review49714

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ee5016917ea1

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/ee5016917ea1