Last Comment Bug 763580 - Clang warning "field is uninitialized when used here" in jsinterp.cpp
: Clang warning "field is uninitialized when used here" in jsinterp.cpp
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla16
Assigned To: Jan de Mooij [:jandem]
:
: Jason Orendorff [:jorendorff]
Mentors:
Depends on:
Blocks: buildwarning
  Show dependency treegraph
 
Reported: 2012-06-11 11:12 PDT by Jan de Mooij [:jandem]
Modified: 2012-06-12 04:40 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Patch (953 bytes, patch)
2012-06-11 11:12 PDT, Jan de Mooij [:jandem]
benjamin: review+
Details | Diff | Splinter Review

Description Jan de Mooij [:jandem] 2012-06-11 11:12:20 PDT
Created attachment 631952 [details] [diff] [review]
Patch

jsinterp.cpp:1032:15: warning: field is uninitialized when used here [-Wuninitialized]
         : cx(cx), arr(scx.cx, scx.arr), count(scx.count) {}
              ^
Comment 1 :Benjamin Peterson 2012-06-11 11:21:11 PDT
Comment on attachment 631952 [details] [diff] [review]
Patch

Review of attachment 631952 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM
Comment 2 Jan de Mooij [:jandem] 2012-06-11 13:46:09 PDT
Thanks for the quick review.

https://hg.mozilla.org/integration/mozilla-inbound/rev/fbbf7c4e598f
Comment 3 Graeme McCutcheon [:graememcc] 2012-06-12 03:05:25 PDT
https://hg.mozilla.org/mozilla-central/rev/fbbf7c4e598f

Note You need to log in before you can comment on or make changes to this bug.