webgpu.h: Reserve 0 for enums, in non-breaking cases
These are the ones that I think these should be safe to land without
breaking anything:
- Output enums
- Enums which are only used as function arguments (not struct members)
Also aligns some formatting and makes some enum ordering consistent with
the spec.
Bug: dawn:2224
Change-Id: I17cc2f3105f234d7919724e5e9f01d71b48b2818
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/166387
Kokoro: Kokoro <noreply+kokoro@google.com>
Reviewed-by: Loko Kung <lokokung@google.com>
Commit-Queue: Loko Kung <lokokung@google.com>
Auto-Submit: Kai Ninomiya <kainino@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
diff --git a/dawn.json b/dawn.json
index f08fa72..2123021 100644
--- a/dawn.json
+++ b/dawn.json
@@ -173,10 +173,10 @@
"category": "enum",
"emscripten_no_enum_table": true,
"values": [
- {"value": 0, "name": "discrete GPU"},
- {"value": 1, "name": "integrated GPU"},
- {"value": 2, "name": "CPU"},
- {"value": 3, "name": "unknown"}
+ {"value": 1, "name": "discrete GPU"},
+ {"value": 2, "name": "integrated GPU"},
+ {"value": 3, "name": "CPU"},
+ {"value": 4, "name": "unknown"}
]
},
"device descriptor": {
@@ -617,9 +617,9 @@
"buffer map state": {
"category": "enum",
"values": [
- {"value": 0, "name": "unmapped"},
- {"value": 1, "name": "pending"},
- {"value": 2, "name": "mapped"}
+ {"value": 1, "name": "unmapped"},
+ {"value": 2, "name": "pending"},
+ {"value": 3, "name": "mapped"}
]
},
"buffer usage": {
@@ -827,11 +827,11 @@
{"value": 0, "name": "undefined", "jsrepr": "undefined", "valid": false},
{"value": 1, "name": "never"},
{"value": 2, "name": "less"},
- {"value": 3, "name": "less equal"},
- {"value": 4, "name": "greater"},
- {"value": 5, "name": "greater equal"},
- {"value": 6, "name": "equal"},
- {"value": 7, "name": "not equal"},
+ {"value": 3, "name": "equal"},
+ {"value": 4, "name": "less equal"},
+ {"value": 5, "name": "greater"},
+ {"value": 6, "name": "not equal"},
+ {"value": 7, "name": "greater equal"},
{"value": 8, "name": "always"}
]
},
@@ -879,9 +879,9 @@
"category": "enum",
"emscripten_no_enum_table": true,
"values": [
- {"value": 0, "name": "error"},
- {"value": 1, "name": "warning"},
- {"value": 2, "name": "info"}
+ {"value": 1, "name": "error"},
+ {"value": 2, "name": "warning"},
+ {"value": 3, "name": "info"}
]
},
"compute pass descriptor": {
@@ -1468,9 +1468,9 @@
"error filter": {
"category": "enum",
"values": [
- {"value": 0, "name": "validation"},
- {"value": 1, "name": "out of memory"},
- {"value": 2, "name": "internal"}
+ {"value": 1, "name": "validation"},
+ {"value": 2, "name": "out of memory"},
+ {"value": 3, "name": "internal"}
]
},
"error type": {
@@ -1489,10 +1489,10 @@
"category": "enum",
"tags": ["dawn"],
"values": [
- {"value": 0, "name": "verbose"},
- {"value": 1, "name": "info"},
- {"value": 2, "name": "warning"},
- {"value": 3, "name": "error"}
+ {"value": 1, "name": "verbose"},
+ {"value": 2, "name": "info"},
+ {"value": 3, "name": "warning"},
+ {"value": 4, "name": "error"}
]
},
"extent 2D": {
@@ -2087,9 +2087,9 @@
"category": "enum",
"emscripten_no_enum_table": true,
"values": [
- {"name": "wait any only", "value": 0},
- {"name": "allow process events", "value": 1},
- {"name": "allow spontaneous", "value": 2}
+ {"value": 0, "name": "wait any only"},
+ {"value": 1, "name": "allow process events"},
+ {"value": 2, "name": "allow spontaneous"}
]
},
"future": {
@@ -2103,12 +2103,12 @@
"_comment": "TODO(crbug.com/dawn/2053): This could possibly be [[nodiscard]].",
"emscripten_no_enum_table": true,
"values": [
- {"name": "success", "value": 0},
- {"name": "timed out", "value": 1},
- {"name": "unsupported timeout", "value": 2},
- {"name": "unsupported count", "value": 3},
- {"name": "unsupported mixed sources", "value": 4},
- {"name": "unknown", "value": 5}
+ {"value": 0, "name": "success"},
+ {"value": 1, "name": "timed out"},
+ {"value": 2, "name": "unsupported timeout"},
+ {"value": 3, "name": "unsupported count"},
+ {"value": 4, "name": "unsupported mixed sources"},
+ {"value": 5, "name": "unknown"}
]
},
"future wait info": {
@@ -3500,7 +3500,7 @@
"texture format": {
"category": "enum",
"values": [
- {"value": 0, "name": "undefined", "valid": false, "jsrepr": "undefined"},
+ {"value": 0, "name": "undefined", "jsrepr": "undefined", "valid": false},
{"value": 1, "name": "R8 unorm"},
{"value": 2, "name": "R8 snorm"},
@@ -3660,7 +3660,7 @@
"texture view dimension": {
"category": "enum",
"values": [
- {"value": 0, "name": "undefined", "valid": false, "jsrepr": "undefined"},
+ {"value": 0, "name": "undefined", "jsrepr": "undefined", "valid": false},
{"value": 1, "name": "1D"},
{"value": 2, "name": "2D"},
{"value": 3, "name": "2D array"},
@@ -3672,7 +3672,7 @@
"vertex format": {
"category": "enum",
"values": [
- {"value": 0, "name": "undefined", "valid": false, "jsrepr": "undefined"},
+ {"value": 0, "name": "undefined", "jsrepr": "undefined", "valid": false},
{"value": 1, "name": "uint8x2"},
{"value": 2, "name": "uint8x4"},
{"value": 3, "name": "sint8x2"},
