Implement reflection properly for fields.

- 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.

1 files changed, 3 insertions, 4 deletions

diff --git a/csharp/src/ProtocolBuffers/IMessage.cs b/csharp/src/ProtocolBuffers/IMessage.cs
index 27bcc117..ad44668c 100644
--- a/csharp/src/ProtocolBuffers/IMessage.cs
+++ b/csharp/src/ProtocolBuffers/IMessage.cs
@@ -40,12 +40,11 @@ namespace Google.Protobuf
     // TODO(jonskeet): Split these interfaces into separate files when we're happy with them.

 

     /// <summary>

-    /// Reflection support for a specific message type. message

+    /// Reflection support for a specific message type.

     /// </summary>

-    /// <typeparam name="T">The message type being reflected.</typeparam>

-    public interface IReflectedMessage<T> where T : IMessage<T>

+    public interface IReflectedMessage

     {

-        FieldAccessorTable<T> Fields { get; }

+        FieldAccessorTable Fields { get; }

         // TODO(jonskeet): Descriptor? Or a single property which has "all you need for reflection"?

     }