Closed Bug 1476657 Opened 2 years ago Closed 2 years ago

[Static Analysis] DEAD_STORE errors in js/src/frontend/Parser.cpp

Categories

(Core :: JavaScript Engine, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: rbartlensky, Assigned: rbartlensky)

References

Details

Attachments

(1 file)

js/src/frontend/Parser.cpp:1837: error: DEAD_STORE
  The value written to &cursor (type js::BindingName*) is never used.
  1835.
  1836.           bindings->constStart = cursor - start;
  1837. >         cursor = FreshlyInitializeBindings(cursor, consts);
  1838.
  1839.           bindings->length = numBindings;

js/src/frontend/Parser.cpp:1907: error: DEAD_STORE
  The value written to &cursor (type js::BindingName*) is never used.
  1905.
  1906.           bindings->constStart = cursor - start;
  1907. >         cursor = FreshlyInitializeBindings(cursor, consts);
  1908.
  1909.           bindings->length = numBindings;

js/src/frontend/Parser.cpp:1947: error: DEAD_STORE
  The value written to &cursor (type js::BindingName*) is never used.
  1945.           BindingName* cursor = start;
  1946.
  1947. >         cursor = FreshlyInitializeBindings(cursor, vars);
  1948.
  1949.           bindings->length = numBindings;

js/src/frontend/Parser.cpp:2046: error: DEAD_STORE
  The value written to &cursor (type js::BindingName*) is never used.
  2044.
  2045.           bindings->varStart = cursor - start;
  2046. >         cursor = FreshlyInitializeBindings(cursor, vars);
  2047.
  2048.           bindings->length = numBindings;

js/src/frontend/Parser.cpp:2087: error: DEAD_STORE
  The value written to &cursor (type js::BindingName*) is never used.
  2085.           BindingName* cursor = start;
  2086.
  2087. >         cursor = FreshlyInitializeBindings(cursor, vars);
  2088.
  2089.           bindings->length = numBindings;

js/src/frontend/Parser.cpp:2144: error: DEAD_STORE
  The value written to &cursor (type js::BindingName*) is never used.
  2142.
  2143.           bindings->constStart = cursor - start;
  2144. >         cursor = FreshlyInitializeBindings(cursor, consts);
  2145.
  2146.           bindings->length = numBindings;

js/src/frontend/Parser.cpp:4857: error: DEAD_STORE
  The value written to &declKind (type int) is never used.
  4855.               // Annex B.3.5 has different early errors for vars in for-of loops.
  4856.               if (declKind == DeclarationKind::Var)
  4857. >                 declKind = DeclarationKind::ForOfVar;
  4858.           } else {
  4859.               *forHeadKind = ParseNodeKind::ForHead;
Comment on attachment 8993048 [details]
Bug 1476657: Fix DEAD_STORE issues in js/src/frontend/Parser.cpp.

https://reviewboard.mozilla.org/r/257854/#review264852

Thanks!
Attachment #8993048 - Flags: review?(arai.unmht) → review+
Keywords: checkin-needed
Pushed by rgurzau@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6d259e78aa9c
Fix DEAD_STORE issues in js/src/frontend/Parser.cpp. r=arai
Keywords: checkin-needed
Backout by ebalazs@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a88fbbebadcc
Backed out changeset 6d259e78aa9c for bustage in /builds/worker/workspace/build/src/js/src/frontend/Parser.cpp on a CLOSED TREE
It seems that someone solved the issues already.
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
My bad, my tree wasn't merged properly...
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
I added 'Unused << ' everywhere, could you confirm that it is fine?
Flags: needinfo?(arai.unmht)
yes, looks good :)
Flags: needinfo?(arai.unmht)
Keywords: checkin-needed
Pushed by cbrindusan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/358b59601fb9
Fix DEAD_STORE issues in js/src/frontend/Parser.cpp. r=arai
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/358b59601fb9
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.