aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Initial value in generated code cannot be used by c extension. (#3367)Paul Yang2017-07-145-2/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the generated code of previous versions, each php field is given an initial value. In c extension, it was assumed that the field order in the generated code is consistent with upb fields order, so that the correct initial value can be bound to the correct upb field. However, this may not be true. The order of fields in generated code is decided by proto compiler, while the order of upb fields is decided by the hash function used in c extension. This PR fixes the issue by reset the initial value at runtime.
* | | | | removes an accidental newline in printing for the php generator (#3333)Brent Shaffer2017-07-131-1/+1
| | | | |
* | | | | Add map iterator for c extension (#3350)Paul Yang2017-07-136-5/+157
| | | | |
* | | | | Add space between arrow and casted type (#3353)Paul Yang2017-07-133-27/+27
| | | | |
* | | | | Merge pull request #3327 from htuch/fix-3322Feng Xiao2017-07-131-7/+0
|\ \ \ \ \ | | | | | | | | | | | | Use consistent hash across NDEBUG/!NDEBUG builds.
| * | | | | Use consistent hash across NDEBUG/!NDEBUG builds.Harvey Tuch2017-07-071-7/+0
| | | | | | | | | | | | | | | | | | | | | | | | Fixes #3322.
* | | | | | Fix repository URL in C# project fileJon Skeet2017-07-131-1/+1
| |_|/ / / |/| | | |
* | | | | C#: Implement IReadOnlyDictionary<K,V> in MapField<K,V>Jean-Rémy Bancel2017-07-122-0/+27
| |_|/ / |/| | |
* | | | Fix missing std::tr1::hash on GCC 4.1 (#2907)Matt Hauck2017-07-101-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Rather than crashing on use (doh!) better to just declare this platform is missing a proper hash_map/hash_set implementation and use the std::map/std::set emulation. Fixes regression introduced by #1913
* | | | Add std:: namespace prefix to set and map (#3332)tanderson-google2017-07-0939-84/+82
| | | | | | | | | | | | * Remove using std::{set,map}
* | | | Uncomment php tests (#3301)Paul Yang2017-07-092-416/+416
|/ / /
* | | Merge pull request #2482 from andreaseger/fix_ruby_timestamp_accuracyAdam Cozzette2017-07-062-3/+11
|\ \ \ | | | | | | | | [Ruby] fix floating point accuracy problem in Timestamp#to_f
| * | | change test for nanosecond accurate timestampsAndreas Eger2017-05-131-2/+5
| | | |
| * | | fix floating point accuracy problem in Timestamp#to_fAndreas Eger2017-05-132-3/+8
| | | | | | | | | | | | | | | | | | | | `.quo` return the most exact devision which fixes accuracy problems for the timestamp coercion
* | | | add Grpc Protobuf validation (#3311)MaDuo2017-07-051-0/+1
| | | | | | | | | | | | * add Grpc Protobuf validation
* | | | Merge pull request #3315 from thomasvl/mutate_unknownsThomas Van Lenten2017-07-052-2/+3
|\ \ \ \ | | | | | | | | | | Expose the initializer for unknown fields.
| * | | | Expose the initializer for unknown fields.Thomas Van Lenten2017-07-052-2/+3
|/ / / /
* | | | Ensure leaveOpen is true when writing to a bufferJon Skeet2017-07-045-12/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Note that the compatibility tests have had to cahnge as well, to cope with internal changes. (The test project has access to internals in the main project.) Fixes #3209.
* | | | Make Any easier to work with in C#Jon Skeet2017-07-032-7/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | - Add a TryUnpack method which doesn't throw if the type is wrong - Make GetTypeName public for easier determination of the message type Fixes #3294.
* | | | Add json encode/decode for php. (#3226)Paul Yang2017-06-3036-2274/+3118
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add json encode/decode for php. * Fix php conformance test on 32-bit machines. * Fix conformance test for c extension. * Fix comments
* | | | Merge pull request #3287 from sergiocampama/initializedThomas Van Lenten2017-06-291-1/+1
|\ \ \ \ | | | | | | | | | | Add initialized as a reserved keyword as that's the actual property name
| * | | | Add initialized as a reserved keyword as that's the actual property nameSergio Campama2017-06-291-1/+1
|/ / / /
* | | | Add scripts to build python wheel for linux. (#2693)Paul Yang2017-06-285-0/+121
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add scripts to build python wheel for linux. Followed PEP513. * Fix dist check for new added files. * Update README for usage example.
* | | | adds PHPDoc @return and @param for getters and setters respectively (#3131)Brent Shaffer2017-06-2736-925/+1220
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * adds PHPDoc @return and @param for getters and setters respectively * addresses changes in PR review * adds documentation tests * Update php_generator: - Prepend \ to names where required - Remove <pre> tags - Update protobuf field comments * Updates class files with the protobuf changes * Addresses review comments * removes Protobuf Type line from PHP generated classes * fixes phpdoc test * adds array types to phpdoc
* | | | Merge pull request #3084 from lukaszx0/patch-1Feng Xiao2017-06-261-1/+1
|\ \ \ \ | | | | | | | | | | Workaround the docker bug when compiling artifacts
| * | | | Workaround the docker bug when compiling artifactsŁukasz Strzałkowski2017-05-111-1/+1
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a workaround (https://github.com/moby/moby/issues/10180#issuecomment-190429512) the docker issue (https://github.com/moby/moby/issues/10180) which breaks protoc-artifacts build process with following error ```Rpmdb checksum is invalid: dCDPT(pkg checksums): devtoolset-1.1-elfutils.x86_64 0:0.154-6.el6 - u The command '/bin/sh -c yum clean all && yum install -y devtoolset-1.1 devtoolset-1.1-libstdc++-devel devtoolset-1.1-libstdc++-devel.i686' returned a non-zero code: 1``` https://github.com/moby/moby/issues/10180#issuecomment-190429512
* | | | Merge pull request #3264 from TeBoring/php-bugPaul Yang2017-06-2434-672/+847
|\ \ \ \ | | | | | | | | | | Enusre public header and generated code have no implicit converion.
| * | | | Fix more implicit type conversions in public headers and generated code.Bo Yang2017-06-2426-449/+606
| | | | |
| * | | | Enusre public header and generated code have no implicit converion.Bo Yang2017-06-2321-229/+247
| | | | |
* | | | | Merge pull request #2968 from ngg/cpp-proper-fwdFeng Xiao2017-06-234-107/+0
|\ \ \ \ \ | |/ / / / |/| | | | C++: Do not forward-declare dependencies in generated .h files
| * | | | C++: Do not forward-declare dependencies in generated .h filesGergely Nagy2017-06-124-107/+0
| | | | |
* | | | | Merge pull request #3266 from mbrickn/patch-1Feng Xiao2017-06-231-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Updated links to use https
| * | | | | Updated links to use httpsMaxwell Paul Brickner2017-06-231-1/+1
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Howdy! I just updated some links to use https instead of http. Thanks! ^ _ ^
* | | | | Merge pull request #3261 from thomasvl/super_oddcaseThomas Van Lenten2017-06-221-1/+1
|\ \ \ \ \ | | | | | | | | | | | | If we fail to get a descriptor just super the method resolving.
| * | | | | If we fail to get a descriptor just super the method resolving.Thomas Van Lenten2017-06-221-1/+1
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | This should never happen, but if someone is swizzling or do other hooking of methods, anything is possible, so this seems slighty safer than they returning NO.
* | | | | Switch to addEnumType to fix fatal error (#3225)Ryan Gordon2017-06-194-160/+235
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Switch to addEnumType to fix fatal error * Fixing more cases of HHVM incompatibility * Updating tests to be hhvm compatible * Fixing tests * Fixing merge * Don't use call_user_func, should hopefully fix tests * Fixing spelling * Fixing another misspelling in a test * Fixing placement of append and kvUpdate functions * Actually fix function helpers * Remove double addEnumType. How did this ever work? * Fixing a couple more tests * Only use the setter if the return value isn't an object
* | | | | Updated upb to fix JSON conformance issues. (#3206)Joshua Haberman2017-06-195-580/+493
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fixed a bunch of Ruby conformance errors. * Fixed some more Ruby conformance errors in JSON.
* | | | | Merge pull request #3243 from yjjnls/masterFeng Xiao2017-06-191-0/+2
|\ \ \ \ \ | | | | | | | | | | | | replenish missed header files in install step
| * | | | | replenish missed header files in install stepyjjnls2017-06-191-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | install 'generated_message_table_driven.h' and 'metadata_lite.h', they are included when compiling the files generated using protoc.exe 3.3.0
* | | | | | Merge pull request #3244 from thomasvl/complete_docsThomas Van Lenten2017-06-193-2/+25
|\ \ \ \ \ \ | | | | | | | | | | | | | | ObjC: Document the exceptions on some of the writing apis.
| * | | | | | ObjC: Document the exceptions on some of the writing apis.Thomas Van Lenten2017-06-193-2/+25
|/ / / / / /
* | | | | | Merge pull request #3240 from thomasvl/float_funThomas Van Lenten2017-06-193-1/+3
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Raise the number of digits used for floats.
| * | | | | Raise the number of digits used for floats.Thomas Van Lenten2017-06-163-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | About 1.5% of all IEEE754 single-precision numbers require nine decimal digits to represent accurately.
* | | | | | Remove inclusion of ext/json/php_json.h. (#3241)Jeff Ching2017-06-171-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | That implementation of json is not being used - this extension is using a json encoder/decoder provided by 'upb'.
* | | | | | Merge pull request #3237 from calder/patch-1Feng Xiao2017-06-161-2/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | Qualify string in java_options.h
| * | | | | | Qualify string in java_options.hCalder Coalson2017-06-161-2/+2
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Building the protobuf compiler with Bazel fails to resolve the unqualifed "string"s in java_options.h: ``` ERROR: .../external/com_google_protobuf/BUILD:248:1: C++ compilation of rule '@com_google_protobuf//:protoc_lib' failed: Process exited with status 1 [sandboxed]. In file included from external/com_google_protobuf/src/google/protobuf/compiler/java/java_context.h:41:0, from external/com_google_protobuf/src/google/protobuf/compiler/java/java_enum_field_lite.cc:38: external/com_google_protobuf/src/google/protobuf/compiler/java/java_options.h:59:3: error: 'string' does not name a type string annotation_list_file; ^ external/com_google_protobuf/src/google/protobuf/compiler/java/java_options.h:62:3: error: 'string' does not name a type string output_list_file; ^ Use --strategy=CppCompile=standalone to disable sandboxing for the failing actions. INFO: Elapsed time: 41.487s, Critical Path: 6.20s //interpreter:eval_test NO STATUS Executed 0 out of 1 test: 1 was skipped. ```
* | | | | | Merge pull request #3236 from buchgr/bazel-linksFeng Xiao2017-06-161-0/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | bazel: add bazel symlinks to .gitignore
| * | | | | | bazel: add bazel symlinks to .gitignoreJakob Buchgraber2017-06-161-0/+3
| |/ / / / /
* | | | | | Merge pull request #3235 from buchgr/java-targetFeng Xiao2017-06-161-1/+2
|\ \ \ \ \ \ | |/ / / / / |/| | | | | bazel: Make compiled jars java 6 binary compatible.
| * | | | | bazel: Make compiled jars java 6 binary compatible.Jakob Buchgraber2017-06-161-1/+2
|/ / / / / | | | | | | | | | | | | | | | See: https://github.com/bazelbuild/bazel/issues/3198