Closed
Bug 1232646
Opened 8 years ago
Closed 8 years ago
[Static Analysis][Uninitialized fields] Function AudioTimelineEvent(MediaStream* ) from AudioEventTimeline.h
Categories
(Core :: Audio/Video, defect)
Core
Audio/Video
Tracking
()
RESOLVED
FIXED
mozilla46
Tracking | Status | |
---|---|---|
firefox46 | --- | fixed |
People
(Reporter: andi, Assigned: andi)
References
(Blocks 1 open bug)
Details
(Keywords: coverity, Whiteboard: CID 1325710)
Attachments
(1 file)
771 bytes,
patch
|
cpearce
:
review+
|
Details | Diff | Splinter Review |
The Static Analysis tool Coverity added that class members mCurve, mTimeConstant and mDuration are not initialized in the constructor. Even though there is no evident bug that is generated from this issue i think it should be better, for future proof, to initialize these 3 variables.
Assignee | ||
Comment 1•8 years ago
|
||
Attachment #8698418 -
Flags: review?(cpearce)
Updated•8 years ago
|
Attachment #8698418 -
Flags: review?(cpearce) → review+
Assignee | ||
Updated•8 years ago
|
Keywords: checkin-needed
Comment 3•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ec15b39b2180
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in
before you can comment on or make changes to this bug.
Description
•