Merge pull request #237 from wak-google/fix-build

WIP: cmake cleanup to support installable host tooling

Merge pull request #236 from wak-google/fix-lib

cmake: Fix library name

cmake: Cleanup the build so that host and runtime tooling is installable

Signed-off-by: William A. Kennington III <wak@google.com>

cmake: Fix library name

The produced static library should be `libprotobuf-nanopb.a`
instead of the current `liblibprotobuf-nanopb.a`.

Signed-off-by: William A. Kennington III <wak@google.com>

Merge pull request #230 from Tobba/nonempty-substring

Fix closing a non-empty substream resulting in an incorrect stream state

Fix closing a non-empty substream resulting in an incorrect stream state

Merge branch 'master' of github.com:nanopb/nanopb

Fix multiple oneofs in same message (issue #229)

Previously the field iterator logic didn't know whether two
oneof fields were part of the same union, or separate. This
caused wrong pointers to be calculated if multiple oneofs were
inside a single message.

This commit fixes this by using dataoffset of PB_SIZE_MAX to
indicate union fields after the first field.

Theoretically PB_SIZE_MAX is also a valid value for data offset,
which could cause errors. Adding a compile-time assert for this
is somewhat difficult. However I consider it extremely unlikely
that there is any platform that could trigger this situation, as
it would require 255 bytes of extra data/padding between two protobuf
oneof fields. On 64-bit architectures the worst case is 16 bytes,
and even esoteric platforms only align to 64 bytes or so. Manual
modification of the generated .pb.h file could trigger this, but
even then it would require pretty bad luck to happen.

Add testcase for issue #229

Update readme

Merge pull request #231 from Tobba/expose-varint32

Make pb_decode_varint32 public API

Make pb_decode_varint32 public API

Allow overriding proto3 mode (#228)

Enable clang integer sanitizer and clean up a few warnings.

Changed to use simple indexing instead of while (count--) in buf_read()/buf_write(),
because the count overflowed from 0 to max on the last iteration. While the unsigned
integer overflow is defined and behaviour was correct, making this simple change
allowed enabling the sanitizer which might catch true errors elsewhere in the code.

Merge branch 'kurddt-fix_unalign' (issues #226, #227)

Add regression test for issue #227

Refactor proto3 logic into pb_check_proto3_default_value()

Mainly to clean up the code. Also fixed handling of
PB_LTYPE_FIXED_LENGTH_BYTES for proto3 files.

Fix potential unaligned access

If the type is string, do not try to deference it as int16, int32 or int64.
This may lead to unalign memory access, which may cause trap on some architectures (ARM)

Merge branch 'wak-google-upstream1' (#223)

Test case for enum_to_string

Small indentation and naming fixes to enum_to_string functionality

nanopb: update generator to emit optional enum->string mapping function

Google-Bug-Id: 28000875
Signed-off-by: William A. Kennington III <wak@google.com>
Change-Id: I1bffd39168abe04593588291b0ebbe5199a00138

Include version number in PlatformIO library.json (issue 222)

Change download links to https

Setting version to 0.3.8-dev

Publishing nanopb-0.3.7

Note about protoc update

Build fix for previous commit

Build fix for previous commit

Strip debug symbols from linux library files

Only run alltypes_proto3 test case if protoc version is new enough

Update changelog and authors list

Add test cases for proto3 has_ field support.

Set the proto3 option by default if file specifies proto3 syntax

Fix missing warning with large bytes fields (issue #220)

Need to generate compile time check if the bytes field + size field
might exceed 255 bytes. Also eliminated spurious checks generated
for some callback fields.

Merge pull request #218 from berni155/master

Fix typo in encoding

Fix typo in encoding

Merge pull request #208 from podsvirov/topic-cmake-project

Added CMake project

Merge pull request #216 from berni155/proto3_singular_fields_support

Proto3 singular fields support

Prevent fields with default value from encoding when proto3 option is set

Add proto3 option to handle singular fields

Fix typo in docs (thanks to @alecdavis)

Merge pull request #213 from PiotrSikora/export_license

Bazel: export LICENSE.txt file.

Bazel: export LICENSE.txt file.

Signed-off-by: Piotr Sikora <piotrsikora@google.com>

Fix formatting error in docs

Merge pull request #211 from tmroeder/feat-inline-bytes

Add inline allocation of bytes fields

Add inline allocation of bytes fields

This commit adds a new FT_INLINE allocation type that forces bytes
fields to be inlined into the struct. E.g., pb_byte_t my_bytes[32].
This requires max_size for the bytes field. The FT_INLINE type is
represented as a new LTYPE: FT_LTYPE_FIXED_LENGTH_BYTES.

This commit also updates the documentation with FT_INLINE and
FT_LTYPE_FIXED_LENGTH_BYTES.

Added an AUTHORS file in apparent order of appearance in the git log
history from $(git log --all).

Merge pull request #209 from LuminateWireless/bazel-build

Add bazel BUILD file for nanopb.

Add bazel BUILD file for nanopb.

Added CMake project

Include package name in include guard (issue #207).

Fix suggested by Ulenspiegel.

Also added testcase for the same.

Setting version to 0.3.7-dev

Publishing nanopb-0.3.6

Update changelog

Clarify security model with regard to pointer _count fields.

Protect against corrupted _count fields in pb_release().

Fixes a potential security issue (#205). Only relevant if the user
code writes untrusted data to _count fields, but this is allowed as
per the security model.

Add testcase for issue #205

Add static repeated submessage field to mem_release test case.

Attempt at reproducing issue #204.

Small updates to docs

Fix error in STATIC_ASSERT with multiple files (issue #203)

The FieldMaxSize class was reusing the same list instance,
causing problems when multiple files were specified on the
protoc command line.

Add testcase for issue #203

Merge pull request #201 from yanivmo/patch-1

Added explanation of `oneof` section usage

Added explanation of `oneof` section usage

Fix formatting in docs

Some more docs updates

Merge pull request #196 from mxk/api-ref

Update API reference to match headers

Update API reference to match headers

Update typedefs and function prototypes in the API reference to match
header files. Delete documentation for pb_skip_varint/pb_skip_string,
which are superseded by pb_skip_field, and add pb_get_encoded_size.

Provide comments about uncalculable message sizes (issue #195)

Merge pull request #194 from isotes/master

Generate MIN/MAX/ARRAYSIZE for enums

Merge pull request #193 from recursify/master

Add -D option to specify output directory

Generate MIN/MAX/ARRAYSIZE for enums

This generates #defines mirroring the following values from the generated C++ code of GPB
* const Foo Foo_MIN: the smallest valid value of the enum (VALUE_A in the example).
* const Foo Foo_MAX: the largest valid value of the enum (VALUE_C in the example).
* const Foo Foo_ARRAYSIZE: always defined as Foo_MAX + 1.

Add -D option to specify output directory

Setting version to 0.3.6-dev

Publishing nanopb-0.3.5

Update pb_release docs

Update changelog

Add code generator insertion points to files (#178).

Fix a few remaining bugs related to CHAR_BIT!=8 platforms.

Remove obsolete __BIG_ENDIAN__ compilation option.

This is now handled automatically using shift operations.

Add note to migration doc

Replace uint8_t with a pb_byte_t typedef.

This supports platforms where uint8_t does not exist.
If you are using a custom pb_syshdr.h, this may require adding
definitions for uint_least8_t etc.

Modify the int size STATIC_ASSERTS to work with CHAR_BITS!=8.

This will still catch the most common bug of long int vs. long long int.
The uint8_t checks do not seem necessary, test for this will be added
in later commit.

Get rid of type punning in pb_encode_fixedXX().

This was never very clean code, but it was fast. Hopefully
compilers are smart enough to optimize it away, or the speed
difference is not very large. This should be checked.

However working code is always more important than fast code,
and the previous way couldn't really work for platforms that
do not have byte-sized memory access. Related to PR #191.

Merge pull request #190 from aeruder/master

pb_istream_from_buffer: add const to prototype

pb_istream_from_buffer: add const to prototype

This commit changes the prototype for pb_istream_from_buffer from:
  pb_istream_t pb_istream_from_buffer(uint8_t *buf, size_t bufsize);
to
  pb_istream_t pb_istream_from_buffer(const uint8_t *buf, size_t bufsize);

This allows pb_istream_from_buffer users to point to const buffers
without having to inspect code (to ensure practical const-ness) and then be
forced to manually cast away const.

In order to not break compatibility with existing programs (by
introducing a const/non-const union in the pb_istream_t state) we simply
cast away the const in pb_istream_from_buffer and re-apply it when
possible in the callbacks.  Unfortunately we lose any compiler help in
the callbacks to ensure we are treating the buffer as const but manual
inspection is easy enough.

Fix mistake in previous commit

Fix generator crash with Enum inside Oneof (issue #188).

Add testcase for the same.

Second fix for -pedantic build issue in tests

Fix -Wno-pedantic on old GCC

Add testcase for anonymous unions + few fixes.

Fixes compilation error with anonymous unions when
it is not the last field in message. Also fixes
extraneous newlines in header file. Cleanup the
pb.h extraneous use of ##.

Add option to allow for anonymous unions

Ignore null pointers in pb_release() (issue #183).

Add contributing.md to provide issue templates on github

Merge pull request #181 from kylemanna/cmake

Cmake updates

examples: cmake_simple: Use auto discovered path

* No need to specify the src path and the module path. Let cmake do it.

cmake: Auto discover NANOPB_SRC_ROOT_FOLDER

* Use CMAKE_CURRENT_LIST_DIR to learn the path of the currently running
  file to determine location of NanoPB.
* Simplifies use in other projects.

cmake: Build generator files in build directory

Treat the source directory as immutable.  Copy the generator directory
which previously generated files in-tree to the build directory and
then generate files.

Many emerging continuous integration build systems test builds across
multiple versions of dependencies protobuf and python versions in
particular.  The previous source tree builds resulted in stale files
from the last build breaking the current build.  By placing the build
files in the build directory, the build system automatically removes
stale files (removes output build directory) and regenerates them as
necessary.

Merge pull request #176 from kylemanna/whitespace

generator: Remove trailing whitespace

generator: Remove trailing whitespace

* Remove trailing whitespace
* No functional changes

Fix regression in generating message size defines (issue #172).

This bug was triggered when:
1. A .proto file included another .proto from a different directory.
2. The another .proto has an associated .options file.

Added regression test for the same.