archives

« Bugzilla Issues Index

#2545 — chapter 26: misc editorial


----------------------------------------
In 26.2.1.1 "%Realm% (options = { }, initialiser = undefined)":

26.2.1.1 / step 29:
If /initializer/ is not undefined, then
s|initializer|initialiser|
Put "undefined" in bold.

26.2.1.1 / step 29.a:
If IsCallable(/initializer/) is *false*, throw a *TypeError* exception.
s|initializer|initialiser|

26.2.1.1 / step 29.b:
... the [[Call]] internal method of the /initializer/ function, ...
s|initializer|initialiser|

----------------------------------------
In 26.2.3 "Properties of the %Realm% Prototype Object":

26.2.3 / header + para 1:
s|%Realm% [Pp]rototype|%RealmPrototype%| ?

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

26.2.3.2 / step 2:
... or /realmObject/ does not have [[Realm]] internal slot, ...
After "have", insert "a".

26.2.3.2 / step 5:
Return the result of IndirectEval(/realm/, /source/).
IndirectEval is not defined.

----------------------------------------
In 26.2.3.3 "get Realm.prototype.global":

26.2.3.3 / header:
s|Realm|%Realm%|

26.2.3.3 / step 2:
... or /realmObject/ does not have [[Realm]] internal slot, ...
After "have", insert "a".

----------------------------------------
In 26.2.4 "Properties of %Realm% Instances":

26.2.4 / para 1:
... that inherit properties from the %Realm% prototype object. ...
s|%Realm% prototype|%RealmPrototype%| ?

----------------------------------------
In 26.3.1.1 "%Loader% (options = { })":

26.3.1.1 / step 8.a:
... or /realmObject/ does not have [[RealmRecord]] internal slot, ...
After "have", insert "a".

26.3.1.1 / step 13:
Set /loader/.[[LoaderRecord]] to /loaderRecord/.
Change
/loader/.[[LoaderRecord]]
to
/loader/'s [[LoaderRecord]] internal slot

----------------------------------------
In 26.3.3 "Properties of the %Loader% Prototype Object":

26.3.3 / header + para 1:
s|%Loader% [Pp]rototype|%LoaderPrototype%| ?

26.3.3 / para 1:
... It does not have [[LoaderRecord]] internal slot.
After "have", insert "a".

----------------------------------------
In 26.3.3.1 "%Loader%.prototype.constructor":

26.3.3.1:
The initial value of %Loader.prototype.constructor is ...
After "%Loader", add "%".

----------------------------------------
In 26.3.3.2 "%Loader%.prototype.define ( name, source, options = undefined )":

26.3.3.2 / para 1:
'define' returns a Promise object that resolves to undefined when ...
Change "undefined" from italic to serif-bold.

26.3.3.2 / step 10:
If /metadata/ is /undefined/ then ...
Change "undefined" from italic to serif-bold.

26.3.3.2 / step 13:
Let /G/ be a new function as defined by ReturnUndefined.
ReturnUndefined is not defined.

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

26.3.3.3 / step 6:
Let /modules/ be the value of of /loaderRecord/. [[Modules]],
Delete extra "of".

Delete the space between dot and "[["

----------------------------------------
In 26.3.3.5 "Loader.prototype.eval ( source )":

26.3.3.5 / step 4:
Return the result IndirectEval(/loaderRecords/.[[ Realm]], /source/).
After "result", insert "of"

IndirectEval is not defined

Drop "s" from "loaderRecords"

Delete the space before "Realm".

----------------------------------------
In 26.3.3.6 "Loader.prototype.get ( name )":

26.3.3.6 / step 6:
Let /modules/ be the value of of /loaderRecord/.[[ Modules]],
Delete extra "of".

Delete space before "Modules".

----------------------------------------
In 26.3.3.8 "Loader.prototype.has ( name )":

26.3.3.8 / step 6:
Let /modules/ be the value of of /loaderRecord/.[[ Modules]],
Delete extra "of".

