aboutsummaryrefslogtreecommitdiff
path: root/csharp/src/Google.Protobuf/CodedInputStream.cs
diff options
context:
space:
mode:
authorJon Skeet <jonskeet@google.com>2015-08-04 11:26:48 +0100
committerJon Skeet <jonskeet@google.com>2015-08-05 07:09:41 +0100
commit9df2defa295e1a3b315ed2a88791db51ff1f53e7 (patch)
tree5a0b130c7e82321c99954c4e86e29983e88d3bb6 /csharp/src/Google.Protobuf/CodedInputStream.cs
parentb6defa7c115a19d3671b2a6c5ebab2a471d273ea (diff)
downloadprotobuf-9df2defa295e1a3b315ed2a88791db51ff1f53e7.tar.gz
protobuf-9df2defa295e1a3b315ed2a88791db51ff1f53e7.tar.bz2
protobuf-9df2defa295e1a3b315ed2a88791db51ff1f53e7.zip
Consume unknown fields when parsing.
This is expected to be the cause of the conformance test failures. Generated code in next commit.
Diffstat (limited to 'csharp/src/Google.Protobuf/CodedInputStream.cs')
-rw-r--r--csharp/src/Google.Protobuf/CodedInputStream.cs33
1 files changed, 33 insertions, 0 deletions
diff --git a/csharp/src/Google.Protobuf/CodedInputStream.cs b/csharp/src/Google.Protobuf/CodedInputStream.cs
index a1abfcb0..fee31e3b 100644
--- a/csharp/src/Google.Protobuf/CodedInputStream.cs
+++ b/csharp/src/Google.Protobuf/CodedInputStream.cs
@@ -309,6 +309,39 @@ namespace Google.Protobuf
}
/// <summary>
+ /// Consumes the data for the field with the tag we've just read.
+ /// This should be called directly after <see cref="ReadTag"/>, when
+ /// the caller wishes to skip an unknown field.
+ /// </summary>
+ public void ConsumeLastField()
+ {
+ if (lastTag == 0)
+ {
+ throw new InvalidOperationException("ConsumeLastField cannot be called at the end of a stream");
+ }
+ switch (WireFormat.GetTagWireType(lastTag))
+ {
+ case WireFormat.WireType.StartGroup:
+ case WireFormat.WireType.EndGroup:
+ // TODO: Work out how to skip them instead? See issue 688.
+ throw new InvalidProtocolBufferException("Group tags not supported by proto3 C# implementation");
+ case WireFormat.WireType.Fixed32:
+ ReadFixed32();
+ break;
+ case WireFormat.WireType.Fixed64:
+ ReadFixed64();
+ break;
+ case WireFormat.WireType.LengthDelimited:
+ var length = ReadLength();
+ SkipRawBytes(length);
+ break;
+ case WireFormat.WireType.Varint:
+ ReadRawVarint32();
+ break;
+ }
+ }
+
+ /// <summary>
/// Reads a double field from the stream.
/// </summary>
public double ReadDouble()