aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* 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
| * | | Merge pull request #601 from anandolee/masterJie Luo2015-07-172-0/+35
| |\ \ \ | | |_|/ | |/| | ignore UTF-8 BOM
| | * | ignore UTF-8 BOM if it is in the begining of a proto fileJie Luo2015-07-162-0/+35
| | |/
| * / Install missing headersRob Earhart2015-07-141-0/+3
| |/ | | | | | | | | | | Install google/protobuf/stubs/status.h, and google/protobuf/stubs/stringpiece.h -- these are required in order to include google/protobuf/util/type_resolver.h. Install google/protobuf/stubs/bytestream.h -- this is required in order to include google/protobuf/util/json_util.h.
* | Merge pull request #628 from jtattermusch/csharp_addressbook_updateJan Tattermusch2015-07-205-47/+47
|\ \ | | | | | | C# addressbook update
| * | update addressbook exampleJan Tattermusch2015-07-203-6/+6
| | |
| * | regenerate Addressbook.csJan Tattermusch2015-07-201-39/+39
| | |
| * | update addressbook.protoJan Tattermusch2015-07-201-2/+2
|/ /
* | Merge pull request #624 from jskeet/wkt-jsonJan Tattermusch2015-07-203-1/+80
|\ \ | | | | | | First part of JSON formatting for well-known types.
| * | First part of JSON formatting for well-known types. I think we need a ↵Jon Skeet2015-07-203-1/+80
|/ / | | | | | | reflection API rethink before doing the rest.
* | Merge pull request #620 from google/jtattermusch-patch-2Jon Skeet2015-07-171-1/+1
|\ \ | | | | | | Fix typo in buildall.sh
| * | Fix typo in buildall.shJan Tattermusch2015-07-171-1/+1
|/ /
* | Merge pull request #616 from jskeet/renameJan Tattermusch2015-07-17101-142/+135
|\ \ | | | | | | First pass at the big rename from ProtocolBuffers to Google.Protobuf.
| * | Updated csharp_EXTRA_DIST for TravisJon Skeet2015-07-171-89/+90
| | |
| * | First pass at the big rename from ProtocolBuffers to Google.Protobuf.Jon Skeet2015-07-17100-53/+45
|/ / | | | | | | We'll see what I've missed when CI fails...
* | Merge pull request #611 from jskeet/csharp-wrappersJon Skeet2015-07-1724-129/+3516
|\ \ | | | | | | C# wrapper types
| * | Fixes from PR review.Jon Skeet2015-07-174-15/+89
| | |
| * | Make the map test actually test the code...Jon Skeet2015-07-161-0/+7
| | |
| * | First pass at wrapper types.Jon Skeet2015-07-1615-475/+927
| | | | | | | | | | | | | | | - We do still generate the message types, as otherwise reflection breaks, even though it doesn't actually use those types. - JSON handling hasn't been implemented yet
| * | Fixes to ByteString's equality handling.Jon Skeet2015-07-162-20/+41
| | |
| * | First part of implementing wrapper types. Not ready yet!Jon Skeet2015-07-1613-20/+2853
| | |
* | | Merge pull request #586 from jtattermusch/csharp_names_exportJan Tattermusch2015-07-167-5/+84
|\ \ \ | | | | | | | | Export c# naming routines publicly
| * | | export c# naming routines publiclyJan Tattermusch2015-07-107-5/+84
| | | |
* | | | Merge pull request #612 from jtattermusch/csharp_distcheck_fixJie Luo2015-07-161-248/+106
|\ \ \ \ | | | | | | | | | | Fix make distcheck
| * | | | updated csharp_EXTRA_DISTJan Tattermusch2015-07-161-248/+106
| | |/ / | |/| |
* | | | Merge pull request #607 from jtattermusch/csharp_appveyorJan Tattermusch2015-07-162-7/+34
|\ \ \ \ | |/ / / |/| | | Run C# tests on Appveyor