protobuf

Commit Graph

Author	SHA1	Message	Date
Eric Salo	3bac8780cb	normalize most of the message accessors: - Rename the accessors from upb_MiniTable_Foo() to upb_Message_Foo() - delete _upb_Message_Clearext() which is now redundant - Allow the getters and setters to accept both extension and non-extension fields - Add a (upb_Arena*) param to setters (only needed for extensions) - Change setters from void to bool (since extensions may require allocations) PiperOrigin-RevId: 493760399	2 years ago
Eric Salo	a07c5d3fe8	snapshot some pending dart work PiperOrigin-RevId: 493073994	2 years ago
Eric Salo	b747edb830	append "ByDef" to names of message accessors that use reflection PiperOrigin-RevId: 493018484	2 years ago
Eric Salo	f458e05718	add some UPB_API / UPB_API_INTERNAL declarations PiperOrigin-RevId: 492551734	2 years ago
Eric Salo	b98c7c7b50	move non-conformance tests and protos into upb/test/ create upb/test/BUILD PiperOrigin-RevId: 492534429	2 years ago
Eric Salo	32f28fcdc6	remove 'platform' param from upb_MiniTable_Build() _upb_MiniTable_Build() is now the general version of the function; upb_MiniTable_Build() calls it and sets the platform to default/native. PiperOrigin-RevId: 491091021	2 years ago
Joshua Haberman	2272970a94	Unified ClearField PiperOrigin-RevId: 490623243	2 years ago
Eric Salo	4f098fd54c	move the extension registry down into mini_table/ Now that :minitable depends on :hash, we may as well put the extreg there also. PiperOrigin-RevId: 490620348	2 years ago
Joshua Haberman	9c6223c058	Unified all hazzers to use MiniTable accessors This required some work to unify map entry messages with regular messages, with respect to presence. Before map entry fields could never have presence. Now they can have presence according to normal rules. Note that this only applies to times that the user constructs a map entry directly. PiperOrigin-RevId: 490611656	2 years ago
Joshua Haberman	e567b7c206	Fixed a bug in GetOrPromoteExtension Adding an extension can invalidate pointers previously returned from upb_Message_FindUnknown(). PiperOrigin-RevId: 490565219	2 years ago
Joshua Haberman	d22d6d71ed	Refactored message accessors to share a common set of functions instead of duplicating logic. Prior to this CL, there were several different code paths for reading/writing message data. Generated code, MiniTable accessors, and reflection all performed direct manipulation of the bits and bytes in a message, but they all had distinct implementations that did not share much of any code. This divergence meant that they could easily have different behavior, bugs could creep into one but not another, and we would need three different sets of tests to get full test coverage. This also made it very difficult to change the internal representation in any way, since it would require updating many places in the code. With this CL, the three different APIs for accessing message data now all share a common set of functions. The common functions all take a `upb_MiniTableField` as the canonical description of a field's type and layout. The lowest-level functions are very branchy, as they must test for every possible variation in the field type (field vs oneof, hasbit vs no-hasbit, different field sizes, whether a nonzero default value exists, extension vs. regular field), however these functions are declared inline and designed to be very optimizable when values are known at compile time. In generated accessors, for example, we can declare constant `upb_MiniTableField` instances so that all values can constant-propagate, and we can get fully specialized code even though we are calling a generic function. On the other hand, when we use the generic functions from reflection, we get runtime branches since values are not known at compile time. But even the function is written to still be as efficient as possible even when used from reflection. For example, we use memcpy() calls with constant length so that the compiler can optimize these into inline loads/stores without having to make an out-of-line call to memcpy(). In this way, this CL should be a benefit to both correctness and performance. It will also make it easier to change the message representation, for example to optimize the encoder by giving hasbits to all fields. Note that we have not completely consolidated all access in this CL: 1. Some functions outside of get/set such as clear and hazzers are not yet unified. 2. The encoder and decoder still touch the message without going through the common functions. The encoder and decoder require a bit more specialized code to get good performance when reading/writing fields en masse. PiperOrigin-RevId: 490016095	2 years ago
Eric Salo	ff6439fba0	move the wire type definitions into upb/wire/ where they belong PiperOrigin-RevId: 489500430	2 years ago
Eric Salo	fb7a67458f	add build targets for :wire and :message and :message_internal PiperOrigin-RevId: 489267642	2 years ago
Eric Salo	27d70edfe2	clean up the :mini_table build target Remove circular dependencies that were bouncing back and forth between msg_internal.h and mini_table/, including: - splitting out each mini table subtype into its own header - moving the non-reflection message code into message/ - moving the accessors from mini_table/ to message/ PiperOrigin-RevId: 489121042	2 years ago

14 Commits (0def88b5829fab20b2064d09752b230bdd068f13)