diff options
Diffstat (limited to 'csharp/ProtocolBuffers.Test/MessageTest.cs')
-rw-r--r-- | csharp/ProtocolBuffers.Test/MessageTest.cs | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/csharp/ProtocolBuffers.Test/MessageTest.cs b/csharp/ProtocolBuffers.Test/MessageTest.cs index 60aa3d17..04bea0a5 100644 --- a/csharp/ProtocolBuffers.Test/MessageTest.cs +++ b/csharp/ProtocolBuffers.Test/MessageTest.cs @@ -153,8 +153,7 @@ namespace Google.ProtocolBuffers { builder[descriptor.FindDescriptor<FieldDescriptor>("optional_message")] = TestRequiredInitialized; Assert.IsTrue(builder.IsInitialized); - // TODO(jonskeet): Remove this nastiness by making IBuilder always generic. - ((IBuilder) builder).AddRepeatedField(descriptor.FindDescriptor<FieldDescriptor>("repeated_message"), TestRequiredUninitialized); + builder.AddRepeatedField(descriptor.FindDescriptor<FieldDescriptor>("repeated_message"), TestRequiredUninitialized); Assert.IsFalse(builder.IsInitialized); builder.SetRepeatedField(descriptor.FindDescriptor<FieldDescriptor>("repeated_message"), 0, TestRequiredInitialized); @@ -266,8 +265,7 @@ namespace Google.ProtocolBuffers { [Test] public void DynamicBuildPartial() { // We're mostly testing that no exception is thrown. - // TODO(jonskeet): Fix this ghastly casting mess - DynamicMessage message = (DynamicMessage) ((IBuilder) DynamicMessage.CreateBuilder(TestRequired.Descriptor)).BuildPartial(); + DynamicMessage message = DynamicMessage.CreateBuilder(TestRequired.Descriptor).BuildPartial(); Assert.IsFalse(message.Initialized); } |