Delete space before "Modules".

----------------------------------------
In 26.3.3.12 "Loader.prototype.module ( source, options )":

26.3.3.12 / step 7:
Set /load/.[[Address]] field of /load/ to /address/.
Delete "field of /load/"

----------------------------------------
In 26.3.3.13 "newModule ( obj )":

26.3.3.13 / step 1:
If Type(/obj/) is not Object, throw a TypeError exception.
Put "TypeError" in bold.

26.3.3.13 / step 2:
Let /mod/ be CreateLinkedModuleInstance( )
CreateLinkedModuleInstance is not defined.

26.3.3.13 / step 3:
Let keys be the result of calling the ObjectKeys abstract operation ...
ObjectLeys is not defined.

26.3.3.13 / step 4.a:
For each key in keys, do
This should be out-dented and made step 5.

26.3.3.13 / step 4.d:
Let /F/ be the result of calling CreateConstantGetter(/key/, /value/).
CreateConstantGetter is not defined

----------------------------------------
In 26.3.3.14 "get Loader.prototype.realm":

26.3.3.14 / step 4:
Return RealmObjectFor(/loaderRecord/.[[Realm]]).
RealmObjectFor is not defined

----------------------------------------
In 26.3.3.15 "Loader.prototype.set ( name, module )":

26.3.3.15 / step 7:
Let /modules/ be the value of of /loaderRecord/.[[ Modules]],
Delete extra "of".

Delete space before "Modules".

26.3.3.15 / step 10:
Append /p/ as the last record of /loaderRecord/.[[ Modules]].
Delete space before "Modules".

----------------------------------------
In 26.3.3.18 "Loader Pipeline Hook Properties":

26.3.3.18 / para 1:
The %Loader% prototype provide default implementations for the hook methods.
s|%Loader% prototype|%LoaderPrototype%| ?

s|provide|provides|

----------------------------------------
In 26.3.3.18.3 "Loader.prototype.fetch (loadRequest)":

26.3.3.18.3 / step 1:
Throw a TypeError exception.
Put "TypeError" in bold

----------------------------------------
In 26.3.5.2.1 "%LoaderIteratorPrototype%.next( )":

26.3.5.2.1 / step 9:
... the value of the /loaderRecord/.[[Modules]] internal slot of /m/.
Change to just
... the value of /loaderRecord/.[[Modules]].

----------------------------------------
In 26.5.1.1 "Proxy (target, handler )":

26.5.1.1 / step 1:
Throw a TypeError exception.
Put "TypeError" in bold.

XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX


Some of these were fixed in rev23, and some of the others have moved.
Here's what's left in rev24:

----------------------------------------
In 26.2.1.1 "Reflect.Realm (options = { }, initialiser = undefined)":

{2}
26.2.1.1 / step 28:
If /initializer/ is not undefined, then
Put "undefined" in bold.

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

{7}
26.2.3.2 / step 2:
... or /realmObject/ does not have [[Realm]] internal slot, ...
After "have", insert "a".

{8}
26.2.3.2 / step 5:
Return the result of IndirectEval(/realm/, /source/).
IndirectEval is not defined.

----------------------------------------
In 26.2.3.3 "get Realm.prototype.global":

{10}
26.2.3.3 / step 2:
... or /realmObject/ does not have [[Realm]] internal slot, ...
After "have", insert "a".

----------------------------------------
In 26.3.1.1 "Reflect.Loader (options = { })":

{12}
26.3.1.1 / step 8.a:
... or /realmObject/ does not have [[RealmRecord]] internal slot, ...
After "have", insert "a".

{13}
26.3.1.1 / step 13:
Set /loader/.[[LoaderRecord]] to /loaderRecord/.
Change
/loader/.[[LoaderRecord]]
to
/loader/'s [[LoaderRecord]] internal slot

----------------------------------------
In 26.3.3 "Properties of the Reflect.Loader Prototype Object":

