Commit 032e628
committed
fix: invalid syntax in descriptor.proto
A recent change (protobufjs#2075) edited descriptor.proto to include a `reserved`
statement that combines a field number and field name in the same line.
This is specifically called out in the proto language guide as invalid.
https://protobuf.dev/programming-guides/editions/#reserved
> Note that you can’t mix field names and numeric values in the same
> `reserved` statement.
Protobuf.js itself seems to handle this syntax just fine, but other
tools like protoc will choke on it.1 parent f42297b commit 032e628
2 files changed
+4
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
588 | 588 | | |
589 | 589 | | |
590 | 590 | | |
591 | | - | |
592 | 591 | | |
593 | 592 | | |
594 | 593 | | |
595 | | - | |
| 594 | + | |
| 595 | + | |
596 | 596 | | |
597 | 597 | | |
598 | 598 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
229 | 229 | | |
230 | 230 | | |
231 | 231 | | |
232 | | - | |
| 232 | + | |
| 233 | + | |
233 | 234 | | |
234 | 235 | | |
235 | 236 | | |
| |||
0 commit comments