Closed
Bug 1232646
Opened 10 years ago
Closed 10 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•10 years ago
|
||
Attachment #8698418 -
Flags: review?(cpearce)
Updated•10 years ago
|
Attachment #8698418 -
Flags: review?(cpearce) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Keywords: checkin-needed
Comment 3•10 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in
before you can comment on or make changes to this bug.
Description
•