----------------------------------------
In 8.1.1.4.8 "HasThisBinding ()":
HasSuperBinding ()
This should be a new section, not a part of 8.1.1.4.8.
----------------------------------------
In 8.2.1 "CreateRealm ( )":
8.2.1 / step 3:
Set /realmRec/.[[intrinsics]] be /intrinsics/.
s|be|to|
8.2.1 / step 6:
Set /realmRec/.[[globalThis]] be /newGlobal/.
s|be|to|
8.2.1 / step 7:
Let /newGlobalEnv/ be NewGlobalEnvironment(/newGlobal/, /intrinsics/).
NewGlobalEnvironment is not defined.
8.2.1 / step 8:
Set /realmRec/.[[globalEnv]] be /newGlobalEnv/.
s|be|to|
----------------------------------------
In 8.3.1 "ResolveBinding(name)":
8.3.1 / header:
Append "Abstract Operation" to header
8.3.1 / step 3:
Return the result of calling GetIdentifierReference abstract operation ...
After "calling", insert "the".
----------------------------------------
In 8.3.2 "GetThisEnvironment":
In 8.3.3 "ResolveThisBinding":
In 8.3.4 "GetGlobalObject":
Add "()" to the section title, so it's clearer that the operation doesn't
have any parameters.
----------------------------------------
In 8.4 "Tasks and Task Queues":
8.4 / para 4:
... and starts execution of associated Task abstraction operation.
s|of|of the|
s|abstraction|abstract|
----------------------------------------
In 8.4.1 "EnqueueTask ( queueName, task, arguments) Abstract Operation":
8.4.1 / step 3:
Assert: /arguments/ is a List whose size is the same as ...
I think this is the only point where the spec says that a List has a 'size'.
Change "whose size is" to "whose number of elements is" ?
8.4.1 / step 5:
Let /callerReam/ be /callerContext/'s Realm.
s|callerReam|callerRealm|
----------------------------------------
In 8.4.2 "NextTask ( result ) Algorithm Step":
8.4.2 / header
Rather than saying 'NextTask' is an "Algorithm Step",
why not define it as an abstract operation?
8.4.2 / para 2
Is used in Task abstract operation in place of:
s|Is|is|
s|operation|operations|
8.4.2 / para 3:
Task abstraction operations must not contain a Return step ...
s|abstraction|abstract|
8.4.2 / step 4:
Let /nextQueue/ be a non-emptry Task Queue ...
s|non-emptry|non-empty|
8.4.2 / step 6:
Let /newContext/ be a new exeution context.
s|exeution|execution|
fixed in rev25 editor's draft
re: section heading for NextTask. This is most like 6.2.2.4 ReturnIfAbrupt, so used the same formatting.
> fixed in rev25 editor's draft
confirmed fixed except for what I re-raised as Bug 2955
> re: section heading for NextTask. This is most like 6.2.2.4 ReturnIfAbrupt, so
> used the same formatting.
Except that it *doesn't* use the same formatting. ReturnIfAbrupt's header doesn't mention its parameter, and invocations of ReturnIfAbrupt parenthesize the argument.
header: 6.2.2.4 ReturnIfAbrupt 8.4.2 NextTask result
invocation: ReturnIfAbrupt(argument) NextTask status