Address reviewers' comments

gpuweb · kainino0x · Jun 18, 2024 · May 6, 2024 · May 6, 2024 · May 7, 2024
commit 4ca64525cac8ef056ba96dce47f4b22afb5bcaee
diff --git a/spec/index.bs b/spec/index.bs
@@ -8148,11 +8148,11 @@ dictionary GPUFragmentState
                             uses the alpha channel of the first input of the corresponding blend unit (is any of
                             {{GPUBlendFactor/"src-alpha"}}, {{GPUBlendFactor/"one-minus-src-alpha"}} or
                             {{GPUBlendFactor/"src-alpha-saturated"}})
-                        - The [=blend_src=] attribute of |output| is 1
+                        - The [=blend_src=] attribute of |output| is 1 and
                             |colorState|.{{GPUColorTargetState/blend}}.{{GPUBlendState/color}}.{{GPUBlendComponent/srcFactor}} or
                             |colorState|.{{GPUColorTargetState/blend}}.{{GPUBlendState/color}}.{{GPUBlendComponent/dstFactor}}
                             uses the alpha channel of the second input of the corresponding blending unit (is any of
-                            {{GPUBlendFactor/"src1-alpha"}} or {{GPUBlendFactor/"one-minus-src1-alpha"}}), then:
+                            {{GPUBlendFactor/"src1-alpha"}} or {{GPUBlendFactor/"one-minus-src1-alpha"}})
 
                         Then:
 

diff --git a/wgsl/index.bs b/wgsl/index.bs
@@ -8635,7 +8635,7 @@ path: syntax/blend_src_attr.syntax.bs.include
 
   <tr>
     <td>Parameters
-    <td>[=shader-creation error|Must=] be `0` or `1`.
+    <td>[=shader-creation error|Must=] be a [=const-expression=] that [=resolves=] to an i32 or u32 with value of `0` or `1`.
 
 </table>
 
@@ -9422,33 +9422,30 @@ Each input-output location can store a value up to 16 bytes in size.
 The byte size of a type is defined using the *SizeOf* column in [[#alignment-and-size]].
 For example, a four-component vector of floating-point values occupies a single location.
 
-IO locations are specified via the [=attribute/location=] attribute. The IO locations of
-[=fragment=] shader [=user-defined output datum|output=]s can also be specified via the both
-[=attribute/location=] attribute and optional [=attribute/blend_src=] attribute when the feature
-[=extension/dual_source_blending=] is enabled. The [=attribute/blend_src=] attribute can only be
-specified on a [=fragment=] shader [=user-defined output datum|output=] and used to specify up to
-two [=fragment=] shader [=user-defined output datum|output=]s for a given color attachment as the
-inputs to the blend equation. By default the value of the [=attribute/blend_src=] attribute for a
-[=fragment=] shader [=user-defined output datum|output=] is `0` when the [=attribute/blend_src=]
-attribute is not specified in the shader.
-
 Each user-defined [=user-defined input datum|input=] and [=user-defined output datum|output=] [=shader-creation error|must=] have an explicitly specified IO location.
 Each structure member in the entry point IO [=shader-creation error|must=] be one of either a built-in value
 (see [[#builtin-inputs-outputs]]), or assigned a location.
 
+A location can be specified via either the [=attribute/location=] attribute only or both the
+[=attribute/location=] attribute and the [=attribute/blend_src=] attribute. The optional
+[=attribute/blend_src=] attribute can only be specified on a [=fragment=] shader
+[=user-defined output datum|output=] and used to specify up to two [=fragment=] shader
+[=user-defined output datum|outputs=] for the only color attachment as the inputs to the blend
+equation.
+
 Locations [=shader-creation error|must not=] overlap within each of the following sets:
 * Members within a structure type.
     This applies to any structure, not just those used in shader stage inputs or outputs.
 * An entry point's shader stage inputs,
     i.e. locations for its formal parameters, or for the members of its formal parameters of structure type.
 
-When [=attribute/blend_src=] attribute is specified:
+When the [=attribute/blend_src=] attribute is specified:
 * There [=shader-creation error|must=] be exactly `2` [=fragment=] shader
-    [=user-defined output datum|output=]s.
+    [=user-defined output datum|outputs=].
 * The [=attribute/location=] attribute of all [=fragment=] shader
-    [=user-defined output datum|output=]s [=shader-creation error|must=] be `0`.
+    [=user-defined output datum|outputs=] [=shader-creation error|must=] be `0`.
 * The value of [=attribute/blend_src=] attribute [=shader-creation error|must=] be `0` for one
-    [=fragment=] shader [=user-defined output datum|output=], and `1` for another [=fragment=]
+    [=fragment=] shader [=user-defined output datum|output=], and `1` for the other [=fragment=]
     shader [=user-defined output datum|output=].
 
 Note: Location numbering is distinct between inputs and outputs: