archives

« Bugzilla Issues Index

#2707 — chapter 26: misc editorial


----------------------------------------
In 26.2.1.1 "Reflect.Realm ( [ options [ , initializer ] ] )":

{1}
26.2.1.1 / step 29.b:
Let /status/ be the result of calling the [[Call]] internal method ..., passing
/realmObject/ as the *this* value and /builtins/ as the single argument.
'builtins' is not defined

----------------------------------------
In 26.2.3.2 "Reflect. Realm.prototype.eval ( source )":

{2}
26.2.3.2 / header:
Reflect. Realm.prototype.eval ( source )
Delete space after "Reflect."

{3}
26.2.3.2 / para 1:
When Reflect.Realm.prototype.eval is called with arguent /source/ ...
s|arguent|argument|

----------------------------------------
In 26.2.3.4 "get Reflect.Realm.prototype.intrinsics":

{4}
26.2.3.4 / step 6.3:
.Perfrom CreateDataProperty(/table/, /key/, /object/).
Delete initial dot.
s|Perfrom|Perform|

----------------------------------------
In 26.2.3.5 "get Reflect.Realm.prototype.stdlib":

{5}
26.2.3.5 / step 6:
For each property of the Global Object specified in clause 18, , do
Delete extra comma.

{6}
26.2.3.5 / step 6.2:
For propties whose values are functions, ...
s|propties|properties|

{7}
26.2.3.5 / step 6.2:
... the value the of the [[Value]] attribute ...
s|the of|of|

{8}
26.2.3.5 / step 6.3:
Let /status/ be DefinePropertyOrTrhow(/props/, /name/, /desc/).
s|Trhow|Throw|

----------------------------------------
In 26.2.3.6 "Reflect.Realm.prototype [ @@toStringTag ]":

{9}
26.2.3.6 / para 1:
The initial value of the @@toStringTag property is the string value *"* Reflect.Realm".
Put 'Reflect' and the preceding quote in monospace.
Delete the space between them.

----------------------------------------
In 26.2.3.7 "Realm Subclass Extension Properties":

{10-14}
26.2.3.7.*:
In 26.2.3.7.*,
-- All preambles think they're defining ".eval".
-- All preamble say "arguent" (-> "argument")
-- All algorithms end with a call to "IndirectEval".

Plus the headers for .2 and .4 both say "indirectEval".

----------------------------------------
In 26.3.3.3 "Loader.prototype.delete ( name )":

{15}
26.3.3.3 / header:
Loader.prototype.delete ( name )
Insert "Reflect." before "Loader"

----------------------------------------
In 26.3.3.6 "get Loader.prototype.global":

{16}
26.3.3.6 / header:
26.3.3.6 get Loader.prototype.global
Insert "Reflect." before "Loader"

{17}
26.3.3.6 / para 1:
Loader.prototype.global is an accessor property ...
Insert "Reflect." before "Loader"

{18}
26.3.3.6 / algorithm
The steps appear as a bulleted list, should be a numbered list.

----------------------------------------
In 26.3.3.12 "Reflect.Loader.prototype.newModule ( obj )":

{19}
26.3.3.12 / step 4.1:
For each key in keys, do
This should presumably be a top-level step.

----------------------------------------
In 26.3.3.13 "get Reflect.Loader.prototype.realm":

{20}
26.3.3.13 / para 1:
Loader.prototype.realm is an accessor property ...
Insert "Reflect." before "Loader"

----------------------------------------
In 26.3.3.17 "Reflect.Loader.prototype [ @@toStringTag ]":

{21}
26.3.3.17 / para 1:
The initial value of the @@toStringTag property is the string value
*"Reflect.Loader"*.
"Reflect.Loader" is in sans-serif bold, should be in monospace

XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX


Bug 2686 came in while I was preparing my comments for submission. I should have gone through and eliminated my duplicates, but didn't, sorry.


fixed in rev24 editor's draft


confirmed fixed


Closing per comments 2 and 3.