1. 30 Oct, 2019 1 commit
  2. 19 Sep, 2019 1 commit
  3. 23 Aug, 2019 1 commit
  4. 03 May, 2019 1 commit
    • Sydney Acksman's avatar
      C# Proto2 feature : Extensions (#5350) · 9e89b6e7
      Sydney Acksman authored
      * Compiler changes (extensions)
      
      * Generated changes (extensions)
      
      * Library changes (extensions)
      
      * Adjusted a summary to indicate ContainingType can be null for extensions
      
      * Compiler changes (custom option review + access level review)
      
      * Generated code changes (custom options + access review)
      
      * Library changes (custom options + access review)
      
      * Support C# 6 with library changes
      
      * Access HasValue by property
      
      * Set access level of all extension classes to internal (revert in next PR)
      
      * Added null checks to custom options
      
      * Rebase on master and regenerate Conformance
      
      * Removed second dictionary from ExtensionSet
      
      * Rebased compiler changes
      
      * Rebased generated code changes
      
      * Rebased library changes + review changes
      
      * Add more safety checks to extension accessors
      
      * Remove instances where extension sets were unnecessarily allocated
      
      * Remove cleared items from sets
      Empty sets are now made null
      IExtensionMessage -> IExtendableMessage
      
      * Remove dead code from IExtensionValue impls
      
      * Clean both repeated and single value extensions
      
      * Add GetOrRegister method for repeated fields and allow clearing repeated extensions
      
      * Add type safe ClearExtension methods, remove non-generic IExtendableMessage interface.
      
      * Simplify ExtensionSet.TryMergeFieldFrom
      
      * Rebase on master to resolve conflicts
      
      * Fix Makefile.am
      
      * Add ObjectIntPair to Makefile.am
      9e89b6e7
  5. 20 Aug, 2018 1 commit
  6. 19 Jan, 2017 1 commit
    • Jon Skeet's avatar
      Support custom options in C# · 047575f2
      Jon Skeet authored
      This consists of:
      - Changing the codegen for the fixed set of options protos, to parse unknown fields instead of skipping them
      - Add a new CustomOptions type in the C# support library
      - Expose CustomOptions properties from the immutable proto wrappers in the support library
      
      Only single-value options are currently supported, and fetching options values requires getting the type right
      and knowing the field number. Both of these can be addressed at a later time.
      
      Fixes #2143, at least as a first pass.
      047575f2
  7. 22 Nov, 2015 1 commit
    • Jon Skeet's avatar
      Tidy up reflection in advance of attempting to implement DynamicMessage. · 72ec3367
      Jon Skeet authored
      There are corner cases where MessageDescriptor.{ClrType,Parser} will return null, and these are now documented. However, normally they *should* be implemented, even for descriptors of for dynamic messages. Ditto FieldDescriptor.Accessor.
      We'll still need a fair amount of work to implement dynamic messages, but this change means that the public API will be remain intact.
      
      Additionally, this change starts making use of C# 6 features in the files that it touches. This is far from exhaustive, and later PRs will have more.
      
      Generated code changes coming in the next commit.
      72ec3367
  8. 04 Aug, 2015 1 commit
  9. 27 Jul, 2015 1 commit
    • Jon Skeet's avatar
      First attempt at using profile 259 for Google.Protobuf. · 0dbd5ec8
      Jon Skeet authored
      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.
      0dbd5ec8
  10. 22 Jul, 2015 2 commits
    • Jon Skeet's avatar
      Implemented Jan's suggestion of FieldCollection, replacing FieldAccessorCollection. · c1c6b2d0
      Jon Skeet authored
      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?)
      c1c6b2d0
    • Jon Skeet's avatar
      Remove the usage of attributes for field/method discovery. · 4668c3dc
      Jon Skeet authored
      Instead, introduce GeneratedCodeInfo which passes in what we need, and adjust the codegen to take account of this.
      4668c3dc
  11. 21 Jul, 2015 1 commit
    • Jon Skeet's avatar
      Revamp to reflection. · 53c399a1
      Jon Skeet authored
      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.
      53c399a1
  12. 17 Jul, 2015 1 commit
  13. 14 Jul, 2015 2 commits
  14. 09 Jul, 2015 1 commit
    • Jon Skeet's avatar
      Fix descriptor reflection in various ways · af259b77
      Jon Skeet authored
      - 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
      af259b77
  15. 30 Jun, 2015 1 commit
  16. 09 Jun, 2015 1 commit
  17. 29 May, 2015 1 commit