aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Changes to generated code from previous commitJon Skeet2016-07-0420-244/+2537
|
* Move DebuggerNonUserCodeAttribute to function membersJon Skeet2016-07-0413-39/+73
| | | | | | | | | I think this has caught everything. I've left a stub for attributes to be applied to the types themselves, but we don't currently need anything. Follow-up commit will include the changes to generated code itself. Fixes #1671.
* Merge pull request #1704 from lizan/json_parse_optionsFeng Xiao2016-07-018-32/+214
|\ | | | | Add JsonParseOptions to ignore unknown fields
| * Add JsonParseOptions to ignore unknown fieldsLizan Zhou2016-06-288-32/+214
| | | | | | | | | | - add JsonParseOptions for JsonToBinaryString allow unknown fields - rename current JsonOptions to JsonPrintOptions
* | Merge pull request #1738 from xfxyjwf/fixbuildFeng Xiao2016-07-014-4/+67
|\ \ | | | | | | Fix some failing travis tests.
| * | Fix some failing travis tests.Feng Xiao2016-07-014-4/+67
|/ / | | | | | | | | | | | | 1. Add missing header file to Makefile.am. 2. Re-generate objectivec generated code for well-known types. Change-Id: If28217c701cf8bd739ea0db240e9eee600f23ee7
* | Merge pull request #1710 from chezRong/masterFeng Xiao2016-07-012-26/+159
|\ \ | | | | | | Add minified Json printing functionality
| * | added minified JSON formatting functionality with testchezRong2016-07-012-26/+159
| | |
* | | Merge pull request #1723 from thomasvl/objc_test_coverageThomas Van Lenten2016-07-016-43/+11
|\ \ \ | |/ / |/| | Xcode project cleanup/setup.
| * | Xcode project cleanup/setup.Thomas Van Lenten2016-06-296-43/+11
| | | | | | | | | | | | | | | | | | | | | | | | - Correct some cases sources were compiled into the static lib and the tests. - Enable Xcodes code coverage support on the unittests. We aren't complete on coverage, but having the data always there should make it easier to chip away at this going forward. - Drop method in tests that isn't used, wire up a validator in another test.
* | | Fix windows build.Feng Xiao2016-06-302-7/+14
| | | | | | | | | | | | Change-Id: Ibf7d1df850f4b497303f9f617751be98327898ef
* | | Updated failure_list_java.txt to remove tests that now passAdam Cozzette2016-06-301-2/+0
| | |
* | | Fixed string formatting in text_format.py to be Python2.6-compatibleAdam Cozzette2016-06-301-2/+2
| | | | | | | | | | | | | | | | | | In Python 2.6 the positional argument specifiers are apparently required to be explicitly specified: http://stackoverflow.com/questions/10054122/valueerror-zero-length-field-name-in-format-python
* | | Integrated internal changes from GoogleAdam Cozzette2016-06-29216-5406/+13676
|/ / | | | | | | This includes all internal changes from around May 20 to now.
* | Validate the tag numbers when parsing. (#1725)Thomas Van Lenten2016-06-295-4/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was a twist code path (that some times showed up due to what happened to be in memory in failure cases), that would cast a bogus wire type into the enum, and then fall through switch statements. Resolve this by validating all wire types when parsing tags and throwing the error at that point so it can't enter the system. As added safety, stick in a few asserts for apis that get passed tags to ensure they also are only seeing valid data. Bonus: Tweak the parsing loop to skip some work when we get the end marker (zero tag) instead of still looping through all the fields.
* | Merge pull request #1720 from thomasvl/issue_1716Thomas Van Lenten2016-06-283-22/+72
|\ \ | |/ |/| Fix GPBGetMessage{Repeated,Map}Field()
| * Fix GPBGetMessage{Repeated,Map}Field()Thomas Van Lenten2016-06-273-22/+72
| | | | | | | | | | | | | | - Correct impl by using helpers the message wiring does. - Add unittests. Fixes https://github.com/google/protobuf/issues/1716
* | Adding conditional compiler symbol to support .NET 3.5 (#1713)detlevschwabe2016-06-287-5/+36
| | | | | | | | * Adding condition compiler symbol to support .NET 3.5
* | Merge pull request #1719 from esteluk/patch-1Thomas Van Lenten2016-06-271-1/+1
|\ \ | |/ |/| Fix Objective-C generator option readme typo
| * Fix Objective-C generator option typoNathan Wong2016-06-271-1/+1
|/ | | Looks like this was changed during #1683, but missed in the README :)
* Merge pull request #1714 from dnkoutso/masterThomas Van Lenten2016-06-263-0/+26
|\ | | | | Get value from text format name in GPBEnumDescriptor
| * Get value from text format name in GPBEnumDescriptorDimitris Koutsogiorgas2016-06-263-0/+26
| |
* | Merge pull request #1705 from haberman/revjsverJoshua Haberman2016-06-231-7/+9
|\ \ | | | | | | Updates to package.json: rev version, add WKT, and recateogrize dependencies
| * | JS package.json: Added author and updated Closure Library version.Josh Haberman2016-06-231-3/+5
| | |
| * | Moved all dependencies to devDependencies.Josh Haberman2016-06-221-6/+5
| | | | | | | | | | | | | | | The runtime doesn't depend on anything, so these packages should always have been in devDependencies.
| * | Add "google" to package.json "files" for WKT.Josh Haberman2016-06-221-0/+1
| | |
| * | Create patch release for JS to include WKT.Josh Haberman2016-06-221-1/+1
| | |
* | | Merge pull request #1707 from jskeet/format-valueJan Tattermusch2016-06-232-25/+58
|\ \ \ | |/ / |/| | Expose JsonFormatter.WriteValue.
| * | Expose JsonFormatter.WriteValue.Jon Skeet2016-06-232-25/+58
|/ / | | | | | | | | This isn't useful to most users, but can be handy in advanced use cases, as requested in #1465.
* | Merge pull request #1700 from jskeet/orderingJon Skeet2016-06-211-3/+4
|\ \ | | | | | | Remove ordering guarantees in the MapField documentation
| * | Remove ordering guarantees in the MapField documentationJon Skeet2016-06-211-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | This doesn't currently change the ordering in the implementation, but allows us to do so in the future. We also need to change https://developers.google.com/protocol-buffers/docs/reference/csharp-generated#singular which states "Finally, unlike Dictionary<TKey, TValue>, MapField<TKey, TValue> preserves insertion order of entries." (We can just remove that sentence, I think.)
* | | Rename methods to avoid ObjC KVC collisions. (#1699)Thomas Van Lenten2016-06-2114-7325/+7448
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Note: Breaking API change on the Dictionary classes. The numeric value classes were using "Value" in the naming, but this silently collided with the KVC category on NSObject; meaning KVC code could break up a keypath and call these selectors with the wrong types leading to crashes (even though the code all would compile cleanly). - Rename the methods to use the "type" instead of literal "Value". - Update all the impls and tests. - Enable the warning that will catch issues like this in the future. Fixes https://github.com/google/protobuf/issues/1616
* | Adds destination flag to xcodebuild to avoid possible flake errors (#1697)Sergio Campamá2016-06-201-1/+6
| | | | | | | | Adds destination flag to xcodebuild to avoid possible flake errors
* | Merge pull request #1666 from yeswalrus/cmake-prerelease-examplesFeng Xiao2016-06-172-15/+22
|\ \ | | | | | | CMake Fix up prerelease version handling
| * | Fix prerelease version matching to be more consistent with the find_package ↵Walter Gray2016-06-092-15/+22
| | | | | | | | | | | | arguments.
* | | Merge pull request #1696 from haberman/jswktJoshua Haberman2016-06-172-5/+46
|\ \ \ | | | | | | | | JS: import well-known types from google-protobuf package.
| * | | Fix tests for CommonJS.Josh Haberman2016-06-161-2/+11
| | | |
| * | | Added plugin.proto to well-known types for JS.Josh Haberman2016-06-161-0/+1
| | | |
| * | | Bugfix: base the require logic on the file being required.Josh Haberman2016-06-161-4/+4
| | | |
| * | | JS: import well-known types from google-protobuf package.Josh Haberman2016-06-162-2/+33
| | | |
* | | | Add new generation option for using proto sources from other frameworks.Thomas Van Lenten2016-06-175-127/+348
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Better docs in the generator for the different options that can be passed during an invoke of protoc. - Add named_framework_to_proto_path_mappings_path to pass the path to a file containing mappings of frameworks for different proto files. - Update the generation to use the mapping to change the #import directives it creates. Note: the changes in helpers is mostly moving code within the fine, and then a small change to expose the parsing so a passed on class can consume the line. Fixes https://github.com/google/protobuf/issues/1457
* | | | Merge pull request #1692 from vjpai/friendlessFeng Xiao2016-06-161-4/+7
|\ \ \ \ | | | | | | | | | | Workaround for gcc-4.4 incompatibility
| * | | | Address review comments on function namevjpai2016-06-161-2/+2
| | | | |
| * | | | Remove a friend-class template that is only used for theVijay Pai2016-06-151-4/+7
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | constructor, and instead create an _internal_only getter that gets the needed information. This is a workaround for a deficiency in gcc-4.4 that does not properly support templated friend classes.
* | | | Merge pull request #1658 from yeswalrus/cmake-fixup-moduleFeng Xiao2016-06-151-24/+9
|\ \ \ \ | | | | | | | | | | Cleanup protobuf-module.cmake.in
| * | | | Removed handling for ALIASED targets since they are unused.Walter Gray2016-06-071-24/+9
| | |/ / | |/| |
* | | | Merge pull request #1665 from yeswalrus/cmake-package-requirementsFeng Xiao2016-06-153-12/+23
|\ \ \ \ | |_|/ / |/| | | CMake package requirements
| * | | make protobuf_MSVC_STATIC_RUNTIME a dependent option to reflect it's use.Walter Gray2016-06-091-1/+3
| | | |
| * | | Save the relevant options used to create a package, allow users to reject ↵Walter Gray2016-06-092-11/+20
| |/ / | | | | | | | | | packages based on them.
* | | Merge pull request #1683 from thomasvl/third_party_frameworkThomas Van Lenten2016-06-155-10/+54
|\ \ \ | | | | | | | | Add support for generation sources into a framework.