Open Bug 1470909 Opened 7 years ago Updated 9 months ago

[meta] Remove all use of C-style varargs from SpiderMonkey (except in SpiderMonkey developer C++ debugging functions, e.g. for dumping stuff to stderr)

Categories

(Core :: JavaScript Engine, enhancement, P3)

Product:
Core
Component:
JavaScript Engine
Type:
enhancement

Tracking

()

Tracking Flags:
Tracking Status
firefox62 --- affected

People

(Reporter: Waldo, Unassigned)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

(Keywords: meta)

C-style varargs using va_list and va_start and such are basically an unchecked (and runtime-uncheckable) hole in the type system. C++ offers type-safe ways to achieve the same ends; we should use them. (Variadic templates if absolutely necessary, but more often pointer+length combinations with bounds-checking at runtime are probably better.) This is a metabug filed to get this into the system -- hang work off it in separate bugs. (I'll file one to change the error reporting system, as the most troublesome abuser I'm aware of.)
Depends on: 1470911
Priority: -- → P3
Summary: Remove all use of C-style varargs from SpiderMonkey (except in SpiderMonkey developer C++ debugging functions, e.g. for dumping stuff to stderr) → [meta] Remove all use of C-style varargs from SpiderMonkey (except in SpiderMonkey developer C++ debugging functions, e.g. for dumping stuff to stderr)
Severity: normal → S3
No longer blocks: sm-meta
You need to log in before you can comment on or make changes to this bug.