| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
Change-Id: Ife609e073bd142d328b680c753f543bb18f4f66d
|
| | | |
| | | |
| | | |
| | | | |
Change-Id: I4c4e2a4a248e4336fef735f75ff5648d8d33627e
|
| | | |
| | | |
| | | |
| | | | |
Also delete some unused files.
|
|\ \ \ \ |
|
| |\ \ \ \
| | | | | |
| | | | | | |
Fix static non-pod type problem with dlopen
|
| | | | | | |
|
| | | | | | |
|
| |/ / / /
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This addresses issue #1008, by creating a JsonFormatter which is private and only different
to JsonFormatter.Default in terms of reference equality.
Other plausible designs:
- The same, but expose the diagnostic-only formatter
- Add something to settings to say "I don't have a type registry at all"
- Change the behaviour of JsonFormatter.Default (bad idea IMO, as we really *don't* want the result of this used as regular JSON to be parsed)
Note that just trying to find a separate fix to issue #933 and using that to override Any.ToString() differently wouldn't work for messages that *contain* an Any.
Generated code changes follow in the next commit.
|
| | | | |
| | | | |
| | | | |
| | | | | |
Change-Id: Ibb3fe6f919cc7ca0df91da5e1697ba33d259e433
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
[skip appveyor]
Change-Id: I700f70e734abb6264db29369ecf227a9e1f783e7
|
|/ / / / |
|
|\ \ \ \
| | | | |
| | | | | |
Arena type traits standardization.
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This is adapted from the branch of @xfxyjwf at:
https://github.com/xfxyjwf/protobuf/commit/494716a682ef854168e92231a3cdcc89d587d9b9
and should solve the protobuf compilation problem against nvcc.
Tested against nvcc 6.5 and 7.0.
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Fix narrowing implicit cast in generated message [NFC]
|
| |/ / / / |
|
|/ / / / |
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
proper platform define used in the right place.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
There are corner cases where MessageDescriptor.{ClrType,Parser} will return null, and these are now documented. However, normally they *should* be implemented, even for descriptors of for dynamic messages. Ditto FieldDescriptor.Accessor.
We'll still need a fair amount of work to implement dynamic messages, but this change means that the public API will be remain intact.
Additionally, this change starts making use of C# 6 features in the files that it touches. This is far from exhaustive, and later PRs will have more.
Generated code changes coming in the next commit.
|
| | | |
| | | |
| | | |
| | | | |
This changes csharp_names.h, which will require a corresponding change in GRPC.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
generated types.
Generated code coming in next commit - in a subsequent PR I want to do a bit of renaming and redocumenting around this, in anticipation of DynamicMessage.
|
|\ \ \ \
| | | | |
| | | | | |
Change the way the "umbrella class" (descriptors) is named/namespaced
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Instead of having a Proto nested namespace to avoid conflicts between the descriptor-holding static class and message classes, just append "Reflection" to the name.
Generated code changes (and corresponding manual changes) in following commit.
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | | |
In the old flow, any 2 char prefix in the expected file was still generating a
warning about being a poor prefix. Now we check the expected file first, so
anything expected is let through.
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This fixes issue #832.
Generated code changes in next commit.
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
1. mathlimits.h must be included before the inclusion of cmath (which
gtest/gtest.h seems to include).
2. hash function for StringPiece doesn't work.
Change-Id: I358a25d941a25b10b39fe76780eda41557699811
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
VC++ up to VS 2015 RTM does not require explicit storage allocation for
static const integers declared in classes. VS 2015 Update 1 requires
these storage definitions in some cases. It's unclear exactly what
cases - simple tests work with and without the explicit storage
allocation.
Many previous versions of VC++ have theoretically *allowed* a
definition to supply storage, but tests on VC++ 2013 show that this
doesn't actually work correctly - it leads to duplicate definition
errors in Chromium. So, the change is scoped to VS 2015 only.
This change also updates the generated files to match the new generator.
TL;DR - this change is necessary in order for Chromium to build with
VS 2015 Update 1.
|
|\ \ \ \
| | | | |
| | | | | |
Use oneof case in equality tests
|
| | | | |
| | | | |
| | | | |
| | | | | |
The included C# test will fail until the regenerated code is used, which is in the next commit.
|
|/ / / /
| | | |
| | | |
| | | | |
consistent with other protobuf library includes.
|
|\ \ \ \
| | | | |
| | | | | |
Improved CMake project
|
| | | | | |
|
| |/ / / |
|
|\ \ \ \
| | | | |
| | | | | |
Expose GetOutputFile in csharp_names.h
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Changed return type from int32 to bool in function returning a bool
|
| | | | | | |
|
|/ / / / / |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Use std::get where it's available
|
| | | | | | |
|
| | | | | |
| | | | | |
| | | | | | |
Used static cast to improve initalization compatibility with MSVC2010
|
| | | | | |
| | | | | |
| | | | | | |
VS2010 cannot compile json_util_test.cc since it assumes NULL as a zero (int), and therefore cannot convert into to char*
|
| | | | | | |
|
| | | | | | |
|
| | | | | | |
|