A few DebugOnly<> variables and #ifdef DEBUG functions. Also removed unused variables in TypedArrayObject.cpp (try build coming soon).

This one might not be of interest, or could be implemented differently. It kills the warning that the return value of read() is unused. We could either do this, or crash if the return value of read(/udev/random) isn't the expected length.

Comment on attachment 8482717 [details] [diff] [review] Make a few variables and functions debug-only in /js Review of attachment 8482717 [details] [diff] [review]: ----------------------------------------------------------------- Nice, thank you for doing this!

Comment on attachment 8482718 [details] [diff] [review] Ignore return value of read() in jsmath.cpp Review of attachment 8482718 [details] [diff] [review]: ----------------------------------------------------------------- Yeah, I think this is fine. It certainly doesn't make the situation worse, and looking at this code any longer than I have to isn't good for my sanity.

Thanks, try builds here: https://tbpl.mozilla.org/?tree=Try&rev=dbe338475118

remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/47bdae7e0574 remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/7a91c0ac0c75

https://hg.mozilla.org/mozilla-central/rev/47bdae7e0574 https://hg.mozilla.org/mozilla-central/rev/7a91c0ac0c75