- 29 Jul, 2017 2 commits
-
-
Robert Edmonds authored
The protobuf project removed "using namespace std" namespace pollution from their stubs/common.h header file. This caused build failures for us since we relied on their namespace pollution. This commit updates protobuf-c to convert: 'map' → 'std::map' 'set' → 'std::set' 'back_insert_iterator' → 'std::back_insert_iterator'
-
Robert Edmonds authored
proto3: make strings default to "" instead of NULL
-
- 12 Jun, 2017 1 commit
-
-
Paolo Borelli authored
The spec talks about "empty string" and other languages like C# return a zero length string and not null. This is useful because when moving from proto2's "required string" to a proto3's plain string we will be guaranteed that we never get a null pointer. The tradeoff is adding a special case to the library but avoiding a lot of null checks in the calling code. The current code is already special casing "" in pack_string.
-
- 10 Apr, 2017 4 commits
-
-
Robert Edmonds authored
Rework is_zeroish one more time
-
Robert Edmonds authored
Update CMakeLists.txt
-
Paolo Borelli authored
The "member" param cannot be NULL, so last patch was not correct. However to be valgrind clean we need to cast to the right size.
-
alex85k authored
-
- 09 Apr, 2017 5 commits
-
-
Robert Edmonds authored
Small cleanup in oneof handling
-
Robert Edmonds authored
Fix proto3 "is zeroish" evaluation
-
Paolo Borelli authored
Small cleanups: pass the oneof case param by value instead of by reference and use an early return if the case does not match.
-
Paolo Borelli authored
Make sure we actually include the proto3 generated header and test nested submessages
-
Paolo Borelli authored
In proto3 the default value is zero (NULL for pointers and 0 for scalars). However when checking we still need to test by type because poinetr types have an extra indirection.
-
- 27 Feb, 2017 1 commit
-
-
Robert Edmonds authored
Remove leftover FIXME comment
-
- 26 Feb, 2017 1 commit
-
-
Paolo Borelli authored
-
- 25 Feb, 2017 17 commits
-
-
Robert Edmonds authored
Add warning flags to my_CFLAGS
-
Robert Edmonds authored
Here 'j' is used as a loop index and there's no need to re-declare it inside this block. Found with -Wshadow.
-
Robert Edmonds authored
-
Robert Edmonds authored
-
Robert Edmonds authored
CMAKE_SOURCE_DIR -> CMAKE_CURRENT_SOURCE_DIR
-
Robert Edmonds authored
Static build for Windows MSVC
-
Robert Edmonds authored
-
Robert Edmonds authored
proto3 support
-
Robert Edmonds authored
Fix "Bad enums with multiple oneofs" (#251)
-
Robert Edmonds authored
-
Robert Edmonds authored
This commit fixes #251, which causes incorrect code to be generated when there are multiple oneofs in the same message. In #221, we added code to force int-sizing for oneof enums, but we only set vars["opt_comma"] initially, before entering the loop over the message's oneofs. This caused commas to be omitted when generating the enums for subsequent oneofs after the first oneof. This commit resets vars["opt_comma"] every time through the loop that generates the enum declarations for the message's oneofs.
-
Robert Edmonds authored
This commit fixes a problem identified in #251. When generating the PROTOBUF_C__FORCE_ENUM_TO_BE_INT_SIZE directive at the end of the enum values generated for a oneof, we inadvertently only included the protobuf class name in the call to the directive. This breaks if there is more than one oneof per protobuf class, which causes duplicate enum names to be generated. This issue is fixed by also appending the oneof's name to the protobuf class name when generating the PROTOBUF_C__FORCE_ENUM_TO_BE_INT_SIZE directive, which should cause unique enum names to be generated. This changes the enum name that is generated, but these names are implementation details since they are sentinel values that only exist to force the compiler to size the enum type correctly. (The test case for #220 is updated since that test relies on the oneof implementation details in the code generator.)
-
Robert Edmonds authored
Document and extend the effect of passing NULL to ..._free_unpacked functions
-
Richard Kettlewell authored
This makes __free_unpacked() consistent with free(), and simplifies callers by allowing them to indiscriminately free message objects without regard to whether they have been allocated or not.
-
Richard Kettlewell authored
-
Robert Edmonds authored
Add test case for #220
-
Robert Edmonds authored
-
- 05 Dec, 2016 1 commit
-
-
Josh authored
This tells CMake to refer to the currently executed `CMakeLists.txt`'s directory instead of the root project directory. This allows protobuf-c to be included as a subdirectory (via `include_subdirectory`) while being innocuous for those that build protobuf-c by itself.
-
- 27 Nov, 2016 1 commit
-
-
AlexMilich authored
-
- 30 Oct, 2016 3 commits
-
-
Paolo Borelli authored
protobuf does not provide configured tarballs anymore, so we need to run autogen.sh ourselves.
-
Paolo Borelli authored
Same code of the proto2 test but with the corresponding proto3 file
-
Paolo Borelli authored
This is a first cut at adding proto3 support. As far as I understand protobuf-c already has pretty much everything needed once it is built using a new version of protobuf itself. The only missing thing is that in proto3 all fields are optional and having to manually set has_foo is inconvenient. This patch special cases the proto3 syntax files so that structs for the bytes, enum and primitive fields do not emit the has_ field. It also adds PROTOBUF_C_LABEL_NONE to the label enum that is used for proto3 fields. When a fields has this label, the quantifier is not consulted and instead the field is packed/unpacked depending on whether it has a value different from NULL/0.
-
- 07 Sep, 2016 4 commits
-
-
Robert Edmonds authored
Make init_value const
-
Robert Edmonds authored
rm argument fix for solaris
-
Robert Edmonds authored
Relax autoconf constraint from v2.64 -> v2.63 (so that it works on SL6)
-
Thomas Köckerbauer authored
-