commit 988ffe0a78ebda0410e61ce31a3bd689c774f59e
author Thomas Van Lenten <thomasvl@google.com> Wed Jan 04 15:03:42 2017 -0500
committer Thomas Van Lenten <thomasvl@google.com> Thu Jan 05 09:15:40 2017 -0500
tree f5e92cdc634e4eef2ef2b891de95ad23bec3fba4
parent 4cb113a91b180559f0eedbca0244ef1181a7204c

Minor fix for autocreated object repeated fields and maps.

- If setting/clearing a repeated field/map that was objects, check the class
  before checking the autocreator.
- Just to be paranoid, don’t mutate within copy/mutableCopy for the autocreated
  classes to ensure there is less chance of issues if someone does something
  really crazy threading wise.
- Some more tests for the internal AutocreatedArray/AutocreatedDictionary
  classes to ensure things are working as expected.
- Add Xcode 8.2 to the full_mac_build.sh supported list.

objectivec/GPBArray.m

diff --git a/objectivec/GPBArray.m b/objectivec/GPBArray.m
index ae57747..f401631 100644
--- a/objectivec/GPBArray.m
+++ b/objectivec/GPBArray.m
@@ -2519,14 +2519,14 @@
 
 - (id)copyWithZone:(NSZone *)zone {
   if (_array == nil) {
-    _array = [[NSMutableArray alloc] init];
+    return [[NSMutableArray allocWithZone:zone] init];
   }
   return [_array copyWithZone:zone];
 }
 
 - (id)mutableCopyWithZone:(NSZone *)zone {
   if (_array == nil) {
-    _array = [[NSMutableArray alloc] init];
+    return [[NSMutableArray allocWithZone:zone] init];
   }
   return [_array mutableCopyWithZone:zone];
 }