Bug 1273103 - initialize |lineno_|, |column_|. r?jorendorff draft
authorAndi-Bogdan Postelnicu <bogdan.postelnicu@softvision.ro>
Tue, 17 May 2016 15:21:59 +0300
changeset 367817 48bfe349caa06d067fd0059961f234ee29f04c70
parent 367573 a884b96685aa13b65601feddb24e5f85ba861561
child 521111 b94be507bf8c6b1e0ed9fd91b6f7fdd0d1f3a6c8
push id18357
push userBogdan.Postelnicu@softvision.ro
push dateTue, 17 May 2016 12:27:05 +0000
reviewersjorendorff
bugs1273103
milestone49.0a1
Bug 1273103 - initialize |lineno_|, |column_|. r?jorendorff MozReview-Commit-ID: KbluCZJMbNy
js/src/vm/Debugger.cpp
--- a/js/src/vm/Debugger.cpp
+++ b/js/src/vm/Debugger.cpp
@@ -5517,17 +5517,17 @@ class FlowGraphSummary {
         static Entry createWithMultipleEdgesFromSingleLine(size_t lineno) {
             return Entry(lineno, SIZE_MAX);
         }
 
         static Entry createWithMultipleEdgesFromMultipleLines() {
             return Entry(SIZE_MAX, SIZE_MAX);
         }
 
-        Entry() {}
+        Entry() : lineno_(SIZE_MAX), column_(0) {}
 
         bool hasNoEdges() const {
             return lineno_ == SIZE_MAX && column_ != SIZE_MAX;
         }
 
         bool hasSingleEdge() const {
             return lineno_ != SIZE_MAX && column_ != SIZE_MAX;
         }
@@ -5569,18 +5569,16 @@ class FlowGraphSummary {
         return entries_[index];
     }
 
     bool populate(JSContext* cx, JSScript* script) {
         if (!entries_.growBy(script->length()))
             return false;
         unsigned mainOffset = script->pcToOffset(script->main());
         entries_[mainOffset] = Entry::createWithMultipleEdgesFromMultipleLines();
-        for (size_t i = mainOffset + 1; i < script->length(); i++)
-            entries_[i] = Entry::createWithNoEdges();
 
         size_t prevLineno = script->lineno();
         size_t prevColumn = 0;
         JSOp prevOp = JSOP_NOP;
         for (BytecodeRangeWithPosition r(cx, script); !r.empty(); r.popFront()) {
             size_t lineno = r.frontLineNumber();
             size_t column = r.frontColumnNumber();
             JSOp op = r.frontOpcode();