Open
Bug 1377979
Opened 8 years ago
Updated 7 months ago
Terminology: consider renaming fixed/reserved slots
Categories
(Core :: JavaScript Engine, enhancement, P3)
Core
JavaScript Engine
Tracking
()
NEW
People
(Reporter: sfink, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: triage-deferred)
It is a longstanding nuisance to me that I am completely incapable of remembering what "fixed slots" and "reserved slots" mean. I *just* realized that "fixed" is so named because they live at a fixed offset from the JSObject pointer. Which feels to me like an implication of the main point, which is that they're allocated inline. And "reserved" always made me wonder "reserved for whom?" The JS engine? The embedding?
After discussing on IRC, my current leaning would be
fixed -> inline
reserved -> internal
I know we use "inline" for a number of different things, but I think it always means the same thing.
"internal" doesn't necessarily answer the engine vs embedding question (in fact, it implies that the embedding can *not* use them, if anything), but at least you know that the embedding gets to pass JSCLASS_HAS_INTERNAL_SLOTS(n) and that the distinction between these slots and other slots is whether they're internal or not. (As opposed to whether they are "reserved" or not, which means... something?)
Alternative possibilies for reserved: hidden/unnamed/nameless/anonymous
Sorry for the bikeshed, but I think this has slowed me down an uncomfortable amount over the years.
Updated•7 years ago
|
Keywords: triage-deferred
Priority: -- → P3
Updated•2 years ago
|
Severity: normal → S3
Updated•7 months ago
|
Blocks: sm-runtime
Updated•7 months ago
|
Severity: S3 → N/A
You need to log in
before you can comment on or make changes to this bug.
Description
•