aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| | * | 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
| | * | | add C# build and tests to appveyorJan Tattermusch2015-07-152-7/+34
| | | | |
| * | | | Merge pull request #610 from jskeet/fix-enumeratorJon Skeet2015-07-162-63/+6
| |\ \ \ \ | | | | | | | | | | | | Remove the struct-based iterator for RepeatedField.
| | * | | | Remove the struct-based iterator for RepeatedField.Jon Skeet2015-07-162-63/+6
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | We don't use it in the runtime or generated code anywhere now, so the extra small performance boost isn't as critical, and it has some undesirable consequences. The tests have needed to change as iterator block enumerators don't throw when we might expect them to.
| * | | | Merge pull request #605 from jtattermusch/csharp_cleanupJon Skeet2015-07-166-10/+5
| |\ \ \ \ | | |/ / / | |/| | | Assorted nits from C# code.
| | * | | added comment for ByteString.CopyFromJan Tattermusch2015-07-151-0/+2
| | | | |
| | * | | C# cleanup - assorted nitsJan Tattermusch2015-07-155-10/+3
| |/ / /
| * | | Merge pull request #606 from jtattermusch/csharp-experimental-mergedJon Skeet2015-07-1639-430/+761
| |\ \ \ | | | | | | | | | | Update csharp-experimental with changes from upstream/master
| | * | | Merge remote-tracking branch 'upstream/master' into csharp-experimentalJan Tattermusch2015-07-1539-430/+761
| |/| | |
| * | | | Merge pull request #599 from jskeet/no-map-entriesJon Skeet2015-07-155-5779/+39
| |\ \ \ \ | | | | | | | | | | | | Don't generate types for map entry messages
| | * | | | Address requested change from code review.Jon Skeet2015-07-151-12/+12
| | | | | |
| | * | | | Don't create nested types (or field accessors) for map types.Jon Skeet2015-07-155-5773/+33
| |/ / / / | | | | | | | | | | | | | | | I'm sure I've implemented this before, but somehow it's been lost in a maze of twisty little branches, all alike.
| * | | | Merge pull request #594 from jskeet/csharp-wellknowntypesJan Tattermusch2015-07-1531-130/+4808
| |\ \ \ \ | | | | | | | | | | | | Introduce C# well-known types
| | * | | | Generated code for well-known typesJon Skeet2015-07-1417-113/+4763
| | | | | |
| | * | | | Generate the well-known types in C#Jon Skeet2015-07-1414-17/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This involves: - Specifying a namespace in each proto (including ones we'd previously missed) - Updating the generation script - Changing codegen to implement IReflectedMessage.Fields explicitly (a good thing anyway) - Changing reflection tests to take account of the explicit interface implementation Non-generated code in this commit; generated code to follow
| * | | | | Merge pull request #593 from jskeet/reflection-namespaceJon Skeet2015-07-1449-3355/+3331
| |\| | | | | | | | | | | | | | | | Change the reflection namespace in C#
| | * | | | Changing reflection namespace (part 4)Jon Skeet2015-07-148-11/+0
| | | | | | | | | | | | | | | | | | | | | | | | Update code already within Google.Protobuf.Reflection which was importing Google.Protobuf.DescriptorProtos
| | * | | | Changing reflection namespace (part 3)Jon Skeet2015-07-1412-252/+251
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change the C# namespace in descriptor.proto to Google.Protobuf.Reflection. This then means changing where the generated code lives, which means updating the project file... It also involves regenerating the C++ - which has updated the well-known types as well, for no terribly obvious reason...
| | * | | | Changing reflection namespace (part 2)Jon Skeet2015-07-147-487/+487
| | | | | | | | | | | | | | | | | | | | | | | | Generated code changes from previous commit
| | * | | | Changing reflection namespace (part 1)Jon Skeet2015-07-1432-2638/+2626
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Move types into Google.Protobuf.Reflection - Change codegen to reflect that in generated types Generated code changes coming in part 2
| * | | | Merge pull request #583 from jskeet/issue312Jon Skeet2015-07-142-3/+11
| |\ \ \ \ | | | | | | | | | | | | Pascal-case namespace automatically in C# codegen
| | * | | | Convert package name to PascalCase for C# namespaceJon Skeet2015-07-142-3/+11
| |/ / / / | | | | | | | | | | | | | | | Fixes issue 312.
| * | | | Merge pull request #582 from jskeet/csharp-jsonJon Skeet2015-07-1421-168/+1494
| |\ \ \ \ | | | | | | | | | | | | JSON formatting in C#
| | * | | | Changes suggested during review.Jon Skeet2015-07-142-27/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Remove the indexers in FieldAccessorTable - Add a TODO for field ordering in oneof
| | * | | | Fixes to JsonFormatterJon Skeet2015-07-102-11/+112
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Handle oneof properly - Omit unknown enum values
| | * | | | Generated code changes for previous submitJon Skeet2015-07-107-113/+113
| | | | | |
| | * | | | Oneof reflection support. (Generated code changes in next commit.)Jon Skeet2015-07-105-35/+87
| | | | | |
| | * | | | Use the new JsonFormatter to implement ToString on generated messages.Jon Skeet2015-07-1010-6/+459
| | | | | |
| | * | | | Initial implementation of JSON formattingJon Skeet2015-07-107-3/+746
| | | |/ / | | |/| | | | | | | | | | | | | | | | | | | | | | - No parsing - Reflection based, so not hugely efficient - No line breaks or indentation
| * | | | Merge pull request #587 from jtattermusch/nuget_pkgJon Skeet2015-07-124-13/+41
| |\ \ \ \ | | |/ / / | |/| | | enable building nuget package
| | * | | Update Google.Protobuf.nuspecJan Tattermusch2015-07-121-1/+1
| | | | |
| | * | | enable building nuget packageJan Tattermusch2015-07-104-13/+41
| |/ / /
| * | | Merge pull request #566 from jskeet/csharp-reflectionJon Skeet2015-07-1052-1960/+2699
| |\ \ \ | | | | | | | | | | Improve C# reflection support
| | * | | Knock-on effects of moving GetSampleMessage.Jon Skeet2015-07-091-8/+8
| | | | |
| | * | | Lots more tests for FieldCodec, MapField, RepeatedFieldJon Skeet2015-07-097-18/+516
| | | | | | | | | | | | | | | | | | | | ... and some implementation changes to go with them.
| | * | | Fix descriptor reflection in various waysJon Skeet2015-07-0917-378/+178
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - The protos are no longer publicly exposed at all - Oneof detection now works (as we default to -1, not 0) - OneofDescriptor exposes the fields in the oneof - Removed unnecessary code for replacing protos - remnant of extensions - There's now just the non-generic form of IDescriptor
| | * | | Regenerated code due to previous commit.Jon Skeet2015-07-098-377/+829
| | | | | | | | | | | | | | | | | | | | Note that now we need a proto3 version of addressbook.proto. This may affect other platforms, and could do with an overhaul to follow proto3 conventions anyway (e.g. repeated field names). Will need to think about that carefully before merging into master. Raised issue #565 for this.
| | * | | Codegen changes to support descriptor runtime changesJon Skeet2015-07-0911-12/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Add a partial method called by all constructors - Generate internal classes for descriptor.proto (only) - Forbid proto2 descriptors except for descriptor.proto
| | * | | Changes to generated code due to previous commit.Jon Skeet2015-07-097-939/+692
| | | | |
| | * | | Implement reflection properly for fields.Jon Skeet2015-07-0916-249/+466
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - FieldAccessorTable is now non-generic - We don't have a static field per message type in the umbrella class. (Message descriptors are accessed via the file descriptor.) - Removed the "descriptor assigner" complication from the descriptor fixup; without extensions, we don't need it - MapField implements IDictionary (more tests would be good...) - RepeatedField implements IList (more tests would be good) - Use expression trees to build accessors. (Will need to test this on various platforms... probably need a fallback strategy just using reflection directly.) - Added FieldDescriptor.IsMap - Added tests for reflection with generated messages Changes to generated code coming in next commit.
| * | | | Merge pull request #577 from jskeet/csharp-fixupJan Tattermusch2015-07-098-105/+104
| |\| | | | | | | | | | | | | Fix-ups suggested in PR #560