Reverted: DefaultGraphStack: Perf-ops

Nucs · Nucs · commit f1bc50ecb75b · 2019-08-28T10:29:14.000+03:00
diff --git a/src/TensorFlowNET.Core/Graphs/DefaultGraphStack.cs b/src/TensorFlowNET.Core/Graphs/DefaultGraphStack.cs
@@ -27,24 +27,24 @@ namespace Tensorflow
     /// </summary>
     public class DefaultGraphStack 
     {
-        private readonly List<StackModel> stack = new List<StackModel>();
+        private readonly List<StackModel> _stack = new List<StackModel>();
 
         public void set_controller(Graph @default)
         {
-            if (!stack.Exists(x => x.Graph == @default))
-                stack.Add(new StackModel {Graph = @default, IsDefault = true});
+            if (!_stack.Exists(x => x.Graph == @default))
+                _stack.Add(new StackModel {Graph = @default, IsDefault = true});
 
-            foreach (var s in stack)
+            foreach (var s in _stack)
                 s.IsDefault = s.Graph == @default;
         }
 
         public Graph get_controller()
         {
-            if (stack.Count(x => x.IsDefault) == 0)
-                stack.Add(new StackModel {Graph = tf.Graph(), IsDefault = true});
-            for (var i = stack.Count - 1; i >= 0; i--)
+            if (_stack.Count(x => x.IsDefault) == 0)
+                _stack.Add(new StackModel {Graph = tf.Graph(), IsDefault = true});
+            for (var i = _stack.Count - 1; i >= 0; i--)
             {
-                var x = stack[i];
+                var x = _stack[i];
                 if (x.IsDefault)
                     return x.Graph;
             }
@@ -54,16 +54,16 @@ public Graph get_controller()
 
         public bool remove(Graph g)
         {
-            if (stack.Count == 0)
+            if (_stack.Count == 0)
                 return false;
 
-            var sm = stack.Find(model => model.Graph == g);
-            return sm != null && stack.Remove(sm);
+            var sm = _stack.Find(model => model.Graph == g);
+            return sm != null && _stack.Remove(sm);
         }
 
         public void reset()
         {
-            stack.Clear();
+            _stack.Clear();
         }
 
         private class StackModel

Original file line number	Diff line number	Diff line change
`@@ -27,24 +27,24 @@ namespace Tensorflow`
`27`	`27`	`/// </summary>`
`28`	`28`	`public class DefaultGraphStack`
`29`	`29`	`{`
`30`		`- private readonly List<StackModel> stack = new List<StackModel>();`
	`30`	`+ private readonly List<StackModel> _stack = new List<StackModel>();`
`31`	`31`
`32`	`32`	`public void set_controller(Graph @default)`
`33`	`33`	`{`
`34`		`- if (!stack.Exists(x => x.Graph == @default))`
`35`		`- stack.Add(new StackModel {Graph = @default, IsDefault = true});`
	`34`	`+ if (!_stack.Exists(x => x.Graph == @default))`
	`35`	`+ _stack.Add(new StackModel {Graph = @default, IsDefault = true});`
`36`	`36`
`37`		`- foreach (var s in stack)`
	`37`	`+ foreach (var s in _stack)`
`38`	`38`	`s.IsDefault = s.Graph == @default;`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`public Graph get_controller()`
`42`	`42`	`{`
`43`		`- if (stack.Count(x => x.IsDefault) == 0)`
`44`		`- stack.Add(new StackModel {Graph = tf.Graph(), IsDefault = true});`
`45`		`- for (var i = stack.Count - 1; i >= 0; i--)`
	`43`	`+ if (_stack.Count(x => x.IsDefault) == 0)`
	`44`	`+ _stack.Add(new StackModel {Graph = tf.Graph(), IsDefault = true});`
	`45`	`+ for (var i = _stack.Count - 1; i >= 0; i--)`
`46`	`46`	`{`
`47`		`- var x = stack[i];`
	`47`	`+ var x = _stack[i];`
`48`	`48`	`if (x.IsDefault)`
`49`	`49`	`return x.Graph;`
`50`	`50`	`}`
`@@ -54,16 +54,16 @@ public Graph get_controller()`
`54`	`54`
`55`	`55`	`public bool remove(Graph g)`
`56`	`56`	`{`
`57`		`- if (stack.Count == 0)`
	`57`	`+ if (_stack.Count == 0)`
`58`	`58`	`return false;`
`59`	`59`
`60`		`- var sm = stack.Find(model => model.Graph == g);`
`61`		`- return sm != null && stack.Remove(sm);`
	`60`	`+ var sm = _stack.Find(model => model.Graph == g);`
	`61`	`+ return sm != null && _stack.Remove(sm);`
`62`	`62`	`}`
`63`	`63`
`64`	`64`	`public void reset()`
`65`	`65`	`{`
`66`		`- stack.Clear();`
	`66`	`+ _stack.Clear();`
`67`	`67`	`}`
`68`	`68`
`69`	`69`	`private class StackModel`