1. 20 Aug, 2018 1 commit
  2. 19 Dec, 2017 1 commit
  3. 12 Nov, 2017 1 commit
  4. 27 Jan, 2017 1 commit
  5. 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
  6. 20 Apr, 2016 1 commit
  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. 21 Nov, 2015 1 commit
  9. 19 Nov, 2015 1 commit
  10. 09 Nov, 2015 1 commit
  11. 14 Aug, 2015 1 commit
  12. 08 Aug, 2015 1 commit
    • Jon Skeet's avatar
      Make FieldDescriptor.IsPacked work appropriately. · 547d8e82
      Jon Skeet authored
      This is a bit of a grotty hack, as we need to sort of fake proto2 field presence, but with only a proto3 version of the descriptor messages (a bit like oneof detection).
      Should be okay, but will need to be careful of this if we ever implement proto2.
      547d8e82
  13. 31 Jul, 2015 1 commit
  14. 25 Jul, 2015 1 commit
  15. 22 Jul, 2015 3 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
      Added newlines · 5e0cfc9a
      Jon Skeet authored
      5e0cfc9a
    • Jon Skeet's avatar
      First pass at making field access simpler. · 20bf6a56
      Jon Skeet authored
      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 :)
      20bf6a56
  16. 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
  17. 17 Jul, 2015 1 commit