Commit 7909b2ed authored by Jon Skeet's avatar Jon Skeet

Merge pull request #659 from jskeet/fix-frozen

Remove a few hangers-on from frozenness.
parents 7a0effb9 dea15a7c
...@@ -54,9 +54,6 @@ namespace Google.Protobuf.Examples.AddressBook { ...@@ -54,9 +54,6 @@ namespace Google.Protobuf.Examples.AddressBook {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Person() { public Person() {
OnConstruction(); OnConstruction();
} }
...@@ -240,9 +237,6 @@ namespace Google.Protobuf.Examples.AddressBook { ...@@ -240,9 +237,6 @@ namespace Google.Protobuf.Examples.AddressBook {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public PhoneNumber() { public PhoneNumber() {
OnConstruction(); OnConstruction();
} }
...@@ -380,9 +374,6 @@ namespace Google.Protobuf.Examples.AddressBook { ...@@ -380,9 +374,6 @@ namespace Google.Protobuf.Examples.AddressBook {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public AddressBook() { public AddressBook() {
OnConstruction(); OnConstruction();
} }
......
...@@ -183,9 +183,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -183,9 +183,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestMap() { public TestMap() {
OnConstruction(); OnConstruction();
} }
...@@ -572,9 +569,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -572,9 +569,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestMapSubmessage() { public TestMapSubmessage() {
OnConstruction(); OnConstruction();
} }
...@@ -687,9 +681,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -687,9 +681,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestMessageMap() { public TestMessageMap() {
OnConstruction(); OnConstruction();
} }
...@@ -788,9 +779,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -788,9 +779,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestSameTypeMap() { public TestSameTypeMap() {
OnConstruction(); OnConstruction();
} }
...@@ -907,9 +895,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -907,9 +895,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestArenaMap() { public TestArenaMap() {
OnConstruction(); OnConstruction();
} }
...@@ -1260,9 +1245,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -1260,9 +1245,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public MessageContainingEnumCalledType() { public MessageContainingEnumCalledType() {
OnConstruction(); OnConstruction();
} }
...@@ -1371,9 +1353,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -1371,9 +1353,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public MessageContainingMapCalledEntry() { public MessageContainingMapCalledEntry() {
OnConstruction(); OnConstruction();
} }
......
...@@ -62,9 +62,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -62,9 +62,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public ImportMessage() { public ImportMessage() {
OnConstruction(); OnConstruction();
} }
......
...@@ -48,9 +48,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -48,9 +48,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public PublicImportMessage() { public PublicImportMessage() {
OnConstruction(); OnConstruction();
} }
......
...@@ -80,9 +80,6 @@ namespace UnitTest.Issues.TestProtos { ...@@ -80,9 +80,6 @@ namespace UnitTest.Issues.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Issue307() { public Issue307() {
OnConstruction(); OnConstruction();
} }
...@@ -164,9 +161,6 @@ namespace UnitTest.Issues.TestProtos { ...@@ -164,9 +161,6 @@ namespace UnitTest.Issues.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public NestedOnce() { public NestedOnce() {
OnConstruction(); OnConstruction();
} }
...@@ -248,9 +242,6 @@ namespace UnitTest.Issues.TestProtos { ...@@ -248,9 +242,6 @@ namespace UnitTest.Issues.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public NestedTwice() { public NestedTwice() {
OnConstruction(); OnConstruction();
} }
...@@ -341,9 +332,6 @@ namespace UnitTest.Issues.TestProtos { ...@@ -341,9 +332,6 @@ namespace UnitTest.Issues.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public NegativeEnumMessage() { public NegativeEnumMessage() {
OnConstruction(); OnConstruction();
} }
...@@ -488,9 +476,6 @@ namespace UnitTest.Issues.TestProtos { ...@@ -488,9 +476,6 @@ namespace UnitTest.Issues.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public DeprecatedChild() { public DeprecatedChild() {
OnConstruction(); OnConstruction();
} }
...@@ -571,9 +556,6 @@ namespace UnitTest.Issues.TestProtos { ...@@ -571,9 +556,6 @@ namespace UnitTest.Issues.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public DeprecatedFieldsMessage() { public DeprecatedFieldsMessage() {
OnConstruction(); OnConstruction();
} }
...@@ -800,9 +782,6 @@ namespace UnitTest.Issues.TestProtos { ...@@ -800,9 +782,6 @@ namespace UnitTest.Issues.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public ItemField() { public ItemField() {
OnConstruction(); OnConstruction();
} }
......
...@@ -232,9 +232,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -232,9 +232,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestAllTypes() { public TestAllTypes() {
OnConstruction(); OnConstruction();
} }
...@@ -1473,9 +1470,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -1473,9 +1470,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public NestedMessage() { public NestedMessage() {
OnConstruction(); OnConstruction();
} }
...@@ -1587,9 +1581,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -1587,9 +1581,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public NestedTestAllTypes() { public NestedTestAllTypes() {
OnConstruction(); OnConstruction();
} }
...@@ -1752,9 +1743,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -1752,9 +1743,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestDeprecatedFields() { public TestDeprecatedFields() {
OnConstruction(); OnConstruction();
} }
...@@ -1862,9 +1850,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -1862,9 +1850,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public ForeignMessage() { public ForeignMessage() {
OnConstruction(); OnConstruction();
} }
...@@ -1971,9 +1956,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -1971,9 +1956,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestReservedFields() { public TestReservedFields() {
OnConstruction(); OnConstruction();
} }
...@@ -2054,9 +2036,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -2054,9 +2036,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestForeignNested() { public TestForeignNested() {
OnConstruction(); OnConstruction();
} }
...@@ -2169,9 +2148,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -2169,9 +2148,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestReallyLargeTagNumber() { public TestReallyLargeTagNumber() {
OnConstruction(); OnConstruction();
} }
...@@ -2304,9 +2280,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -2304,9 +2280,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestRecursiveMessage() { public TestRecursiveMessage() {
OnConstruction(); OnConstruction();
} }
...@@ -2445,9 +2418,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -2445,9 +2418,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestMutualRecursionA() { public TestMutualRecursionA() {
OnConstruction(); OnConstruction();
} }
...@@ -2560,9 +2530,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -2560,9 +2530,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestMutualRecursionB() { public TestMutualRecursionB() {
OnConstruction(); OnConstruction();
} }
...@@ -2701,9 +2668,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -2701,9 +2668,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestCamelCaseFieldNames() { public TestCamelCaseFieldNames() {
OnConstruction(); OnConstruction();
} }
...@@ -2968,9 +2932,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -2968,9 +2932,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestFieldOrderings() { public TestFieldOrderings() {
OnConstruction(); OnConstruction();
} }
...@@ -3162,9 +3123,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -3162,9 +3123,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public NestedMessage() { public NestedMessage() {
OnConstruction(); OnConstruction();
} }
...@@ -3302,9 +3260,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -3302,9 +3260,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public SparseEnumMessage() { public SparseEnumMessage() {
OnConstruction(); OnConstruction();
} }
...@@ -3411,9 +3366,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -3411,9 +3366,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public OneString() { public OneString() {
OnConstruction(); OnConstruction();
} }
...@@ -3520,9 +3472,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -3520,9 +3472,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public MoreString() { public MoreString() {
OnConstruction(); OnConstruction();
} }
...@@ -3621,9 +3570,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -3621,9 +3570,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public OneBytes() { public OneBytes() {
OnConstruction(); OnConstruction();
} }
...@@ -3730,9 +3676,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -3730,9 +3676,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public MoreBytes() { public MoreBytes() {
OnConstruction(); OnConstruction();
} }
...@@ -3839,9 +3782,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -3839,9 +3782,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Int32Message() { public Int32Message() {
OnConstruction(); OnConstruction();
} }
...@@ -3948,9 +3888,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -3948,9 +3888,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Uint32Message() { public Uint32Message() {
OnConstruction(); OnConstruction();
} }
...@@ -4057,9 +3994,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -4057,9 +3994,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Int64Message() { public Int64Message() {
OnConstruction(); OnConstruction();
} }
...@@ -4166,9 +4100,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -4166,9 +4100,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Uint64Message() { public Uint64Message() {
OnConstruction(); OnConstruction();
} }
...@@ -4275,9 +4206,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -4275,9 +4206,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public BoolMessage() { public BoolMessage() {
OnConstruction(); OnConstruction();
} }
...@@ -4384,9 +4312,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -4384,9 +4312,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestOneof() { public TestOneof() {
OnConstruction(); OnConstruction();
} }
...@@ -4579,9 +4504,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -4579,9 +4504,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestPackedTypes() { public TestPackedTypes() {
OnConstruction(); OnConstruction();
} }
...@@ -4928,9 +4850,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -4928,9 +4850,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestUnpackedTypes() { public TestUnpackedTypes() {
OnConstruction(); OnConstruction();
} }
...@@ -5277,9 +5196,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -5277,9 +5196,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestRepeatedScalarDifferentTagSizes() { public TestRepeatedScalarDifferentTagSizes() {
OnConstruction(); OnConstruction();
} }
...@@ -5474,9 +5390,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -5474,9 +5390,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestCommentInjectionMessage() { public TestCommentInjectionMessage() {
OnConstruction(); OnConstruction();
} }
...@@ -5583,9 +5496,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -5583,9 +5496,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public FooRequest() { public FooRequest() {
OnConstruction(); OnConstruction();
} }
...@@ -5666,9 +5576,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -5666,9 +5576,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public FooResponse() { public FooResponse() {
OnConstruction(); OnConstruction();
} }
...@@ -5749,9 +5656,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -5749,9 +5656,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public FooClientMessage() { public FooClientMessage() {
OnConstruction(); OnConstruction();
} }
...@@ -5832,9 +5736,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -5832,9 +5736,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public FooServerMessage() { public FooServerMessage() {
OnConstruction(); OnConstruction();
} }
...@@ -5915,9 +5816,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -5915,9 +5816,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public BarRequest() { public BarRequest() {
OnConstruction(); OnConstruction();
} }
...@@ -5998,9 +5896,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -5998,9 +5896,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public BarResponse() { public BarResponse() {
OnConstruction(); OnConstruction();
} }
......
...@@ -183,9 +183,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -183,9 +183,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public TestWellKnownTypes() { public TestWellKnownTypes() {
OnConstruction(); OnConstruction();
} }
...@@ -833,9 +830,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -833,9 +830,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public RepeatedWellKnownTypes() { public RepeatedWellKnownTypes() {
OnConstruction(); OnConstruction();
} }
...@@ -1240,9 +1234,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -1240,9 +1234,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public OneofWellKnownTypes() { public OneofWellKnownTypes() {
OnConstruction(); OnConstruction();
} }
...@@ -1910,9 +1901,6 @@ namespace Google.Protobuf.TestProtos { ...@@ -1910,9 +1901,6 @@ namespace Google.Protobuf.TestProtos {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public MapWellKnownTypes() { public MapWellKnownTypes() {
OnConstruction(); OnConstruction();
} }
......
...@@ -54,7 +54,6 @@ namespace Google.Protobuf.Collections ...@@ -54,7 +54,6 @@ namespace Google.Protobuf.Collections
{ {
// TODO: Don't create the map/list until we have an entry. (Assume many maps will be empty.) // TODO: Don't create the map/list until we have an entry. (Assume many maps will be empty.)
private readonly bool allowNullValues; private readonly bool allowNullValues;
private bool frozen;
private readonly Dictionary<TKey, LinkedListNode<KeyValuePair<TKey, TValue>>> map = private readonly Dictionary<TKey, LinkedListNode<KeyValuePair<TKey, TValue>>> map =
new Dictionary<TKey, LinkedListNode<KeyValuePair<TKey, TValue>>>(); new Dictionary<TKey, LinkedListNode<KeyValuePair<TKey, TValue>>>();
private readonly LinkedList<KeyValuePair<TKey, TValue>> list = new LinkedList<KeyValuePair<TKey, TValue>>(); private readonly LinkedList<KeyValuePair<TKey, TValue>> list = new LinkedList<KeyValuePair<TKey, TValue>>();
...@@ -254,7 +253,7 @@ namespace Google.Protobuf.Collections ...@@ -254,7 +253,7 @@ namespace Google.Protobuf.Collections
public bool AllowsNullValues { get { return allowNullValues; } } public bool AllowsNullValues { get { return allowNullValues; } }
public int Count { get { return list.Count; } } public int Count { get { return list.Count; } }
public bool IsReadOnly { get { return frozen; } } public bool IsReadOnly { get { return false; } }
public override bool Equals(object other) public override bool Equals(object other)
{ {
...@@ -416,10 +415,6 @@ namespace Google.Protobuf.Collections ...@@ -416,10 +415,6 @@ namespace Google.Protobuf.Collections
set set
{ {
if (frozen)
{
throw new NotSupportedException("Dictionary is frozen");
}
this[(TKey)key] = (TValue)value; this[(TKey)key] = (TValue)value;
} }
} }
......
...@@ -40,7 +40,7 @@ namespace Google.Protobuf.Collections ...@@ -40,7 +40,7 @@ namespace Google.Protobuf.Collections
{ {
/// <summary> /// <summary>
/// The contents of a repeated field: essentially, a collection with some extra /// The contents of a repeated field: essentially, a collection with some extra
/// restrictions (no null values) and capabilities (deep cloning and freezing). /// restrictions (no null values) and capabilities (deep cloning).
/// </summary> /// </summary>
/// <typeparam name="T">The element type of the repeated field.</typeparam> /// <typeparam name="T">The element type of the repeated field.</typeparam>
public sealed class RepeatedField<T> : IList<T>, IList, IDeepCloneable<RepeatedField<T>>, IEquatable<RepeatedField<T>> public sealed class RepeatedField<T> : IList<T>, IList, IDeepCloneable<RepeatedField<T>>, IEquatable<RepeatedField<T>>
...@@ -48,7 +48,6 @@ namespace Google.Protobuf.Collections ...@@ -48,7 +48,6 @@ namespace Google.Protobuf.Collections
private static readonly T[] EmptyArray = new T[0]; private static readonly T[] EmptyArray = new T[0];
private const int MinArraySize = 8; private const int MinArraySize = 8;
private bool frozen;
private T[] array = EmptyArray; private T[] array = EmptyArray;
private int count = 0; private int count = 0;
...@@ -66,7 +65,6 @@ namespace Google.Protobuf.Collections ...@@ -66,7 +65,6 @@ namespace Google.Protobuf.Collections
public RepeatedField<T> Clone() public RepeatedField<T> Clone()
{ {
RepeatedField<T> clone = new RepeatedField<T>(); RepeatedField<T> clone = new RepeatedField<T>();
// Clone is implicitly *not* frozen, even if this object is.
if (array != EmptyArray) if (array != EmptyArray)
{ {
clone.array = (T[])array.Clone(); clone.array = (T[])array.Clone();
......
...@@ -173,9 +173,6 @@ namespace Google.Protobuf.Reflection { ...@@ -173,9 +173,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public FileDescriptorSet() { public FileDescriptorSet() {
OnConstruction(); OnConstruction();
} }
...@@ -274,9 +271,6 @@ namespace Google.Protobuf.Reflection { ...@@ -274,9 +271,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public FileDescriptorProto() { public FileDescriptorProto() {
OnConstruction(); OnConstruction();
} }
...@@ -627,9 +621,6 @@ namespace Google.Protobuf.Reflection { ...@@ -627,9 +621,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public DescriptorProto() { public DescriptorProto() {
OnConstruction(); OnConstruction();
} }
...@@ -913,9 +904,6 @@ namespace Google.Protobuf.Reflection { ...@@ -913,9 +904,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public ExtensionRange() { public ExtensionRange() {
OnConstruction(); OnConstruction();
} }
...@@ -1048,9 +1036,6 @@ namespace Google.Protobuf.Reflection { ...@@ -1048,9 +1036,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public ReservedRange() { public ReservedRange() {
OnConstruction(); OnConstruction();
} }
...@@ -1188,9 +1173,6 @@ namespace Google.Protobuf.Reflection { ...@@ -1188,9 +1173,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public FieldDescriptorProto() { public FieldDescriptorProto() {
OnConstruction(); OnConstruction();
} }
...@@ -1544,9 +1526,6 @@ namespace Google.Protobuf.Reflection { ...@@ -1544,9 +1526,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public OneofDescriptorProto() { public OneofDescriptorProto() {
OnConstruction(); OnConstruction();
} }
...@@ -1653,9 +1632,6 @@ namespace Google.Protobuf.Reflection { ...@@ -1653,9 +1632,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public EnumDescriptorProto() { public EnumDescriptorProto() {
OnConstruction(); OnConstruction();
} }
...@@ -1812,9 +1788,6 @@ namespace Google.Protobuf.Reflection { ...@@ -1812,9 +1788,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public EnumValueDescriptorProto() { public EnumValueDescriptorProto() {
OnConstruction(); OnConstruction();
} }
...@@ -1979,9 +1952,6 @@ namespace Google.Protobuf.Reflection { ...@@ -1979,9 +1952,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public ServiceDescriptorProto() { public ServiceDescriptorProto() {
OnConstruction(); OnConstruction();
} }
...@@ -2138,9 +2108,6 @@ namespace Google.Protobuf.Reflection { ...@@ -2138,9 +2108,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public MethodDescriptorProto() { public MethodDescriptorProto() {
OnConstruction(); OnConstruction();
} }
...@@ -2383,9 +2350,6 @@ namespace Google.Protobuf.Reflection { ...@@ -2383,9 +2350,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public FileOptions() { public FileOptions() {
OnConstruction(); OnConstruction();
} }
...@@ -2860,9 +2824,6 @@ namespace Google.Protobuf.Reflection { ...@@ -2860,9 +2824,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public MessageOptions() { public MessageOptions() {
OnConstruction(); OnConstruction();
} }
...@@ -3065,9 +3026,6 @@ namespace Google.Protobuf.Reflection { ...@@ -3065,9 +3026,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public FieldOptions() { public FieldOptions() {
OnConstruction(); OnConstruction();
} }
...@@ -3340,9 +3298,6 @@ namespace Google.Protobuf.Reflection { ...@@ -3340,9 +3298,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public EnumOptions() { public EnumOptions() {
OnConstruction(); OnConstruction();
} }
...@@ -3493,9 +3448,6 @@ namespace Google.Protobuf.Reflection { ...@@ -3493,9 +3448,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public EnumValueOptions() { public EnumValueOptions() {
OnConstruction(); OnConstruction();
} }
...@@ -3620,9 +3572,6 @@ namespace Google.Protobuf.Reflection { ...@@ -3620,9 +3572,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public ServiceOptions() { public ServiceOptions() {
OnConstruction(); OnConstruction();
} }
...@@ -3747,9 +3696,6 @@ namespace Google.Protobuf.Reflection { ...@@ -3747,9 +3696,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public MethodOptions() { public MethodOptions() {
OnConstruction(); OnConstruction();
} }
...@@ -3874,9 +3820,6 @@ namespace Google.Protobuf.Reflection { ...@@ -3874,9 +3820,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public UninterpretedOption() { public UninterpretedOption() {
OnConstruction(); OnConstruction();
} }
...@@ -4132,9 +4075,6 @@ namespace Google.Protobuf.Reflection { ...@@ -4132,9 +4075,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public NamePart() { public NamePart() {
OnConstruction(); OnConstruction();
} }
...@@ -4272,9 +4212,6 @@ namespace Google.Protobuf.Reflection { ...@@ -4272,9 +4212,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public SourceCodeInfo() { public SourceCodeInfo() {
OnConstruction(); OnConstruction();
} }
...@@ -4374,9 +4311,6 @@ namespace Google.Protobuf.Reflection { ...@@ -4374,9 +4311,6 @@ namespace Google.Protobuf.Reflection {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Location() { public Location() {
OnConstruction(); OnConstruction();
} }
......
...@@ -51,9 +51,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -51,9 +51,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Any() { public Any() {
OnConstruction(); OnConstruction();
} }
......
...@@ -61,9 +61,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -61,9 +61,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Api() { public Api() {
OnConstruction(); OnConstruction();
} }
...@@ -264,9 +261,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -264,9 +261,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Method() { public Method() {
OnConstruction(); OnConstruction();
} }
......
...@@ -52,9 +52,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -52,9 +52,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Duration() { public Duration() {
OnConstruction(); OnConstruction();
} }
......
...@@ -51,9 +51,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -51,9 +51,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Empty() { public Empty() {
OnConstruction(); OnConstruction();
} }
......
...@@ -51,9 +51,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -51,9 +51,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public FieldMask() { public FieldMask() {
OnConstruction(); OnConstruction();
} }
......
...@@ -52,9 +52,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -52,9 +52,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public SourceContext() { public SourceContext() {
OnConstruction(); OnConstruction();
} }
......
...@@ -70,9 +70,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -70,9 +70,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Struct() { public Struct() {
OnConstruction(); OnConstruction();
} }
...@@ -171,9 +168,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -171,9 +168,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Value() { public Value() {
OnConstruction(); OnConstruction();
} }
...@@ -459,9 +453,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -459,9 +453,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public ListValue() { public ListValue() {
OnConstruction(); OnConstruction();
} }
......
...@@ -52,9 +52,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -52,9 +52,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Timestamp() { public Timestamp() {
OnConstruction(); OnConstruction();
} }
......
...@@ -82,9 +82,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -82,9 +82,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Type() { public Type() {
OnConstruction(); OnConstruction();
} }
...@@ -277,9 +274,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -277,9 +274,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Field() { public Field() {
OnConstruction(); OnConstruction();
} }
...@@ -594,9 +588,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -594,9 +588,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Enum() { public Enum() {
OnConstruction(); OnConstruction();
} }
...@@ -771,9 +762,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -771,9 +762,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public EnumValue() { public EnumValue() {
OnConstruction(); OnConstruction();
} }
...@@ -924,9 +912,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -924,9 +912,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Option() { public Option() {
OnConstruction(); OnConstruction();
} }
......
...@@ -61,9 +61,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -61,9 +61,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public DoubleValue() { public DoubleValue() {
OnConstruction(); OnConstruction();
} }
...@@ -170,9 +167,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -170,9 +167,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public FloatValue() { public FloatValue() {
OnConstruction(); OnConstruction();
} }
...@@ -279,9 +273,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -279,9 +273,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Int64Value() { public Int64Value() {
OnConstruction(); OnConstruction();
} }
...@@ -388,9 +379,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -388,9 +379,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public UInt64Value() { public UInt64Value() {
OnConstruction(); OnConstruction();
} }
...@@ -497,9 +485,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -497,9 +485,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public Int32Value() { public Int32Value() {
OnConstruction(); OnConstruction();
} }
...@@ -606,9 +591,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -606,9 +591,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public UInt32Value() { public UInt32Value() {
OnConstruction(); OnConstruction();
} }
...@@ -715,9 +697,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -715,9 +697,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public BoolValue() { public BoolValue() {
OnConstruction(); OnConstruction();
} }
...@@ -824,9 +803,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -824,9 +803,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public StringValue() { public StringValue() {
OnConstruction(); OnConstruction();
} }
...@@ -933,9 +909,6 @@ namespace Google.Protobuf.WellKnownTypes { ...@@ -933,9 +909,6 @@ namespace Google.Protobuf.WellKnownTypes {
get { return Descriptor; } get { return Descriptor; }
} }
private bool _frozen = false;
public bool IsFrozen { get { return _frozen; } }
public BytesValue() { public BytesValue() {
OnConstruction(); OnConstruction();
} }
......
...@@ -134,9 +134,7 @@ void MessageGenerator::Generate(io::Printer* printer) { ...@@ -134,9 +134,7 @@ void MessageGenerator::Generate(io::Printer* printer) {
"pbr::MessageDescriptor pb::IMessage.Descriptor {\n" "pbr::MessageDescriptor pb::IMessage.Descriptor {\n"
" get { return Descriptor; }\n" " get { return Descriptor; }\n"
"}\n" "}\n"
"\n" "\n");
"private bool _frozen = false;\n"
"public bool IsFrozen { get { return _frozen; } }\n\n");
// Parameterless constructor and partial OnConstruction method. // Parameterless constructor and partial OnConstruction method.
printer->Print( printer->Print(
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment