WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 74977
[Coverity] Address some uninit constructor issues in WebCore/dom
https://bugs.webkit.org/show_bug.cgi?id=74977
Summary
[Coverity] Address some uninit constructor issues in WebCore/dom
Greg Billock
Reported
2011-12-20 17:04:14 PST
[Coverity] Address some uninit constructor issues in WebCore/dom
Attachments
Patch
(2.44 KB, patch)
2011-12-20 17:05 PST
,
Greg Billock
no flags
Details
Formatted Diff
Diff
Patch
(2.46 KB, patch)
2011-12-22 08:49 PST
,
Greg Billock
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Greg Billock
Comment 1
2011-12-20 17:05:31 PST
Created
attachment 120117
[details]
Patch
Alexey Proskuryakov
Comment 2
2011-12-21 16:22:04 PST
Comment on
attachment 120117
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=120117&action=review
> Source/WebCore/dom/DeviceMotionData.cpp:79 > + , m_interval(0.0)
Please, no ".0".
> Source/WebCore/dom/DeviceOrientation.cpp:48 > + , m_alpha(0.0) > + , m_beta(0.0) > + , m_gamma(0.0)
Ditto.
Greg Billock
Comment 3
2011-12-22 08:49:35 PST
Created
attachment 120326
[details]
Patch
Greg Billock
Comment 4
2011-12-22 08:51:46 PST
Comment on
attachment 120117
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=120117&action=review
>> Source/WebCore/dom/DeviceMotionData.cpp:79 >> + , m_interval(0.0) > > Please, no ".0".
Done
>> Source/WebCore/dom/DeviceOrientation.cpp:48 >> + , m_gamma(0.0) > > Ditto.
Done.
Greg Billock
Comment 5
2012-01-04 13:15:03 PST
Any further comments on this patch?
WebKit Review Bot
Comment 6
2012-01-11 12:42:14 PST
Comment on
attachment 120326
[details]
Patch Clearing flags on attachment: 120326 Committed
r104734
: <
http://trac.webkit.org/changeset/104734
>
WebKit Review Bot
Comment 7
2012-01-11 12:42:18 PST
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug