- 27 Mar, 2014 1 commit
-
-
Robert Edmonds authored
this is conditional on whether the linker supports version scripts, for which we use the gl_LD_VERSION_SCRIPT macro from the gnulib project. on platforms without version scripts, we fall back to libtool's -export-symbols-regex.
-
- 26 Mar, 2014 5 commits
-
-
Robert Edmonds authored
-
Robert Edmonds authored
-
Robert Edmonds authored
-
Robert Edmonds authored
-
Robert Edmonds authored
-
- 24 Mar, 2014 3 commits
-
-
Robert Edmonds authored
it's possible for the <google/protobuf/compiler/> header files to be shipped in a separate package (e.g., debian's libprotoc-dev). check for this at configure time rather than allowing the build process to fail.
-
Oleg Efimov authored
-
Robert Edmonds authored
there is some confusion with regard to the use of lower case letters in enum values. take the following message definition: message LowerCase { enum CaseEnum { UPPER = 1; lower = 2; } optional CaseEnum value = 1 [default = lower]; } this generates the following C enum: typedef enum _LowerCase__CaseEnum { LOWER_CASE__CASE_ENUM__UPPER = 1, LOWER_CASE__CASE_ENUM__lower = 2 _PROTOBUF_C_FORCE_ENUM_TO_BE_INT_SIZE(LOWER_CASE__CASE_ENUM) } LowerCase__CaseEnum; note that the case of the enum value 'lower' was preserved in the C symbol name as 'LOWER_CASE__CASE_ENUM__lower', but that the _INIT macro references the same enum value with the (non-existent) C symbol name 'LOWER_CASE__CASE_ENUM__LOWER': #define LOWER_CASE__INIT \ { PROTOBUF_C_MESSAGE_INIT (&lower_case__descriptor) \ , 0,LOWER_CASE__CASE_ENUM__LOWER } additionally, the ProtobufCEnumValue array generated also refers to the same enum value with the (non-existent) upper cased version: const ProtobufCEnumValue lower_case__case_enum__enum_values_by_number[2] = { { "UPPER", "LOWER_CASE__CASE_ENUM__UPPER", 1 }, { "lower", "LOWER_CASE__CASE_ENUM__LOWER", 2 }, }; we should preserve the existing behavior of copying the case from the enum values in the message definition and fix up the places where the (non-existent) upper case version is used, rather than changing the enum definition itself to match the case used in the _INIT macro and enum_values_by_number array, because it's possible that there might be existing working code that uses enum values with lower case letters that would be affected by such a change. incidentally, google's C++ protobuf implementation preserves case in enum values. protoc --cpp_out generates the following enum declaration for the message descriptor above: enum LowerCase_CaseEnum { LowerCase_CaseEnum_UPPER = 1, LowerCase_CaseEnum_lower = 2 };
-
- 19 Mar, 2014 2 commits
-
-
Robert Edmonds authored
-
Robert Edmonds authored
this function is now static, so it shouldn't have the "protobuf_c_" prefix in its name.
-
- 14 Mar, 2014 3 commits
-
-
Tomasz Wasilczyk authored
-
Tomasz Wasilczyk authored
-
Tomasz Wasilczyk authored
-
- 07 Mar, 2014 1 commit
-
-
Robert Edmonds authored
-
- 17 Feb, 2014 2 commits
-
-
Kevin Lyda authored
-
Kevin Lyda authored
Also, don't default dot support to on and emit something to say if the dogygen-doc target is enabled.
-
- 16 Feb, 2014 1 commit
-
-
Kevin Lyda authored
Still need to add the comments in the source code. Currently I've seeded it with the libprotobuf-c files. I've configured it to make man pages and html pages. Might not be ideal, but makes it easy for me to check things (html is nicer, but man pages are handier for remote servers).
-
- 10 Feb, 2014 1 commit
-
-
Robert Edmonds authored
-
- 25 Jan, 2014 1 commit
-
-
Kevin Lyda authored
Use MKDIR_P and LN_S rather than their usual expansions.
-
- 17 Jan, 2014 1 commit
-
-
Robert Edmonds authored
This reverts commit aaa40c58.
-
- 14 Jan, 2014 19 commits
-
-
Robert Edmonds authored
of course, protobuf-c uses both C and C++.
-
Robert Edmonds authored
-
Robert Edmonds authored
It’s important to note that, differently from what we’ve seen for the serial test harness (see Parallel Test Harness), the AM_TESTS_ENVIRONMENT and TESTS_ENVIRONMENT variables cannot be use to define a custom test runner; the LOG_COMPILER and LOG_FLAGS (or their extension-specific counterparts) should be used instead: ## This is WRONG! AM_TESTS_ENVIRONMENT = PERL5LIB='$(srcdir)/lib' $(PERL) -Mstrict -w ## Do this instead. AM_TESTS_ENVIRONMENT = PERL5LIB='$(srcdir)/lib'; export PERL5LIB; LOG_COMPILER = $(PERL) AM_LOG_FLAGS = -Mstrict -w (http://www.gnu.org/software/automake/manual/html_node/Parallel-Test-Harness.html)
-
Robert Edmonds authored
"As with the serial harness above, by default one status line is printed per completed test, and a short summary after the suite has completed. However, standard output and standard error of the test are redirected to a per-test log file, so that parallel execution does not produce intermingled output. The output from failed tests is collected in the test-suite.log file. If the variable ‘VERBOSE’ is set, this file is output after the summary." (http://www.gnu.org/software/automake/manual/html_node/Parallel-Test-Harness.html)
-
Ilya Lipnitskiy authored
-
Ilya Lipnitskiy authored
Makefile.am: add valgrind to the AM_TESTS_ENVIRONMENT configure.ac: enable valgrind testing option for ./configure m4/valgrind-tests.m4: enable tracing children for libtool wrapper script compatibility, but ignore standard binaries in /usr or /bin
-
Ilya Lipnitskiy authored
-
Robert Edmonds authored
-
Robert Edmonds authored
-
Robert Edmonds authored
-
Robert Edmonds authored
it turned out to be a bad idea to not export the default allocator's methods via the ProtobufCAllocator's function pointers. there is at least one user (protobuf-c-rpc) that makes allocations by directly invoking those methods.
-
Robert Edmonds authored
use a linker script rather than relying on libtool's -export-symbols-regex. this lets us start versioning the library's exported symbols.
-
Robert Edmonds authored
in particular we no longer use abort().
-
Robert Edmonds authored
according to microsoft's platform evangelist, "we recommend that you consider using a different compiler such as Intel or gcc" if you need a conforming C compiler. since there's already a project that maintains the stdint.h / inttypes.h headers for microsoft compilers (https://code.google.com/p/msinttypes/) there's not much point in maintaining this ourselves.
-
Robert Edmonds authored
there's not much point to having the "private" definitions split out into a separate header file. they're still in the namespace and there's nothing that can be done to prevent "unauthorized" uses. just integrate the definitions into the main header file but put them in the bottom and note that they're "private". this makes it very slightly easier to copy the protobuf-c support library into another project wholesale, since one less file is required.
-
Robert Edmonds authored
i'm not quite sure what this thing is used for or how useful it could possibly be. it's not used anywhere in protobuf-c or protobuf-c-rpc, and it wasn't in protobuf-c 0.14 or earlier. just delete it; if anything actually needs this kind of logic, it's easy enough to replicate it.
-
Robert Edmonds authored
libraries should never generate output on their own to stdout/stderr. remove the PRINT_UNPACK_ERRORS macro and rename UNPACK_ERROR to PROTOBUF_C_UNPACK_ERROR. the error strings are left in but compiled out by default. they could theoretically be re-enabled for a debugging session by changing the PROTOBUF_C_UNPACK_ERROR macro to something like: #define PROTOBUF_C_UNPACK_ERROR(...) do { fprintf(stderr, __VA_ARGS__); fputc('\n', stderr); } while (0)
-
Robert Edmonds authored
-
Robert Edmonds authored
it might be premature to remove the <google/> prefix from the include path. set up a compatibility symlink so that old code continues to compile.
-