{15}
26.3.3 / para 1:
... It does not have [[LoaderRecord]] internal slot.
After "have", insert "a".

----------------------------------------
In 26.3.3.2 "Reflect.Loader.prototype.define ( ... )":

{17}
26.3.3.2 / para 1:
'define' returns a Promise object that resolves to undefined when ...
Change "undefined" from italic to serif-bold.

{18}
26.3.3.2 / step 10:
If /metadata/ is /undefined/ then ...
Change "undefined" from italic to serif-bold.

{19}
26.3.3.2 / step 13:
Let /G/ be a new function as defined by ReturnUndefined.
ReturnUndefined is not defined.

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

{20,21}
26.3.3.3 / step 6:
Let /modules/ be the value of of /loaderRecord/. [[Modules]],
Delete extra "of".

Delete the space between dot and "[["

----------------------------------------
In 26.3.3.5 "Reflect.Loader.prototype.get ( name )":

{26,27}
26.3.3.5 / step 6:
Let /modules/ be the value of of /loaderRecord/.[[ Modules]],
Delete extra "of".

Delete space before "Modules".

----------------------------------------
In 26.3.3.7 "Reflect.Loader.prototype.has ( name )":

{28,29}
26.3.3.7 / step 6:
Let /modules/ be the value of of /loaderRecord/.[[ Modules]],
Delete extra "of".

Delete space before "Modules".

----------------------------------------
In 26.3.3.11 "Reflect.Loader.prototype.module ( source, options )":

{30}
26.3.3.11 / step 7:
Set /load/.[[Address]] field of /load/ to /address/.
Delete "field of /load/"

----------------------------------------
In 26.3.3.12 "newModule ( obj )":

{31}
26.3.3.12 / step 1:
If Type(/obj/) is not Object, throw a TypeError exception.
Put "TypeError" in bold.

{32}
26.3.3.12 / step 2:
Let /mod/ be CreateLinkedModuleInstance( )
CreateLinkedModuleInstance is not defined.

{33}
26.3.3.12 / step 3:
Let keys be the result of calling the ObjectKeys abstract operation ...
ObjectKeys is not defined.


{35}
26.3.3.12 / step 4.d:
Let /F/ be the result of calling CreateConstantGetter(/key/, /value/).
CreateConstantGetter is not defined

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

{36}
26.3.3.13 / step 4:
Return RealmObjectFor(/loaderRecord/.[[Realm]]).
RealmObjectFor is not defined

----------------------------------------
In 26.3.3.14 "Reflect.Loader.prototype.set ( name, module )":

{37,38}
26.3.3.14 / step 7:
Let /modules/ be the value of of /loaderRecord/.[[ Modules]],
Delete extra "of".

Delete space before "Modules".

{39}
26.3.3.14 / step 10:
Append /p/ as the last record of /loaderRecord/.[[ Modules]].
Delete space before "Modules".

----------------------------------------
In 26.3.3.18 "Reflect.Loader Pipeline Hook Properties":

{40,41}
26.3.3.18 / para 1:
The %Loader% prototype provide default implementations for the hook methods.
s|%Loader% prototype|Reflect.Loader prototype| ?

s|provide|provides|

----------------------------------------
In 26.3.3.18.3 "Reflect.Loader.prototype.fetch (loadRequest)":

{42}
26.3.3.18.3 / step 1:
Throw a TypeError exception.
Put "TypeError" in bold

----------------------------------------
In 26.3.5.2.1 "%LoaderIteratorPrototype%.next( )":

{43}
26.3.5.2.1 / step 9:
... the value of the /loaderRecord/.[[Modules]] internal slot of /m/.
Change to just
... the value of /loaderRecord/.[[Modules]].


XXX


fixed in rev 25 editor's draft

except TODO's for undefined abstract operations is now bug 2852


confirmed fixed except for:
{8}, raised independently as Bug 2820,
{35}, added to Bug 2852,
{40}, re-raised as Bug 2966, and
{43}, re-raised as Bug 2967.