Add [Clamp] to maxAnisotropy

kainino0x · kainino0x · commit 6088af821415 · 2020-12-07T15:03:09.000-08:00
The default behavior (`value % 65536`) doesn't make any sense here. We could use [EnforceRange], but [Clamp] makes sense: - Any value of maxAnisotropy > 65535 behaves identically to 65535 (that is, it's as if there's no maximum). - Any value of maxAnisotropy < 0 behaves identically to 0 (it's a validation error). Fixes #1269.
diff --git a/spec/index.bs b/spec/index.bs
@@ -2286,7 +2286,7 @@ dictionary GPUSamplerDescriptor : GPUObjectDescriptorBase {
     float lodMinClamp = 0;
     float lodMaxClamp = 0xffffffff; // TODO: What should this be? Was Number.MAX_VALUE.
     GPUCompareFunction compare;
-    unsigned short maxAnisotropy = 1;
+    [Clamp] unsigned short maxAnisotropy = 1;
 };
 </script>
 
@@ -2303,6 +2303,7 @@ dictionary GPUSamplerDescriptor : GPUObjectDescriptorBase {
 - If {{GPUSamplerDescriptor/compare}} is provided, the sampler will be a comparison sampler with the specified
     {{GPUCompareFunction}}.
 - {{GPUSamplerDescriptor/maxAnisotropy}} specifies the maximum anisotropy value clamp used by the sampler.
+    - It must be &ge; 1.
 
     Note: most implementations support {{GPUSamplerDescriptor/maxAnisotropy}} values in range between 1 and 16, inclusive.
 
