| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| | |
Update instructions about getting protobuf source.
|
| | |
|
| |
| |
| |
| |
| | |
Since the generated header import GPBProtocolBuffers.h, there is no need
to generate imports for the WKTs as they will have already been imported.
|
|/
|
|
|
|
|
|
| |
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
|
| |
|
|
|
| |
Add googletest as a submodule in third_party/googletest.
|
|
|
|
|
|
|
|
|
| |
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
|
|\
| |
| | |
Remove javanano.
|
| | |
|
| |
| |
| |
| | |
(#4350)
|
|\ \
| | |
| | | |
Fix bug: whether always_print_enums_as_ints is true or false, it alwa…
|
| |\ \ |
|
| | | |
| | | |
| | | |
| | | | |
DefaultValueObjectWriter::FindEnumDefault
|
| | | |
| | | |
| | | |
| | | | |
print the default value of enums as strings
|
| |_|/
|/| |
| | | |
This makes possible to build grpc_php_plugin with protobuf in shared mode.
|
| |/
|/| |
|
|\ \
| | |
| | | |
Integrated internal changes from Google
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
Pre-5.1.0 versions of GCC do not support these particular type traits
(see https://github.com/google/protobuf/issues/417).
|
| | |
| | |
| | |
| | |
| | | |
We no longer need this, now that we have finished the switch to C++11
and are using std::unique_ptr.
|
| | | |
|
| |\ \ |
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
Fix spelling error of __GNUC_MINOR__
|
| | |/ /
| |/| | |
|
|\ \ \ \
| |/ / /
|/| | | |
Support using MSVC intrinsics in Log2FloorNonZero
|
| |/ / |
|
| |/
|/|
| |
| | |
This reverts commit 724f0be33d0eb42055ebe81ce0c8c7bf88ea666c.
|
|\ \
| | |
| | | |
Add support for libc++ on Windows.
|
| | |
| | |
| | |
| | |
| | | |
This disables a couple of workarounds which are only necessary with
MSVC's standard library and cause problems with libc++.
|
|\ \ \
| | | |
| | | | |
Blacklist all WELL_KNOWN_PROTOS from Bazel C++ code generation.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
This lets all values of `WELL_KNOWN_PROTOS` be treated the same with
regard to Bazel's protobuf blacklisting.
|
|\ \ \ \
| |/ / /
|/| | | |
Adds an option in C++ JSON parser to ignore unrecognized enum values
|
| |/ /
| | |
| | |
| | | |
unrecognized string values in enum fields as default ones.
|
|\ \ \
| | | |
| | | | |
Rename a shadowed variable.
|
| | |/
| |/|
| | |
| | |
| | |
| | | |
Shadowed variables can cause readability issues. Ensure a shadowed
variable isn't used in header files which may be used in a dependent
project that explicitly disables them.
|
|\ \ \
| |/ /
|/| | |
Remove unused output_file variable from js_embed
|
| | |
| | |
| | |
| | |
| | | |
The js_embed tool outputs to stdout, so the output_file variable is
unnecessary and unused.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Chrome is running into two issues with the use of this macro
in open-source protobuf (https://crbug.com/809157):
1. GOOGLE_FALLTHROUGH_INTENDED is defined to nothing on __APPLE__
platforms, which blocks us from enabling -Wimplicit-fallthrough
on Mac and iOS. (We use a hermetic self-built modern clang,
so whatever Xcode bug that exclusion might be for doesn't apply
to us.)
2. It's in a public header file, and it's included in a public header file.
When clang suggests adding [[clang::fallthrough]], it checks if it knows of
a macro expanding to that and if so, suggests inserting that. Since lots of
chrome code includes protobuf headers, it often suggests inserting
GOOGLE_FALLTHROUGH_INTENDED (from protobuf) instead of the correct
FALLTHROUGH (from chrome's base).
Since the fallthrough doens't do anyting useful, just remove it.
Long ago, this might have had perf impact, but d64a2d9941c36a7bc added a
parsing fast path that calls this switch as slow fallback, so it should
be off the hot path nowadays.
No intended behavior change.
This is the public version of internal change 184824132.
|
|\ \ \
| | | |
| | | | |
Create std::string in Arena memory
|
| | | |
| | | |
| | | | |
'<:' is an alternate spelling for '['.
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
Fixes issue #1154 by noting that `vcpkg` contains protobuf. Potential improvements: also remark how to use `vcpkg` to get dependencies when building from source via CMake.
|
|\ \ \ \
| | | | |
| | | | | |
Java: Generate warning free code
|
| | | | |
| | | | |
| | | | |
| | | | | |
Partially fixes #4230.
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | | |
These statements pulled a bunch of symbols from the std namespace into
the global namespace. This commit removes all of them except for
std::string, which is a bit trickier to remove.
|
|\ \ \ \
| | | | |
| | | | | |
Remove the use of BUILT_SOURCES
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fixes https://github.com/google/protobuf/issues/3912
[BUILT_SOURCES](https://www.gnu.org/software/automake/manual/html_node/Sources.html)
is used with [Automake's automatic dependency tracking](https://www.gnu.org/software/automake/manual/html_node/Dependencies.html#Dependencies)
but doesn't work well in protobuf when cross-compiling. Their presence
causes maek to always generate them even when they are not requested
and as a result causes cross-compilation to fail because the built protoc
cannot be used to generate unittest protos (see: https://github.com/google/protobuf/issues/3912).
Removing it with explicit dependencies fixes the build issue when running
'make', 'make all', 'make install' at cross-compilation. It doesn't affect
'make protoc' because BUILT_SOURCES only works for the implicit targets
'all', 'check' or 'install'.
|