| Commit message (Collapse) | Author | Age |
|\
| |
| | |
Remove the executable bit from several source code files
|
|\ \
| | |
| | | |
Add "override" keyword for overridden virtual functions
|
| |/
|/|
| |
| | |
This potentially avoids compiler warnings.
|
|\ \
| | |
| | | |
protostream_objectwriter: fix bug when Any is directly embedded in a …
|
|\ \ \
| | | |
| | | | |
protostream_objectsource: preserve print options across Any.
|
| | | |
| | | |
| | | |
| | | | |
Signed-off-by: Harvey Tuch <htuch@google.com>
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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>
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Harvey Tuch <htuch@google.com>
|
|/ /
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| | |
Add "override" for overridden virtual functions.
Please refer following issue for discussion on this.
https://github.com/google/protobuf/issues/67
|
|/
|
| |
* proto_writer.h: RenderBytes is an override. Marking it.
|
| |
|
|\ |
|
| |
| |
| |
| | |
DefaultValueObjectWriter::FindEnumDefault
|
| |
| |
| |
| | |
print the default value of enums as strings
|
| |\
| |/
|/| |
|
| | |
|
|\ \
| |/
|/| |
Adds an option in C++ JSON parser to ignore unrecognized enum values
|
| |
| |
| |
| | |
unrecognized string values in enum fields as default ones.
|
|/
|
|
|
|
| |
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.
|
|\ |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| | |
Make it clear default value is only omitted for proto3.
|
|/ |
|
| |
|
|
|
|
| |
See https://github.com/google/protobuf/issues/3752 for context.
|
| |
|
| |
|
|\ |
|
| | |
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Comment out unused arguments.
These last few are all that's needed to compile with -Wunused-arguments.
* Fix missing struct field initializer.
With this fix, everything compiles with -Wmissing-field-initializers.
* Add support for disabling unaligned memory accesses on x86 too.
ubsan doesn't like these because they are technically undefined
behavior, so -DGOOGLE_PROTOBUF_DONT_USE_UNALIGNED will disable them easily.
* Avoid undefined integer overflow.
ubsan catches all of these.
|
|\ |
|
|\ \
| | |
| | | |
Remove "using namespace std" from stubs/common.h
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This prevents the contents of the std namespace from being effectively
pulled into the top-level namespace in all translation units that
include common.h. I left in individual using statements for a few common
things like std::set and std::map, because it did not seem worth going
through the churn of updating the whole codebase to fix those right now.
|
|/ /
| |
| |
| |
| | |
When compiling with -Werror, -Wunused-const-variable the build fails due
to those two constants not being used.
|
|/ |
|
|\
| |
| | |
migrate delimited messages functions to util package
|
|\ \
| | |
| | | |
Add preserve_proto_field_names option to JsonPrintOptions
|
|\ \ \
| | | |
| | | | |
DefaultValueObjectWriter should populate oneof message field
|
| | | | |
|
| | | | |
|
| |/ /
|/| | |
|
| | | |
|
| | | |
|
| |/
|/| |
|
| | |
|