aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #659 from jskeet/fix-frozenJon Skeet2015-07-3021-327/+3
|\ | | | | Remove a few hangers-on from frozenness.
| * Generated code from previous commit.Jon Skeet2015-07-3018-315/+0
| |
| * More freezing tidy-up; generated code in next commit.Jon Skeet2015-07-302-5/+2
| |
| * Minor bits of left-over frozenness.Jon Skeet2015-07-302-7/+1
|/
* Merge pull request #654 from jtattermusch/csharp_hide_freezeJon Skeet2015-07-3035-1498/+8
|\ | | | | Remove the C# Freeze API
| * regenerate codeJan Tattermusch2015-07-2918-1192/+0
| |
| * remove the freeze APIJan Tattermusch2015-07-2917-306/+8
| |
* | Merge pull request #655 from jtattermusch/csharp_expose_more_info_about_serviceJon Skeet2015-07-301-0/+10
|\ \ | |/ |/| Expose IsClientStreaming and IsServerStreaming in MethodDescriptor
| * add IsClientStreaming and IsServerStreaming to MethodDescriptorJan Tattermusch2015-07-291-0/+10
|/
* Merge pull request #638 from jskeet/portableJan Tattermusch2015-07-2919-21/+457
|\ | | | | First attempt at using profile 259 for Google.Protobuf.
| * Update the readme file to indicate supported platformsJon Skeet2015-07-281-0/+11
| |
| * Humbug - previous commit didn't include project file changes :(Jon Skeet2015-07-281-0/+2
| |
| * Tweaks to Profile259 supportJon Skeet2015-07-286-11/+303
| | | | | | | | | | | | - Fix nupec paths - Remove an obsolete part of the JSON build - Add documentation and tests to reflection extension methods, and improve implementations
| * First attempt at using profile 259 for Google.Protobuf.Jon Skeet2015-07-2714-15/+146
|/ | | | | | | | This requires .NET 4.5, and there are a few compatibility changes required around reflection. Creating a PR from this to see how our CI systems handle it. Will want to add more documentation, validation and probably tests before merging. This is in aid of issue #590.
* Merge pull request #641 from jtattermusch/csharp_descriptor_databaseJon Skeet2015-07-252-4/+21
|\ | | | | Expose original binary data for file descriptor
| * expose original binary data for filedescriptorJan Tattermusch2015-07-242-4/+21
|/
* Merge pull request #634 from jskeet/reflection2Jon Skeet2015-07-239-199/+342
|\ | | | | Reflection part 2 - for discussion
| * Implemented Jan's suggestion of FieldCollection, replacing ↵Jon Skeet2015-07-226-119/+128
| | | | | | | | | | | | | | | | | | | | FieldAccessorCollection. I think Jan was actually suggesting keeping both, but that feels redundant to me. The test diff is misleading here IMO, because I wouldn't expect real code using reflection to use several accessors one after another like this, unless it was within a loop. Evidence to the contrary would be welcome :) This change also incidentally goes part way to fixing the issue of the JSON formatter not writing out the fields in field number order - with this change, it does except for oneofs, which we can fix in a follow-up change. I haven't actually added a test with a message with fields deliberately out of order - I'm happy to do so though. It feels like it would make sense to be in google/src/protobuf, but it's not entirely clear what the rules of engagement are for adding new messages there. (unittest_proto3.proto?)
| * Added newlinesJon Skeet2015-07-222-2/+2
| |
| * First pass at making field access simpler.Jon Skeet2015-07-227-175/+309
|/ | | | This is definitely not ready to ship - I'm "troubled" by the disconnect between a list of fields in declaration order, and a mapping of field accessors by field number/name. Discussion required, but I find that easier when we've got code to look at :)
* Merge pull request #625 from jskeet/reflection-refactorJan Tattermusch2015-07-2248-1274/+772
|\ | | | | Reflection refactor
| * Fix typo in Makefile.amJon Skeet2015-07-221-1/+1
| |
| * Fix attribute mistake and regenerate code.Jon Skeet2015-07-224-10/+0
| |
| * Fix file lists for C#Jon Skeet2015-07-221-1/+1
| |
| * Generated code for previous commit.Jon Skeet2015-07-2217-511/+129
| |
| * Remove the usage of attributes for field/method discovery.Jon Skeet2015-07-2222-405/+243
| | | | | | | | Instead, introduce GeneratedCodeInfo which passes in what we need, and adjust the codegen to take account of this.
| * Fix comment typoJon Skeet2015-07-211-1/+1
| |
| * Generated code changes following previous commit.Jon Skeet2015-07-2118-728/+686
| |
| * Revamp to reflection.Jon Skeet2015-07-2135-278/+372
|/ | | | | | | | | | | | | | | | | Changes in brief: 1. Descriptor is now the entry point for all reflection. 2. IReflectedMessage has gone; there's now a Descriptor property in IMessage, which is explicitly implemented (due to the static property). 3. FieldAccessorTable has gone away 4. IFieldAccessor and OneofFieldAccessor still exist; we *could* put the functionality straight into FieldDescriptor and OneofDescriptor... I'm unsure about that. 5. There's a temporary property MessageDescriptor.FieldAccessorsByFieldNumber to make the test changes small - we probably want this to go away 6. Discovery for delegates is now via attributes applied to properties and the Clear method of a oneof I'm happy with 1-3. 4 I'm unsure about - feedback welcome. 5 will go away 6 I'm unsure about, both in design and implementation. Should we have a ProtobufMessageAttribute too? Should we find all the relevant attributes in MessageDescriptor and pass them down, to avoid an O(N^2) scenario? Generated code changes coming in the next commit.
* Merge pull request #629 from google/masterJon Skeet2015-07-2136-2000/+2393
|\ | | | | Integrate changes from upstream/master into csharp-experimental
| * Merge pull request #627 from jtattermusch/addressbook_proto3Jan Tattermusch2015-07-207-26/+26
| |\ | | | | | | Update addressbook.proto and examples code to proto3
| | * rename persons to peopleJan Tattermusch2015-07-207-8/+8
| | |
| | * use Google.Protobuf namespace for C#Jan Tattermusch2015-07-201-1/+1
| | |
| | * fixed java exampleJan Tattermusch2015-07-202-5/+5
| | |
| | * fix python exampleJan Tattermusch2015-07-202-5/+5
| | |
| | * fix C++ exampleJan Tattermusch2015-07-202-7/+7
| | |
| | * update addressbook.proto to proto3Jan Tattermusch2015-07-201-8/+8
| | |
| * | Merge pull request #621 from nico/staticassJisi Liu2015-07-201-2/+4
| |\ \ | | |/ | |/| Let GOOGLE_COMPILE_ASSERT use static_assert if available.
| | * Let GOOGLE_COMPILE_ASSERT use static_assert if available.Nico Weber2015-07-171-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The motivation is that gcc 4.8+ and clang trunk warn on unused local typedefs, which COMPILE_ASSERT adds. After this change, the warning will be happy at least in C++11 builds. static_assert also produces a slighly nicer diagnostic than the typedef method. https://github.com/google/re2/commit/eb93e8bc43ac8d05322fb3e9fc885898ad924f8a did the same change in re2.
| * | Merge pull request #604 from haberman/ruby-conformanceJoshua Haberman2015-07-2016-579/+694
| |\ \ | | | | | | | | Added Ruby to conformance tests.
| | * | Fixed lint errors and responded to CR comments.Josh Haberman2015-07-172-27/+30
| | | | | | | | | | | | | | | | Change-Id: If7b1cc0f03f609a7f43ddafc8509b44207c60910
| | * | Exclude JRuby from conformance tests for now.Josh Haberman2015-07-171-7/+16
| | | | | | | | | | | | | | | | Change-Id: Id008ebac5159f773e1bde8b85acb2626cbd16de8
| | * | Added Ruby to conformance tests.Josh Haberman2015-07-1616-575/+678
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This involved fixing a few important bugs in the Ruby implementation -- mostly cases of mixing upb field types and descriptor types (upb field types do not distinguish between int/sint/fixed/sfixed like descriptor types do). Also added protobuf-specific exceptions so parse errors can be caught specifically. Change-Id: Ib49d3db976900b2c6f3455c8b88af52cfb86e036
| * | | Merge pull request #622 from pherl/splitheadersJisi Liu2015-07-172-0/+2
| |\ \ \ | | |_|/ | |/| | Addming missing includes in stubs headers.
| | * | Addming missing includes in stubs headers.Jisi Liu2015-07-172-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | This is to prepare removing those additional headers in common.h Change-Id: I707d6d8eaf193eb28195a827c7fec738b968ab3e
| * | | Merge pull request #613 from pherl/splitheadersJisi Liu2015-07-1710-1404/+1640
| |\| | | | | | | | | | Split up common.h headers
| | * | update header list.Jisi Liu2015-07-171-0/+3
| | | | | | | | | | | | | | | | Change-Id: I75459b8b6562c3ac58fa4b1e4513accdda43ad64
| | * | Merge branch 'master' of github.com:google/protobuf into splitheaderJisi Liu2015-07-171-0/+3
| | |\ \ | | | | | | | | | | | | | | | Change-Id: I5f6cbc2430200e3751ad754cc7ae604338db6f91
| | * | | Update make file for extra headers.Jisi Liu2015-07-172-0/+12
| | | | | | | | | | | | | | | | | | | | Change-Id: Ifbc415755266ba3d0ffa5661fcd7c7d2e3b32a22
| | * | | Split up common.h headersJisi Liu2015-07-168-1404/+1625
| | | |/ | | |/| | | | | | | | | Change-Id: I223783111d743aa5193bf70fa1b9b54c7b4389c3