protobuf

Commit Graph

Author	SHA1	Message	Date
Jakob Buchgraber	8cdc700b5b	Implement ctype=CORD for singular string & byte fields - We introduce two new view types ProtoStringCow and ProtoBytesCow. - In UPB, for cord field accessors we always return a Cow::Borrowed. - In C++, for coed field accessors we check if the underlying absl::Cord is flat (contigous) and if so return a Cow::Borrowed. If it's not flat we copy the data to a ProtoString and return a Cow::Owned. - We expect the absl::Cord to be flat almost all the time. We have experimentally verified that for small strings (<4 KiB) and less than 6 appends the cord is in fact flat [1]. - This change lifts the requirement of all ViewProxy types to be Copy. Our Cow types cannot be Copy because the owned types aren't copy. [1] https://source.corp.google.com/piper///depot/google3/experimental/users/buchgr/cords/cords.cc PiperOrigin-RevId: 655485943	4 months ago
Dmitri Gribenko	0a917b95f2	Use `#[googletest::test]` in Protobuf Rust PiperOrigin-RevId: 654726974	4 months ago
Protobuf Team Bot	f9dd9ce66e	UPB text encoder without using reflection for Rust (used for a message's Debug trait) that will print out field number to value entries instead of field name to value entries of a message like how it's expected for the usual text format using reflection. General test for it is done in Rust, and then extensions are tested in UPB as they're not currently supported in Rust-upb. PiperOrigin-RevId: 651113583	5 months ago
Jakob Buchgraber	0d6e9794d1	Migrate Repeated::{push, set} and Map::insert to use the IntoProxied trait. * The public Repeated::{push, set} and Map::insert methods now accept any value that implements IntoProxied<T>, allowing us to move owned values instead of copying them. * This change also updates the FFI layer for strings/bytes in the repeated and maps thunks to accept a std::string* that can be moved rather than a PtrAndLen type that needs to be copied. * Tests are updated to no longer .as_view() when setting a message / string on a repeated / map field. The IntoProxied trait makes calling .as_view() obsolete. PiperOrigin-RevId: 650580788	5 months ago
Protobuf Team Bot	6429885156	Emit all string fields on Rust-upb regardless of ctype set. As ctype only affects cpp implementation, these fields necessarily already work the same as any other string field. PiperOrigin-RevId: 646156640	5 months ago
Alyssa Haroldsen	3f493d9e52	Use the same set of exports as regular users in shared tests The shared tests which access `protobuf_upb` or `protobuf_cpp` have access to more items than the `protobuf` library itself. This is because the former don't go through the same re-exporting based on kernel. I fix this by creating two test-only libraries that perform the same re-exporting as the `protobuf` library, but with the kernel explicitly set, and changing the shared tests to reference that instead of the inner runtime library. This is needed to reliably test macros, where item paths are relative to the invocation, not eagerly checked at the macro source. PiperOrigin-RevId: 624328817	8 months ago
Jakob Buchgraber	0ecb729906	Implement 'bytes' accessors for UPB :) Move the 'accessors_test.rs' to shared/ as both kernels now support the same accessor types. PiperOrigin-RevId: 526591987	2 years ago
Marcel Hlopko	5091c9fd10	Enable sanitized CI builds PiperOrigin-RevId: 524773049	2 years ago
Marcel Hlopko	a1ba8d238a	Generate bindings under the correct package. Before this CL all messages were generated in the top-level crate module. With this change we generate messages under the module specified by the package declaration in the .proto file. Dots are interpreted as submodule separator in consistency with how C++ namespaces are handled. Note that name of the proto_library target still remains to be used as the crate name. This CL only adds crate submodules dependeing on the specified package. PiperOrigin-RevId: 524235162	2 years ago
Jakob Buchgraber	0bb76d70f7	Implement 'optional int64' accessors per v0 API spec. PiperOrigin-RevId: 523667647	2 years ago
Jakob Buchgraber	c575758ad9	Implement accessors for 'optional bool' per v0 API spec See 'accessors_test.rs' for the generated Rust API. PiperOrigin-RevId: 523650122	2 years ago
Marcel Hlopko	f7a2f4acea	Add initial end-to-end test directory structure. This turns out to be quite of a yak shave to be able to perfectly test both kernels without having to pass extra Blaze flags. PiperOrigin-RevId: 521850709	2 years ago

12 Commits (1aedb97602b084b51c903c6c82b89e39b6d5078a)