From 932cf1a232f518d497f830adf9f88a795757a689 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?David=20Srbeck=C3=BD?= <dsrbecky@gmail.com>
Date: Mon, 28 Jan 2008 17:12:38 +0000
Subject: [PATCH] NRefactory: INode.Children type changed from List<T> to more
 general IList<T>

---
 lib/NRefactory/Project/Src/Ast/AbstractNode.cs           | 6 +++---
 lib/NRefactory/Project/Src/Ast/INode.cs                  | 2 +-
 lib/NRefactory/Project/Src/Parser/AbstractParser.cs      | 2 +-
 lib/NRefactory/Project/Src/Parser/CSharp/CSharpParser.cs | 2 +-
 lib/NRefactory/Project/Src/Parser/IParser.cs             | 2 +-
 lib/NRefactory/Project/Src/Parser/VBNet/VBNetParser.cs   | 2 +-
 lib/NRefactory/Project/Src/SnippetParser.cs              | 8 ++++----
 7 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/lib/NRefactory/Project/Src/Ast/AbstractNode.cs b/lib/NRefactory/Project/Src/Ast/AbstractNode.cs
index a4ff1b5e0..e700e5359 100644
--- a/lib/NRefactory/Project/Src/Ast/AbstractNode.cs
+++ b/lib/NRefactory/Project/Src/Ast/AbstractNode.cs
@@ -15,14 +15,14 @@ namespace ICSharpCode.NRefactory.Ast
 {
 	public abstract class AbstractNode : INode
 	{
-		List<INode> children = new List<INode>();
+		IList<INode> children = new List<INode>();
 		
 		public INode Parent { get; set; }
 		public Location StartLocation { get; set; }
 		public Location EndLocation { get; set; }
 		public object UserData { get; set; }
 		
-		public List<INode> Children {
+		public IList<INode> Children {
 			get {
 				return children;
 			}
@@ -49,7 +49,7 @@ namespace ICSharpCode.NRefactory.Ast
 			return data;
 		}
 		
-		public static string GetCollectionString(ICollection collection)
+		public static string GetCollectionString<T>(IList<T> collection)
 		{
 			StringBuilder output = new StringBuilder();
 			output.Append('{');
diff --git a/lib/NRefactory/Project/Src/Ast/INode.cs b/lib/NRefactory/Project/Src/Ast/INode.cs
index 044731a42..e5ea897d6 100644
--- a/lib/NRefactory/Project/Src/Ast/INode.cs
+++ b/lib/NRefactory/Project/Src/Ast/INode.cs
@@ -17,7 +17,7 @@ namespace ICSharpCode.NRefactory.Ast
 			set;
 		}
 		
-		List<INode> Children {
+		IList<INode> Children {
 			get;
 		}
 		
diff --git a/lib/NRefactory/Project/Src/Parser/AbstractParser.cs b/lib/NRefactory/Project/Src/Parser/AbstractParser.cs
index 221222b33..e54867102 100644
--- a/lib/NRefactory/Project/Src/Parser/AbstractParser.cs
+++ b/lib/NRefactory/Project/Src/Parser/AbstractParser.cs
@@ -65,7 +65,7 @@ namespace ICSharpCode.NRefactory.Parser
 		
 		public abstract Expression ParseExpression();
 		public abstract BlockStatement ParseBlock();
-		public abstract List<INode> ParseTypeMembers();
+		public abstract IList<INode> ParseTypeMembers();
 		
 		protected abstract void SynErr(int line, int col, int errorNumber);
 		
diff --git a/lib/NRefactory/Project/Src/Parser/CSharp/CSharpParser.cs b/lib/NRefactory/Project/Src/Parser/CSharp/CSharpParser.cs
index d01a48949..8645e665b 100644
--- a/lib/NRefactory/Project/Src/Parser/CSharp/CSharpParser.cs
+++ b/lib/NRefactory/Project/Src/Parser/CSharp/CSharpParser.cs
@@ -84,7 +84,7 @@ namespace ICSharpCode.NRefactory.Parser.CSharp
 			return blockStmt;
 		}
 		
-		public override List<INode> ParseTypeMembers()
+		public override IList<INode> ParseTypeMembers()
 		{
 			lexer.NextToken();
 			compilationUnit = new CompilationUnit();
diff --git a/lib/NRefactory/Project/Src/Parser/IParser.cs b/lib/NRefactory/Project/Src/Parser/IParser.cs
index beb2d988c..9f8534f71 100644
--- a/lib/NRefactory/Project/Src/Parser/IParser.cs
+++ b/lib/NRefactory/Project/Src/Parser/IParser.cs
@@ -36,6 +36,6 @@ namespace ICSharpCode.NRefactory
 		
 		Expression ParseExpression();
 		BlockStatement ParseBlock();
-		List<INode> ParseTypeMembers();
+		IList<INode> ParseTypeMembers();
 	}
 }
diff --git a/lib/NRefactory/Project/Src/Parser/VBNet/VBNetParser.cs b/lib/NRefactory/Project/Src/Parser/VBNet/VBNetParser.cs
index adfdf5ebe..80858a55e 100644
--- a/lib/NRefactory/Project/Src/Parser/VBNet/VBNetParser.cs
+++ b/lib/NRefactory/Project/Src/Parser/VBNet/VBNetParser.cs
@@ -80,7 +80,7 @@ namespace ICSharpCode.NRefactory.Parser.VB
 			return st as BlockStatement;
 		}
 		
-		public override List<INode> ParseTypeMembers()
+		public override IList<INode> ParseTypeMembers()
 		{
 			lexer.NextToken();
 			compilationUnit = new CompilationUnit();
diff --git a/lib/NRefactory/Project/Src/SnippetParser.cs b/lib/NRefactory/Project/Src/SnippetParser.cs
index ddbf78a38..2fa5ba204 100644
--- a/lib/NRefactory/Project/Src/SnippetParser.cs
+++ b/lib/NRefactory/Project/Src/SnippetParser.cs
@@ -79,7 +79,7 @@ namespace ICSharpCode.NRefactory
 			}
 			if (errors.Count > 0) {
 				parser = ParserFactory.CreateParser(language, new StringReader(code));
-				List<INode> members = parser.ParseTypeMembers();
+				IList<INode> members = parser.ParseTypeMembers();
 				if (members != null && members.Count > 0 && parser.Errors.Count < errors.Count) {
 					errors = parser.Errors;
 					specials = parser.Lexer.SpecialTracker.RetrieveSpecials();
@@ -91,9 +91,9 @@ namespace ICSharpCode.NRefactory
 		
 		sealed class NodeListNode : INode
 		{
-			List<INode> nodes;
+			IList<INode> nodes;
 			
-			public NodeListNode(List<INode> nodes)
+			public NodeListNode(IList<INode> nodes)
 			{
 				this.nodes = nodes;
 			}
@@ -103,7 +103,7 @@ namespace ICSharpCode.NRefactory
 				set { throw new NotSupportedException(); }
 			}
 			
-			public List<INode> Children {
+			public IList<INode> Children {
 				get { return nodes; }
 			}