613984 - warning: passing NULL to non-pointer argument 5 of 'bool js::RecordTree' / argument 6 of 'bool js::AttemptToStabilizeTree'

Status: RESOLVED WORKSFORME

(Core :: JavaScript Engine, defect)

Description

[Daniel Holbert [:dholbert]]

Filing bug on these newish GCC warnings in jstracer.cpp:

> js/src/jstracer.cpp: In function ‘js::MonitorResult js::RecordLoopEdge(JSContext*, uintN&)’:
> js/src/jstracer.cpp:7018:74: warning: passing NULL to non-pointer argument 5 of ‘bool js::RecordTree(JSContext*, js::TreeFragment*, JSScript*, jsbytecode*, uint32, js::SlotList*)’
> js/src/jstracer.cpp:7070:73: warning: passing NULL to non-pointer argument 6 of ‘bool js::AttemptToStabilizeTree(JSContext*, JSObject*, js::VMSideExit*, JSScript*, jsbytecode*, uint32)’
> js/src/jstracer.cpp: In function ‘js::TracePointAction js::RecordTracePoint(JSContext*, uintN&, bool*, bool)’:
> js/src/jstracer.cpp:16252:81: warning: passing NULL to non-pointer argument 6 of ‘bool js::AttemptToStabilizeTree(JSContext*, JSObject*, js::VMSideExit*, JSScript*, jsbytecode*, uint32)’
> js/src/jstracer.cpp:16296:68: warning: passing NULL to non-pointer argument 5 of ‘bool js::RecordTree(JSContext*, js::TreeFragment*, JSScript*, jsbytecode*, uint32, js::SlotList*)’

In all these cases, we're passing NULL for the argument "uint32 outerArgc". 

Presumably, we're intending to pass the value 0, not NULL -- right?

Comment 1

[Daniel Holbert [:dholbert]]

HG blame says that each of these lines was added in this changeset for Bug 596364:
http://hg.mozilla.org/mozilla-central/rev/78b0a2bd07fb