aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/google/protobuf
Commit message (Collapse)AuthorAge
* Merge pull request #4824 from sschuberth/remove-exec-bitGravatar Feng Xiao2018-06-25
|\ | | | | Remove the executable bit from several source code files
* \ Merge pull request #4608 from khuzemap/masterGravatar Feng Xiao2018-06-25
|\ \ | | | | | | Add "override" keyword for overridden virtual functions
* \ \ Merge pull request #4172 from OskarKinomoto/masterGravatar Adam Cozzette2018-06-25
|\ \ \ | | | | | | | | Add guards against bswap_XX redefnitions
| | | * Remove the executable bit from several source code filesGravatar Sebastian Schuberth2018-06-25
| |_|/ |/| | | | | | | | This potentially avoids compiler warnings.
* | | Fix parameters declarationGravatar vervaekejonathan2018-06-22
| | |
* | | Merge pull request #4813 from htuch/fix-any-in-mapGravatar Feng Xiao2018-06-21
|\ \ \ | | | | | | | | protostream_objectwriter: fix bug when Any is directly embedded in a …
* \ \ \ Merge pull request #4812 from htuch/fix-any-caseGravatar Feng Xiao2018-06-21
|\ \ \ \ | | | | | | | | | | protostream_objectsource: preserve print options across Any.
* \ \ \ \ Merge pull request #4811 from htuch/invalid-name-jsonGravatar Feng Xiao2018-06-21
|\ \ \ \ \ | | | | | | | | | | | | json: include field name in InvalidName status messages.
| | * | | | Added TODO for grouping options.Gravatar Harvey Tuch2018-06-21
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Harvey Tuch <htuch@google.com>
| | | * | | protostream_objectwriter: fix bug when Any is directly embedded in a map.Gravatar Harvey Tuch2018-06-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This came up when trying to round-trip the JSON<->proto3 conversion for Envoy's envoy.admin.v2alpha.ConfigDump (https://github.com/envoyproxy/envoy/blob/b903c1dc984593f8e95e0569c1503dc5d85fbe34/api/envoy/admin/v2alpha/config_dump.proto#L29). Validated fix on the Envoy code base and via the provided unit test. Signed-off-by: Harvey Tuch <htuch@google.com>
| | * | | | protostream_objectsource: preserve print options across Any.Gravatar Harvey Tuch2018-06-21
| | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #4771. Based on the solution included in the issues from @wangjinhua. Validated this works with Envoy's /config_dump JSON rendering. Signed-off-by: Harvey Tuch <htuch@google.com>
| * | | | Additional whitespace.Gravatar Harvey Tuch2018-06-21
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Harvey Tuch <htuch@google.com>
| * | | | json: include field name in InvalidName status messages.Gravatar Harvey Tuch2018-06-21
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is invaluable for chasing down the culprit in failed JSON -> proto conversions. We often see in Envoy that a failed JSON (or indirectly via YAML) configuration won't load but it's unclear why. Signed-off-by: Harvey Tuch <htuch@google.com>
* | | | Merge pull request #1261 from gjasny/disable-tls-for-osx-10.6Gravatar Adam Cozzette2018-06-20
|\ \ \ \ | | | | | | | | | | Disable thread local storage for OSX < 10.7
* \ \ \ \ Merge pull request #4780 from mwei0210/masterGravatar Adam Cozzette2018-06-20
|\ \ \ \ \ | |_|/ / / |/| | | | check provided size to check whether to export
| | | | * Add "override" for overridden virtual functionsGravatar Khuzema Pithewan2018-06-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add "override" for overridden virtual functions. Please refer following issue for discussion on this. https://github.com/google/protobuf/issues/67
| * | | | not emptyGravatar ming2018-06-19
| | | | |
* | | | | Merge pull request #4626 from robhancocksed/masterGravatar Feng Xiao2018-06-18
|\ \ \ \ \ | | | | | | | | | | | | Declare some generated C++ methods as static #4602
| | * | | | check provided sizeGravatar ming2018-06-19
| |/ / / / |/| | | |
* | | | | Merge pull request #2121 from mazurkin/issue-2108Gravatar Feng Xiao2018-06-18
|\ \ \ \ \ | | | | | | | | | | | | issue-2108 [protoc/Java] Excessive copying on buildPartial()
* | | | | | proto_writer.h: RenderBytes is an override. Marking it. (#4707)Gravatar medinandres2018-06-18
| | | | | | | | | | | | | | | | | | * proto_writer.h: RenderBytes is an override. Marking it.
* | | | | | Merge pull request #4739 from asimshankar/tf-bytesizeGravatar Feng Xiao2018-06-18
|\ \ \ \ \ \ | | | | | | | | | | | | | | Graceful failure in SerializeToArray().
* \ \ \ \ \ \ Merge pull request #3901 from wijagels/masterGravatar Feng Xiao2018-06-11
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix unused parameter warnings in generated code with arenas enabled
| | * | | | | | Graceful failure in SerializeToArray().Gravatar Asim Shankar2018-06-07
| |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See https://github.com/tensorflow/tensorflow/issues/19657 for motivation. But long story short, without this change: m.SerializeToArray(buffer, m.ByteSizeLong()); would result in a CHECK failure if m.ByteSizeLong() returned a value >2GB.
* | | | | | | Remove js_embed binary. (#4709)Gravatar Feng Xiao2018-06-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove js_embed binary.
* | | | | | | Added Clang thread-safety annotations in mutex.hGravatar Adam Cozzette2018-05-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For Clang's thread safety analysis (-Wthread-safety) to pass without warnings, we need to annotate WrappedMutex to indicate that Lock() acquires the lock and Unlock() releases it. This CL adds the annotations and guards them with an ifdef to make sure they're a no-op on compilers other than Clang. This is a cherry-pick of a change I already made to the Google-internal codebase.
* | | | | | | Move methods out of class (#4697)Gravatar Paul Yang2018-05-29
| | | | | | |
* | | | | | | Add back GeneratedClassName to public (#4686)Gravatar Paul Yang2018-05-25
| | | | | | | | | | | | | | | | | | | | | These methods are used by grpc
* | | | | | | PHP namespaces for nested messages and enums (#4536)Gravatar Brent Shaffer2018-05-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * uses namespaces for nested messages and enums * fixes namespaces for PHP dist * fixes namespace for Descriptors, adds Cardinality and Kind * fixes nested namespaces for reserved words and adds tests * adds tests and generator fix for php class prefixes * fixes escaping of protobuf packages, enum comments, misc others * nice refactor of generated code * adds class files for backwards compatibility * simplifies code with templates * adds compatibility files to makefile * cleanup of generator and fixes nested namespace bug * regenerates proto types * remove internal BC classes * adds deprecated warning, adds methods back * simplifies if statement * fixes dist files * addresses review comments * adds back TYPE_URL_PREFIX constant * adds @deprecated to old nested class files * skips tests which require a separate process when protobuf.so is enabled * Adds tests for legacy nested classes that do not require separate processes to test * uses legacy names for GPBUtil message check * adds block for IDE @deprecated message * Namespace for nested message/enum in c extension * Remove unused code
* | | | | | | PHP array constructors for protobuf messages (#4530)Gravatar Brent Shaffer2018-05-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * PHP array constructors for protobuf messages * removes Descriptor from error message * allows mergeFrom to accept an array * only use initWithDescriptor if instanceof MapEntry * adds doc comments * removes ability for constructors to take arrays for submessages * Revert "allows mergeFrom to accept an array" This reverts commit b7b72182d561634af12c5c5c56a7cda3b33241f9. * makes mergeFromArray protected and fixes mergeFrom whitespace * Separates merging from JSON and merging from PHP array * removes well-known types and json keys from array construction * Addresses PR review comments * cleans up tests * fixes exception messages
* | | | | | | Adopt ruby_package in ruby generated code. (#4627)Gravatar Paul Yang2018-05-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Adopt ruby_package in ruby generated code. * Add test for ruby_package
* | | | | | | re-write int128 long division to avoid license impact from stackoverflow ↵Gravatar NickFengIBM2018-05-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | references (#4633) * rewrite int128 long divison to avoid stackoverflow hit Protobuf was showing Stackoverflow hits in the code base, primarily code written to calculate long division. This code was copied from a stackoverflow post, which means it would be licensed under CC BY-SA 3.0. Due to this license, IBM Legal did not want to include this OSS in our products and advised us to re-write this particular piece of code to avoid the license restriction. We have re-written the code for our own distribution, and are willing to merge it into the main code base for others who want to avoid the stackoverflow license issues to benefit as well.
* | | | | | | Merge pull request #4624 from eughermann/hash-ub-fixGravatar Adam Cozzette2018-05-15
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix undefined behavior in hash function.
| | | | * | | | Declare some generated C++ methods as static #4602Gravatar Robert Hancock2018-05-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This avoids compile warnings when compiling generated code with the -Wmissing-declarations option.
* | | | | | | | Adopt php_metadata_namespace in php code generator (#4622)Gravatar Paul Yang2018-05-14
| |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Adopt php_metadata_namespace in php code generator The php_metadata_namespace is corresponded to the relative directory of the metadata file. e.g., previously, the metadata file of foo.proto was GPBMetadata/Foo.php. If the php_metadata_namespace is "Metadata\\Bar", the metadata file will be Metadata/Bar/Foo.php. * Handle empty php_metadata_namespace
| * | | | | | Remove undefined behavior from the hash function.Gravatar Eugene Hermann2018-05-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed integer overflow creates undefined behavior that may lead to unpredictable fails on different platforms. One known example of the hardware where this code did fail is Apple A6 (32-bit Apple Swift CPU) 16777619, 16777499 - two prime numbers that typically used to get better dispersion.
* | | | | | | fix index out of range error in C# generation using msvc (#1329)Gravatar Warren Falk2018-05-14
|/ / / / / /
* | | | | / Add file option php_metadata_namespace and ruby_package (#4609)Gravatar Paul Yang2018-05-11
| |_|_|_|/ |/| | | |
* | | | | Fix error in Clang UndefinedBehaviorSanitizerGravatar Petter Strandmark2018-05-04
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Pointer Arguments to memcpy can not be null in UndefinedBehaviorSanitizer. In this case, both the memory and the size was zero. This change allows protoc to run under UndefinedBehaviorSanitizer.
* | | | | MinGW build failedGravatar Ivan Shynkarenka2018-05-02
| | | | |
* | | | | typoGravatar Brady Killeen2018-04-26
| | | | |
* | | | | Write messages to backing field in generated C# cloning code (#4440)Gravatar Sydney Acksman2018-04-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Edited MessageFieldGenerator to clone to backing field instead of property * Generated C# proto code
* | | | | Merge pull request #4504 from xfxyjwf/liteGravatar Feng Xiao2018-04-18
|\ \ \ \ \ | | | | | | | | | | | | Cleanup + documentation for Java Lite runtime.
* | | | | | Fixed a Visual Studio 2017 build error. (#4488)Gravatar Ronny Krüger2018-04-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current 15.6.x versions of Visual Studio 2017 contain a bug that prevent them from compiling the following construct under certain conditions: std::unique_ptr<std::unique_ptr<Foo> []> foos; This will fail to compile if Foo is an abstract class. To work-around the problem the whole construct was change into: std::vector<std::unique_ptr<Foo>> foos; This not only fixes the compiler error but is also more readable than previous version.
| * | | | | Cleanup + documentation for Java Lite runtime.Gravatar Feng Xiao2018-04-12
|/ / / / /
* | | | | Don't generate imports for the WKTs unless generating the WKTs.Gravatar Thomas Van Lenten2018-04-02
| | | | | | | | | | | | | | | | | | | | | | | | | Since the generated header import GPBProtocolBuffers.h, there is no need to generate imports for the WKTs as they will have already been imported.
* | | | | Trim imports for bundled generated protos.Gravatar Thomas Van Lenten2018-04-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To avoid a cycle between headers, have the WKTs use minimal imports instead of using the helper to get everything from the library. Fixes https://github.com/google/protobuf/issues/4301 Fixes https://github.com/google/protobuf/issues/4403
* | | | | Removed atomicops.h since it is no longer usedGravatar Adam Cozzette2018-03-27
| | | | |
* | | | | source code info for interpreted options; fix source code info for extension ↵Gravatar Joshua Humphries2018-03-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | range options (#4342) * when interpreting options, rewrite file descriptor's source code info - so that interpreted option paths have correct location information - so that corresponding uninterpreted option paths are removed also includes a fix to source code locations for extension range options
* | | | | Merge pull request #4431 from xfxyjwf/rmnanoGravatar Feng Xiao2018-03-26
|\ \ \ \ \ | | | | | | | | | | | | Remove javanano.