a2e5ab82c7
Change all required proto fields to optional. While the proto file is supposed to be a backwards compatible API, and fields are not supposed to be deprecated, this commit will allow for that option if needed. Implementation wise this change doesn't matter much: any parsed data needs additional verification outside what the protobuf crate's parser provides anyway, so adding checks to verify that all required fields, even though marked optional in the proto file, were found is a minor increase in code complexity. If in the future a proto field should no longer be used: - keep the field in the proto, still marked optional and clearly document that it is no longer in use - change protos.rs from checking struct.has_field() to explicitly dropping any value via struct.clear_field() Bug: 286337317 Test: atest aconfig.test Change-Id: Iad1ccfe50ecac286ff7a796aec909bec70b9520d |
||
---|---|---|
.. | ||
protos | ||
src | ||
templates | ||
tests | ||
.gitignore | ||
Android.bp | ||
build.rs | ||
Cargo.toml | ||
MODULE_LICENSE_APACHE2 | ||
OWNERS | ||
PREUPLOAD.cfg | ||
rustfmt.toml |