Constructors of abstract classes should not be public

17 files changed, 18 insertions, 18 deletions

diff --git a/gee/abstractmultimap.vala b/gee/abstractmultimap.vala
index af60470..b777f4e 100644
--- a/gee/abstractmultimap.vala
+++ b/gee/abstractmultimap.vala
@@ -39,7 +39,7 @@ public abstract class Gee.AbstractMultiMap<K,V> : Object, MultiMap<K,V> {
 	protected Map<K, Collection<V>> _storage_map;
 	private int _nitems = 0;
 
-	public AbstractMultiMap (Map<K, Collection<V>> storage_map) {
+	protected AbstractMultiMap (Map<K, Collection<V>> storage_map) {
 		this._storage_map = storage_map;
 	}
 
diff --git a/gee/abstractmultiset.vala b/gee/abstractmultiset.vala
index b237814..ecdc8ac 100644
--- a/gee/abstractmultiset.vala
+++ b/gee/abstractmultiset.vala
@@ -42,7 +42,7 @@ public abstract class Gee.AbstractMultiSet<G> : AbstractCollection<G>, MultiSet<
 	/**
 	 * Constructs a new, empty abstract multi set.
 	 */
-	public AbstractMultiSet (Map<G, int> storage_map) {
+	protected AbstractMultiSet (Map<G, int> storage_map) {
 		this._storage_map = storage_map;
 	}
 
diff --git a/gee/hashmap.vala b/gee/hashmap.vala
index 83243a7..8d6b158 100644
--- a/gee/hashmap.vala
+++ b/gee/hashmap.vala
@@ -491,12 +491,12 @@ public class Gee.HashMap<K,V> : Gee.AbstractMap<K,V> {
 	}
 
 	private abstract class NodeIterator<K,V> : Object {
-		public NodeIterator (HashMap map) {
+		protected NodeIterator (HashMap map) {
 			_map = map;
 			_stamp = _map._stamp;
 		}
 
-		public NodeIterator.from_iterator (NodeIterator<K,V> iter) {
+		protected NodeIterator.from_iterator (NodeIterator<K,V> iter) {
 			_map = iter._map;
 			_index = iter._index;
 			_node = iter._node;
diff --git a/tests/testbidirlist.vala b/tests/testbidirlist.vala
index 7e53f5b..8ad3cc9 100644
--- a/tests/testbidirlist.vala
+++ b/tests/testbidirlist.vala
@@ -25,7 +25,7 @@ using Gee;
 
 public abstract class BidirListTests : ListTests {
 
-	public BidirListTests (string name) {
+	protected BidirListTests (string name) {
 		base (name);
 		add_test ("[List] bi-directional list iterator", test_bidir_list_iterator);
 	}
diff --git a/tests/testbidirsortedmap.vala b/tests/testbidirsortedmap.vala
index 385d715..7bbadf3 100644
--- a/tests/testbidirsortedmap.vala
+++ b/tests/testbidirsortedmap.vala
@@ -23,7 +23,7 @@ using GLib;
 using Gee;
 
 public abstract class BidirSortedMapTests : SortedMapTests {
-	public BidirSortedMapTests(string name) {
+	protected BidirSortedMapTests(string name) {
 		base (name);
 		add_test ("[SortedMap] bi-directional iterators can go backward",
 		          test_bidir_iterator_can_go_backward);
diff --git a/tests/testbidirsortedset.vala b/tests/testbidirsortedset.vala
index df5da60..1c8bc8e 100644
--- a/tests/testbidirsortedset.vala
+++ b/tests/testbidirsortedset.vala
@@ -23,7 +23,7 @@ using GLib;
 using Gee;
 
 public abstract class BidirSortedSetTests : SortedSetTests {
-	public BidirSortedSetTests(string name) {
+	protected BidirSortedSetTests(string name) {
 		base (name);
 		add_test ("[SortedSet] bi-directional iterators can go backward",
 		          test_bidir_iterator_can_go_backward);
diff --git a/tests/testcase.vala b/tests/testcase.vala
index 83a37ae..79a72a1 100644
--- a/tests/testcase.vala
+++ b/tests/testcase.vala
@@ -27,7 +27,7 @@ public abstract class Gee.TestCase : Object {
 
 	public delegate void TestMethod ();
 
-	public TestCase (string name) {
+	protected TestCase (string name) {
 		this.suite = new GLib.TestSuite (name);
 	}
 
diff --git a/tests/testcollection.vala b/tests/testcollection.vala
index 819e130..c2ce182 100644
--- a/tests/testcollection.vala
+++ b/tests/testcollection.vala
@@ -29,7 +29,7 @@ using Gee;
 
 public abstract class CollectionTests : Gee.TestCase {
 
-	public CollectionTests (string name) {
+	protected CollectionTests (string name) {
 		base (name);
 		add_test ("[Collection] type correctness", test_type_correctness);
 		add_test ("[Collection] iterator returns all elements once",
diff --git a/tests/testdeque.vala b/tests/testdeque.vala
index ff2d320..6dd4831 100644
--- a/tests/testdeque.vala
+++ b/tests/testdeque.vala
@@ -24,7 +24,7 @@ using Gee;
 
 public abstract class DequeTests : QueueTests {
 
-	public DequeTests (string name) {
+	protected DequeTests (string name) {
 		base (name);
 		add_test ("[Deque] queue use", test_queue_use);
 		add_test ("[Deque] stack use", test_stack_use);
diff --git a/tests/testlist.vala b/tests/testlist.vala
index 1873dfc..a2aa923 100644
--- a/tests/testlist.vala
+++ b/tests/testlist.vala
@@ -28,7 +28,7 @@ using Gee;
 
 public abstract class ListTests : CollectionTests {
 
-	public ListTests (string name) {
+	protected ListTests (string name) {
 		base (name);
 		add_test ("[List] iterator is ordered", test_iterator_is_ordered);
 		add_test ("[List] list iterator", test_list_iterator);
diff --git a/tests/testmap.vala b/tests/testmap.vala
index 468d477..36cc5f1 100644
--- a/tests/testmap.vala
+++ b/tests/testmap.vala
@@ -27,7 +27,7 @@ using Gee;
 
 public abstract class MapTests : Gee.TestCase {
 
-	public MapTests (string name) {
+	protected MapTests (string name) {
 		base (name);
 		add_test ("[Map] type correctness", test_type_correctness);
 		add_test ("[Map] has_key, size and is_empty",
diff --git a/tests/testmultimap.vala b/tests/testmultimap.vala
index 0531be6..c8bd4da 100644
--- a/tests/testmultimap.vala
+++ b/tests/testmultimap.vala
@@ -28,7 +28,7 @@ using Gee;
 
 public abstract class MultiMapTests : Gee.TestCase {
 
-	public MultiMapTests (string name) {
+	protected MultiMapTests (string name) {
 		base (name);
 		add_test ("[MultiMap] type correctness", test_type_correctness);
 		add_test ("[MultiMap] size", test_size);
diff --git a/tests/testmultiset.vala b/tests/testmultiset.vala
index 6812bdd..2cb965c 100644
--- a/tests/testmultiset.vala
+++ b/tests/testmultiset.vala
@@ -28,7 +28,7 @@ using Gee;
 
 public abstract class MultiSetTests : CollectionTests {
 
-	public MultiSetTests (string name) {
+	protected MultiSetTests (string name) {
 		base (name);
 		add_test ("[MultiSet] duplicates are retained",
 		          test_duplicates_are_retained);
diff --git a/tests/testqueue.vala b/tests/testqueue.vala
index aaeb545..a0069b6 100644
--- a/tests/testqueue.vala
+++ b/tests/testqueue.vala
@@ -24,7 +24,7 @@ using Gee;
 
 public abstract class QueueTests : CollectionTests {
 
-	public QueueTests (string name) {
+	protected QueueTests (string name) {
 		base (name);
 		add_test ("[Queue] capacity bound", test_capacity_bound);
 		add_test ("[Queue] one element operation", test_one_element_operation);
diff --git a/tests/testset.vala b/tests/testset.vala
index e2fc83f..eb07707 100644
--- a/tests/testset.vala
+++ b/tests/testset.vala
@@ -28,7 +28,7 @@ using Gee;
 
 public abstract class SetTests : CollectionTests {
 
-	public SetTests (string name) {
+	protected SetTests (string name) {
 		base (name);
 		add_test ("[Set] duplicates are ignored", test_duplicates_are_ignored);
 	}
diff --git a/tests/testsortedmap.vala b/tests/testsortedmap.vala
index abfbfc8..4574a99 100644
--- a/tests/testsortedmap.vala
+++ b/tests/testsortedmap.vala
@@ -21,7 +21,7 @@
  */
 
 public abstract class Gee.SortedMapTests : MapTests {
-	public SortedMapTests (string name) {
+	protected SortedMapTests (string name) {
 		base (name);
 		add_test ("[SortedMap] key ordering", test_key_ordering);
 		add_test ("[SortedMap] first", test_first);
diff --git a/tests/testsortedset.vala b/tests/testsortedset.vala
index 00c9bd2..029a516 100644
--- a/tests/testsortedset.vala
+++ b/tests/testsortedset.vala
@@ -24,7 +24,7 @@ using GLib;
 using Gee;
 
 public abstract class SortedSetTests : SetTests {
-	public SortedSetTests (string name, bool strict = true) {
+	protected SortedSetTests (string name, bool strict = true) {
 		base (name);
 		this.strict = strict;
 		add_test ("[SortedSet] first", test_first);