| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
| |
It turns out calling StringOutputStream::Next()/BackUp() repeatedly is
very costly in opensource protobuf because it keeps resize() the string
back and forth. The current JSON conversion API suffers this problem and
leads to ridiculously long parsing time:
https://github.com/google/protobuf/issues/2305#issuecomment-257785492
This change fixes the problem but caching the buffer of Next() and avoid
calling BackUp() as much as possible.
|
|\
| |
| | |
Fixes deprecation warnings/errors on Xcode
|
| |
| |
| |
| | |
mac version to that if atomic is enabled
|
|/ |
|
|
|
|
|
|
| |
The size parameter is only used in cases where the compiler supports
sized delete, so when that's not available we need to specifically cast
it to void to prevent a warning.
|
|\
| |
| | |
[arm/gcc] Don't rely on KUSER_HELPERS feature for atomics
|
| |
| |
| |
| |
| |
| | |
ARM specific CONFIG_KUSER_HELPERS feature can be disabled in Linux kernel,
and in this case, we shouldn't crash. Use gcc built-in functions instead
of arm specific code if they are available.
|
|\ \
| | |
| | | |
generic atomicops: Use strong compare_exchange
|
| |/
| |
| |
| |
| | |
Weak compare-and-exchange are allowed to fail spuriously, so we have
to use the strong variation here.
|
|/
|
| |
When running >bazel test :protobuf_test a few compiler warnings came up. They were centered around the usage of "int" as the loop counter where the value should have been a size_t.
|
| |
|
| |
|
| |
|
|\ |
|
| |\
| | |
| | | |
Fix MSVC build when HAVE_LONG_LONG is defined.
|
| | | |
|
| |\ \
| | | |
| | | | |
Fixed references to foreign nested messages with CommonJS-style imports
|
| | |/
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
A bug was causing generated JSPB code with CommonJS-style imports to
refer incorrectly to nested messages from other .proto files. The
generated code would have things like "test_pb.InnerMessage" instead of
"test_pb.OuterMessage.InnerMessage". This commit fixes the problem by
correctly taking into account any message nesting.
|
| |/
| |
| |
| |
| |
| |
| |
| | |
There seem to already be .proto files out there that have conflicting
enum names, which will not be able to build successfully for some
languages (like C#). To prevent this problem from spreading, let's make
it an error for proto3 but just issue a warning for proto2. This fixes
issue #2179.
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| | |
Change-Id: I6a5078b6bcdf4637e11c1cb9da1f74298e6fc26e
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
(#2146)
Currently deserialization of a non-packed binary repeated field is quadratic in
the number of elements, because each time we parse a new element we copy over
all elements we have parsed so far. This CL fixes the performance problem by
having the generated deserialization code just call addX() instead of using
getX() and setX().
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
There seem to already be .proto files out there that have conflicting
enum names, which will not be able to build successfully for some
languages (like C#). To prevent this problem from spreading, let's make
it an error for proto3 but just issue a warning for proto2. This fixes
issue #2179.
|
| |
| |
| |
| |
| |
| |
| |
| | |
A bug was causing generated JSPB code with CommonJS-style imports to
refer incorrectly to nested messages from other .proto files. The
generated code would have things like "test_pb.InnerMessage" instead of
"test_pb.OuterMessage.InnerMessage". This commit fixes the problem by
correctly taking into account any message nesting.
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| | |
Change-Id: I6a5078b6bcdf4637e11c1cb9da1f74298e6fc26e
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
(#2146)
Currently deserialization of a non-packed binary repeated field is quadratic in
the number of elements, because each time we parse a new element we copy over
all elements we have parsed so far. This CL fixes the performance problem by
having the generated deserialization code just call addX() instead of using
getX() and setX().
|
| | |
|
| | |
|
| | |
|
| | |
|
|\ \
| | |
| | | |
Merge 3.0.x into master.
|
| | | |
|
| | | |
|
| |/
|/| |
|
|\ \
| | |
| | | |
Merge 3.0.x into master
|
| |\| |
|
| | |
| | |
| | |
| | |
| | |
| | | |
Parameter deterministic is unused in
InternalSerializeWithCachedSizesToArray(), which generates unused
parameter warning in every message.
|
| | |\
| | | |
| | | | |
Cherry pick c# changes from master
|