Merge pull request #44 from dkavolis/fix#43

Fix kerbal NRE spam on EVA

FerramAerospaceResearch/FARPartGeometry/GeometryPartModule.cs

@@ -229,6 +229,13 @@ public bool IgnoreForMainAxis
             get { return ignoreForMainAxis; }
         }
 
+        public override void OnAwake()
+        {
+            base.OnAwake();
+            ignoredTransforms = new List<string>();
+            unignoredTransforms = new List<string>();
+        }
+
         void Start()
         {
             destroyed = false;
@@ -1020,8 +1027,8 @@ public override void OnLoad(ConfigNode node)
             LoadBool(node, "ignoreForMainAxis", out ignoreForMainAxis);
             LoadBool(node, "ignoreIfNoRenderer", out ignoreIfNoRenderer);
             LoadBool(node, "rebuildOnAnimation", out rebuildOnAnimation);
-            ignoredTransforms = LoadList(node, "ignoreTransform");
-            unignoredTransforms = LoadList(node, "unignoreTransform");
+            LoadList(node, "ignoreTransform", ref ignoredTransforms);
+            LoadList(node, "unignoreTransform", ref unignoredTransforms);
         }
 
         private void LoadBool(ConfigNode node, string name, out bool value)
@@ -1037,9 +1044,9 @@ private void LoadBool(ConfigNode node, string name, out bool value)
             }
         }
 
-        private List<string> LoadList(ConfigNode node, string name)
+        private void LoadList(ConfigNode node, string name, ref List<string> list)
         {
-            var list = new List<string>();
+            list.Clear();
             if (node.HasValue(name))
             {
                 foreach (string _name in node.GetValues(name))
@@ -1048,7 +1055,6 @@ private List<string> LoadList(ConfigNode node, string name)
                 }
                 _ready = false;
             }
-            return list;
         }
     }
 }

README.md

@@ -64,6 +64,10 @@ Set all the other winglet/control surface values to zero
 
 CHANGELOG
 =======================================================
+-------------------master branch-------------------------  
+
+Fix NRE spam on EVA [#43](https://github.com/dkavolis/Ferram-Aerospace-Research/issues/43) ([#44](https://github.com/dkavolis/Ferram-Aerospace-Research/pull/44))  
+
 0.15.10.0V "Lundgren"------------------------------------
 
 **Note for Kopernicus users: DO NOT overwrite MFI that comes with Kopernicus since it is locked to that particular version**