aboutsummaryrefslogtreecommitdiff
path: root/src/google
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | Merge pull request #1349 from gvaish/masterJon Skeet2016-04-0530-84/+175
|\ \ \ \ \ \ | | | | | | | | | | | | | | Added CLI option internal_access for types (C#)
| * | | | | | Added access_level for typesGaurav Vaish2016-04-0530-84/+175
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * `csharp_options`: Added `Options` to encapsulate generator options. Supported options for now - file_extension, base_namespace * `{Blah}Generator`: Now accept `Options*` as parameter to constructor * `csharp_generator.cc`: Parse and populate options * `Makefile.am`: Added `csharp_options.h` * `extract_includes.bat.in`: Added `csharp_options.h` Refactoring code to two commits. This is the first commit
* / | | | | Added deprecated option handling for objective-c generatorOleg Vereshko2016-04-045-11/+41
|/ / / / /
* | | / / Replace #include <iostream> with #include <ostream>Adam Michalik2016-03-301-1/+1
| |_|/ / |/| | | | | | | | | | | | | | | iostream is not actually necessary here, and it introduces unnecessary static initializers.
* | | | Use the T() instead of NULL for the default value.Jisi Liu2016-03-291-1/+1
| | | | | | | | | | | | | | | | | | | | The template can be specialized on primitives, e.g. double, where converting NULL will trigger a warning.
* | | | Update coded_stream.htopillar2016-03-221-1/+1
| | | | | | | | | | | | fix warning treated as error prevents building on 64-bit windows.
* | | | Shrink ObjC overhead (generated size and some runtime sizes)Thomas Van Lenten2016-03-1716-276/+357
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | NOTE: This is a binary breaking change as structure sizes have changed size and/or order. - Drop capturing field options, no other options were captured and other mobile targeted languages don't try to capture this sort information (saved 8 bytes for every field defined (in static data and again in field descriptor instance size data). - No longer generate/compile in the messages/enums in descriptor.proto. If developers need it, they should generate it and compile it in. Reduced the overhead of the core library. - Compute the number of has_bits actually needs to avoid over reserving. - Let the boolean single fields store via a has_bit to avoid storage, makes the common cases of the instance size smaller. - Reorder some flags and down size the enums to contain the bits needed. - Reorder the items in the structures to manually ensure they are are packed better (especially when generating 64bit code - 8 bytes for every field, 16 bytes for every extension, instance sizes 8 bytes also). - Split off the structure initialization so when the default is zero, the generated static storage doesn't need to reserve the space. This is batched at the message level, so all the fields for the message have to have zero defaults to get the saves. By definition all proto3 syntax files fall into this case but it also saves space for the proto2 that use the standard defaults. (saves 8 bytes of static data for every field that had a zero default) - Don't track the enums defined by a message. Nothing in the runtime needs it and it was just generation and runtime overhead. (saves 8 bytes per enum) - Ensure EnumDescriptors are started up threadsafe in all cases. - Split some of the Descriptor initialization into multiple methods so the generated code isn't padded with lots of zero/nil args. - Change how oneof info is feed to the runtime enabling us to generate less static data (8 bytes saved per oneof for 64bit). - Change how enum value informat is capture to pack the data and only decode it if it ends up being needed. Avoids padding issues causing bloat of 64bit, and removes the needs for extra pointers in addition to the data (just the data and one pointer now).
* | | | Merge pull request #1291 from sergiocampama/develThomas Van Lenten2016-03-141-1/+2
|\ \ \ \ | | | | | | | | | | Adds more information to Objective C error.
| * | | | Adds more information to Objective C error when the expected ↵Sergio Campama2016-03-031-1/+2
| | |/ / | |/| | | | | | | | | | objc_class_prefix option is missing.
* | | | Fixing compilation error when building with emscripten.Ben Vanik2016-03-111-1/+3
| | | | | | | | | | | | This change was previously done in //third_party in CL 108656107 but never made it to the open source project and was overwritten in an update.
* | | | Merge pull request #1299 from tatraian/masterFeng Xiao2016-03-081-1/+5
|\ \ \ \ | | | | | | | | | | Fix compiling clang/libc++ builds. (Issue: #1266)
| * | | | Comment has been added to fix (issue #1266)Antal Tátrai2016-03-081-0/+4
| | | | |
| * | | | Fix compiling clang/libc++ builds. (Issue: #1266)Antal Tátrai2016-03-051-1/+1
| | | | |
* | | | | Fix up handing of fields with leading names that should be all caps.Thomas Van Lenten2016-03-081-6/+9
| | | | | | | | | | | | | | | | | | | | Add a compile test to confirm things are working as expected.
* | | | | HeaderDoc support in the library and generated sourcesThomas Van Lenten2016-03-077-20/+33
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | - Convert most of the core library headers over to HeaderDoc format. - Switch the generated comments over to HeaderDoc. - Create GPBCodedOutputStream_PackagePrivate and move some things into there that should be more internal.
* / / / Generate well-known types in Ruby extension and prune unneeded proto2 ↵Josh Haberman2016-03-031-11/+73
|/ / / | | | | | | | | | dependencies.
* | | Merge pull request #1260 from legrosbuffle/masterPaul Yang2016-03-021-2/+4
|\ \ \ | | | | | | | | Make cpp generated enum constants constexpr when Options::proto_h is specified
| * | | Make cpp generated enum constants constexpr when Options::proto_h isClement Courbet2016-02-191-2/+4
| | | | | | | | | | | | | | | | specified.
* | | | Fix CommonJS relative require generation, and test itmurgatroid992016-02-241-1/+15
| | | |
* | | | Merge pull request #804 from bsilver8192/masterFeng Xiao2016-02-223-0/+160
|\ \ \ \ | | | | | | | | | | Add atomics support for 32-bit PPC.
| * | | | Add atomics support for 32-bit PPC.Brian Silverman2015-09-093-0/+160
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was originally submitted as a patch on the Google Code issue tracker, but the original author didn't submit a pull request. They did, however, give permission for somebody else to do so. I cleaned up a few things myself and am now submitting it. I can't find any record of what the original patch was, but there is a copy of the discussion at <https://groups.google.com/forum/#!topic/protobuf/SMwF5fYHIi0>. Fixes google/protobuf#581
* | | | | Merge pull request #1215 from haberman/commonjsJoshua Haberman2016-02-192-57/+236
|\ \ \ \ \ | | | | | | | | | | | | Added support for CommonJS require()
| * | | | | Fixed definition of extensions, and added CommonJS tests to Travis.Josh Haberman2016-02-181-1/+8
| | | | | |
| * | | | | Fixed nested message scopes for CommonJS.Josh Haberman2016-02-181-7/+5
| | | | | |
| * | | | | CommonJS tests are now passing.Josh Haberman2016-02-181-1/+5
| | | | | |
| * | | | | WIP.Josh Haberman2016-02-182-57/+227
| | |/ / / | |/| | |
* / | | | Support ObjC Generic CollectionsThomas Van Lenten2016-02-1818-132/+237
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | - Extend GPB*ObjectDictionary to support generic syntax. - Update the generator to output generics so the enclosed type is exposed for compiler checks. - Use generics in a the public interfaces. - Update the generated sources that are checked in.
* | | | Merge pull request #260 from ejsd1989/issue-#242Joshua Haberman2016-02-162-4/+2
|\ \ \ \ | | | | | | | | | | Removes redundant null pointer checks checks
| * | | | Removal of null checkeissajamil@gmail.com2015-03-291-3/+1
| | | | | | | | | | | | | | | | | | | | | | | | | Removed the redundant check for NULL which is already handled by using delete
| * | | | Removal of null checkeissajamil@gmail.com2015-03-291-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | Removed the redundant check for NULL which is already correctly handled by used the delete
* | | | | Merge pull request #1144 from dongjoon-hyun/remove_redundant_theJoshua Haberman2016-02-114-5/+5
|\ \ \ \ \ | | | | | | | | | | | | Remove redundant `the`.
| * | | | | Remove redundant `the` in comments.Dongjoon Hyun2016-02-034-5/+5
| | | | | |
* | | | | | Merge pull request #1194 from Photonios/masterJoshua Haberman2016-02-116-37/+37
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fixed unused parameters and extra ';'
| * | | | | | Fixed unused parameters and extra ';'Swen Kooij2016-01-286-37/+37
| | | | | | |
* | | | | | | Merge pull request #1217 from jhump/jh/make-warning-usefulJoshua Haberman2016-02-111-3/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | make 'no syntax' warning useful by showing which file
| * | | | | | | make 'no syntax' warning useful by showing which fileJoshua Humphries2016-02-061-3/+3
| | | | | | | |
* | | | | | | | Merge pull request #1185 from kkm000/icl-fixJoshua Haberman2016-02-102-4/+4
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | Fix Intel compiler warnings in include files on Windows
| * | | | | | | Fix Intel compiler warnings in include files on Windowskkm2016-01-272-4/+4
| |/ / / / / /
* | | | | | | Rename Preconditions to ProtoPreconditionsJon Skeet2016-02-041-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | (Generated code changes in next commit.)
* | | | | | | Rename GeneratedCodeInfo to GeneratedClrTypeInfoJon Skeet2016-02-041-4/+4
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Recently, descriptor.proto gained a GeneratedCodeInfo message, which means the generated code conflicts with our type. Unfortunately this affects codegen as well, although this is a part of the public API which is very unlikely to affect hand-written code. Generated code changes in next commit.
* | | | | | Integrate from google internal.Jisi Liu2016-01-297-61/+1432
|/ / / / / | | | | | | | | | | | | | | | | | | | | Java files are moved to un-do the hack in the prevous commit, which moved the java files to the original position for integration.
* | | | | Avoid upcasting uninitialized pointersBrian Silverman2016-01-201-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes google/protobuf#693 msan flags this as being undefined behavior. I think it's triggering because the compiler has to insert a branch to avoid changing the pointer's value if it starts out NULL. I can't figure out if this is actually undefined behavior or not, but it definitely seems to be a gray area of the standard which is best avoided.
* | | | | Merge pull request #789 from motahan/solaris64_fixFeng Xiao2016-01-203-2/+7
|\ \ \ \ \ | | | | | | | | | | | | Fixing compile errors on Solaris in 64-bit mode
| * | | | | Fixing compile errors on Solaris in 64-bit modeMohamed El-Tahan2015-09-023-2/+7
| | | | | |
* | | | | | Fix compiler warning from repeated_field.hJohn Burke2016-01-191-1/+1
| | | | | |
* | | | | | Merge pull request #896 from ↵Feng Xiao2016-01-162-10/+9
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | jhump/jh/fix-ioexception-vs-invalidprotobuf-exception throw IOException instead of InvalidProtocolBufferException when appropriate
| * | | | | | fully qualify invocations of static methods on GeneratedMessageJoshua Humphries2016-01-151-6/+6
| | | | | | |
| * | | | | | throw IOException instead of InvalidProtocolBufferException when appropriateJoshua Humphries2016-01-152-10/+9
| | | | | | |
* | | | | | | Make sure thatJon Skeet2016-01-151-0/+2
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "valueField": null is parsed appropriately, i.e. that it remembers that the field is set.
* | | | | | Prohibit null values in map fieldsJon Skeet2016-01-111-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On deserialization, missing values for message types are replaced with a "default" message.