pyrobuf icon indicating copy to clipboard operation
pyrobuf copied to clipboard

Implement rudimentary "reserved" field support

Open thirtythreeforty opened this issue 5 years ago • 1 comments

This implementation does not validate that the field names and/or numbers are not reused; it simply ignores them. This allows parsing files that contain the reserved keyword.

thirtythreeforty avatar Feb 20 '20 16:02 thirtythreeforty

Thanks! Looks pretty good but...

  1. We should add a test
  2. I think this only partially works, based on the example in the language guide:
message Foo {
  reserved 2, 15, 9 to 11;
  reserved "foo", "bar";
}

As is, something like reserved 2, 15 will work, but this will fail for 9 to 11 or "foo", "bar".

tburmeister avatar Mar 21 '20 20:03 tburmeister