diff options
| author |  Jon Skeet <jonskeet@google.com> | 2015-08-04 11:26:48 +0100 |
|---|---|---|
| committer | Jon Skeet <jonskeet@google.com> | 2015-08-05 07:09:41 +0100 |
| commit | 9df2defa295e1a3b315ed2a88791db51ff1f53e7 (patch) | |
| tree | 5a0b130c7e82321c99954c4e86e29983e88d3bb6 /csharp/src/Google.Protobuf | |
| parent | b6defa7c115a19d3671b2a6c5ebab2a471d273ea (diff) | |
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')
| -rw-r--r-- | csharp/src/Google.Protobuf/CodedInputStream.cs | 33 |
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()
|