| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. Fix C++ tests.
* Add missing files to Makefile.am and fix distcheck in tests.sh
* Remove BUILT_SOURCES from conformance/Makefile.am.
* Add some missing override keyword.
* Add a type cast to int64 because our StrCat() in stubs can't handle size_t.
2. Fix Java tests.
* Add missing test dependency on guava in pom.xml.
* Include newly referenced test data in test resources.
* Manually fix map_lite_test.proto which is overwritten because it's mapped
from map_test.proto in google3.
* Add back "optimize_for = LITE_RUNTIME" which is still needed to keep the
opensource test passing as it's still running lite tests.
* Add a type cast in newBuilder() because without it the code doesn't compile
with openjdk javac 1.8 (the compiler can't figure if it's the right type
due to complex generic typing).
3. Fix Python tests.
* Remove/replace references to <hash_map>.
* Suppress more warnings in setup.py.
* Replace incorrect header inclusion for google/protobuf/proto_api.h.
* Add strings::EndsWith to google/protobuf/stubs/strutil.h because it's
referenced in the updated python C extension code.
* Replace proto2 with google::protobuf. The proto2 name is leaked to
opensource because we removed the subsitition rule for proto2 namespace
but only fixed C++ source code and forgot to update python C extension code.
|
|\ |
|
| | |
|
| |\ |
|
| | | |
|
|/ / |
|
| |
| |
| |
| | |
https://github.com/google/protobuf/issues/4887
|
|\|
| |
| | |
Merge 3.6.x into master
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
It appears that Visual Studio does not work well with std::once_flag
because it has a bug causing it to initialize that during dynamic
initialization instead of constant initialization. This change works
around the problem by using function static initializers instead.
@gerben-s originally wrote this change for the Google-internal codebase
but I am just cherry-picking it here.
This fixes #4773.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The __MIPSEL__ is mips little endian, specific and we needed
to compile for a big endian mips target that also happened
to be a mips64, so replaced the __MIPSEL__ with a __mip__
which should preserve the build on little endian and allow a
build for a big endian setup.
$ echo | mips64-linux-gcc -E -dM -
... snip snip ...
#define __mips__ 1
#define __MIPSEB__ 1
Signed-off-by: Charles Hardin <ckhardin@gmail.com>
|
| |
| |
| |
| |
| | |
Removed forward declaration of MemBlock class.
Declaration is in conflict with declaration `::strings::MemBlock`
|
|\ \
| | |
| | | |
Merge 3.6.x branch into master
|
| |\| |
|
| | | |
|
|\ \ \
| | | |
| | | | |
Remove the executable bit from several source code files
|
| |/ /
| | |
| | |
| | | |
This potentially avoids compiler warnings.
|
|\ \ \
| | | |
| | | | |
Add "override" keyword for overridden virtual functions
|
| | |/
| |/|
| | |
| | |
| | |
| | | |
Add "override" for overridden virtual functions.
Please refer following issue for discussion on this.
https://github.com/google/protobuf/issues/67
|
|\ \ \
| |_|/
|/| | |
Add guards against bswap_XX redefnitions
|
| | |
| | |
| | |
| | | |
Signed-off-by: Oskar Świtalski <oskar.switalski@indoorway.com>
|
|\ \ \
| | | |
| | | | |
Disable thread local storage for OSX < 10.7
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
| |_|/
|/| |
| | |
| | |
| | |
| | |
| | | |
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.
|
| | | |
|
| | | |
|
| | | |
|
|\ \ \
| | | |
| | | | |
Integrated internal changes from Google
|
| | | |
| | | |
| | | |
| | | |
| | | | |
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
|
| |/ / / |
|
|\ \ \ \
| |/ / /
|/| | | |
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++.
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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.
|
|/ /
| |
| |
| |
| |
| | |
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.
|
|\ \
| | |
| | | |
Add more tests to time_test
|
| | |
| | |
| | | |
Better test coverage for datetime validation.
|
|/ /
| |
| |
| | |
Found with PVS-Studio static analyser, see
https://www.viva64.com/en/b/0550/
|
|\ \ |
|
| | | |
|
|\| | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | | |
Unlike GetEnvironmentVariableW,
GetCurrentDirectoryW doesn't set
ERROR_INSUFFICIENT_BUFFER.
|
| | | |
|