commit | 6e25d8c6a6cdd072e374d152a81c0b04b43ec8c1 | [log] [tgz] |
---|---|---|
author | Tuo Shan <shantuo@google.com> | Thu Aug 22 18:52:43 2019 -0700 |
committer | Tuo Shan <shantuo@google.com> | Fri Aug 23 21:35:40 2019 +0000 |
tree | 421bfcbfc2aca59b7cf6190f3f6030dc16e53f3c | |
parent | 08ff73004835d46c133844f593fe74596d3e8194 [diff] |
proto: add tests for unmarshalling invalid field numbers This change adds tests for unmarshalling fields with various invalid field numbers. Our current behavior is that proto.Unmarshal will return an error when it sees zero and larger than max field numbers and return nil for reserved ones, which matches the C++ behavior. (Note: depending on which parser helper in the C++ implementation, one may need to call additional method to check the result, which we don't have in Go) Change-Id: I8791fd077f25656107556f5606d55d05c1b4a120 Reviewed-on: https://go-review.googlesource.com/c/protobuf/+/191459 Reviewed-by: Joe Tsai <thebrokentoaster@gmail.com>
WARNING: This repository is in active development. There are no guarantees about API stability. Breaking changes will occur until a stable release is made and announced.
This repository is for the development of the next major Go implementation of protocol buffers. This library makes breaking API changes relative to the existing Go protobuf library. Of particular note, this API aims to make protobuf reflection a first-class feature of the API and implements the protobuf ecosystem in terms of reflection.
List of relevant design documents:
We appreciate community contributions. See CONTRIBUTING.md.
Issues regarding the new API can be filed at github.com/golang/protobuf. Please use a APIv2:
prefix in the title to make it clear that the issue is regarding the new API work.