diff options
author | Travis Reitter <travis.reitter@collabora.co.uk> | 2011-02-03 11:41:38 -0800 |
---|---|---|
committer | Philip Withnall <philip.withnall@collabora.co.uk> | 2011-04-23 22:00:09 +0100 |
commit | eca4e5453097b76557533eadab8d21b7aca90c1a (patch) | |
tree | 157040fa6acfe98c1f607e6aa8e3475a888b1437 | |
parent | cc7b9ac3b058d0734f2df749abc50e71d5797149 (diff) |
Remove use of GLib.List in public BackendStore API.
Helps bgo#640092 - Folks should minimize use of GLib.List (including its public
API)
-rw-r--r-- | NEWS | 4 | ||||
-rw-r--r-- | folks/backend-store.vala | 16 | ||||
-rw-r--r-- | tests/folks/backend-loading.vala | 44 | ||||
-rw-r--r-- | tests/telepathy/individual-retrieval.vala | 2 | ||||
-rw-r--r-- | tests/tracker/add-persona.vala | 2 | ||||
-rw-r--r-- | tests/tracker/duplicated-emails.vala | 2 | ||||
-rw-r--r-- | tests/tracker/duplicated-phones.vala | 2 | ||||
-rw-r--r-- | tests/tracker/link-personas-via-local-ids.vala | 2 | ||||
-rw-r--r-- | tests/tracker/link-personas.vala | 2 | ||||
-rw-r--r-- | tests/tracker/match-all.vala | 2 | ||||
-rw-r--r-- | tests/tracker/match-email-addresses.vala | 2 | ||||
-rw-r--r-- | tests/tracker/match-im-addresses.vala | 2 | ||||
-rw-r--r-- | tests/tracker/match-known-emails.vala | 2 | ||||
-rw-r--r-- | tests/tracker/match-name.vala | 2 | ||||
-rw-r--r-- | tests/tracker/match-phone-number.vala | 2 | ||||
-rw-r--r-- | tests/tracker/remove-persona.vala | 2 | ||||
-rw-r--r-- | tests/tracker/set-duplicate-email.vala | 2 |
17 files changed, 40 insertions, 52 deletions
@@ -25,6 +25,10 @@ API changes: * EmailDetails.email_addresses is now of type Set<FieldDetails> * PhoneDetails.phone_numbers is now of type Set<FieldDetails> * UrlDetails.urls is now of type Set<FieldDetails> +* BackendStore.enabled_backends now has type Map<string, Backend> + - The corresponding C function folks_backend_store_dup_enabled_backends() + has been renamed folks_backend_store_get_enabled_backends(), and no longer + returns an owned variable. Overview of changes from libfolks 0.4.0 to libfolks 0.5.0 ========================================================= diff --git a/folks/backend-store.vala b/folks/backend-store.vala index b36a334..8fa7dcc 100644 --- a/folks/backend-store.vala +++ b/folks/backend-store.vala @@ -43,7 +43,7 @@ public class Folks.BackendStore : Object { /* this contains all backends, regardless of enabled or prepared state */ private HashMap<string,Backend> _backend_hash; - private HashMap<string,Backend> _prepared_backends; + private HashMap<string, Backend> _prepared_backends; private File _config_file; private GLib.KeyFile _backends_key_file; private HashMap<string,unowned Module> _modules; @@ -87,19 +87,11 @@ public class Folks.BackendStore : Object { * * The backends in this list have been prepared and are ready to use. * - * @since 0.3.5 + * @since UNRELEASED */ - public GLib.List<Backend> enabled_backends + public Map<string, Backend> enabled_backends { - [CCode (cname = "folks_backend_store_dup_enabled_backends")] - owned get - { - var backends = new GLib.List<Backend> (); - foreach (var backend in this._prepared_backends.values) - backends.prepend (backend); - - return backends; - } + get { return this._prepared_backends; } private set {} } diff --git a/tests/folks/backend-loading.vala b/tests/folks/backend-loading.vala index 3fd9adb..92ecc9b 100644 --- a/tests/folks/backend-loading.vala +++ b/tests/folks/backend-loading.vala @@ -83,13 +83,11 @@ public class BackendLoadingTests : Folks.TestCase { store.load_backends.end (r); - store.enabled_backends.foreach ((i) => + foreach (var backend in store.enabled_backends.values) { - var backend = (Backend) i; - assert (backends_expected.contains (backend.name)); backends_expected.remove (backend.name); - }); + } assert (backends_expected.size == 0); main_loop.quit (); @@ -129,12 +127,11 @@ public class BackendLoadingTests : Folks.TestCase { yield store.load_backends (); - store.enabled_backends.foreach ((i) => + foreach (var backend in store.enabled_backends.values) { - var backend = (Backend) i; assert (backends_expected.contains (backend.name)); backends_expected.remove (backend.name); - }); + } assert (backends_expected.size == 0); this.main_loop.quit (); @@ -173,12 +170,11 @@ public class BackendLoadingTests : Folks.TestCase { yield store.load_backends (); - store.enabled_backends.foreach ((i) => + foreach (var backend1 in store.enabled_backends.values) { - var backend = (Backend) i; - assert (backends_expected.contains (backend.name)); - backends_expected.remove (backend.name); - }); + assert (backends_expected.contains (backend1.name)); + backends_expected.remove (backend1.name); + } assert (backends_expected.size == 0); } @@ -201,12 +197,11 @@ public class BackendLoadingTests : Folks.TestCase { yield store.load_backends (); - store.enabled_backends.foreach ((i) => + foreach (var backend2 in store.enabled_backends.values) { - var backend = (Backend) i; - assert (backends_expected.contains (backend.name)); - backends_expected.remove (backend.name); - }); + assert (backends_expected.contains (backend2.name)); + backends_expected.remove (backend2.name); + } assert (backends_expected.size == 0); } @@ -230,12 +225,11 @@ public class BackendLoadingTests : Folks.TestCase { yield store.load_backends (); - store.enabled_backends.foreach ((i) => + foreach (var backend3 in store.enabled_backends.values) { - var backend = (Backend) i; - assert (backends_expected.contains (backend.name)); - backends_expected.remove (backend.name); - }); + assert (backends_expected.contains (backend3.name)); + backends_expected.remove (backend3.name); + } assert (backends_expected.size == 0); } @@ -257,13 +251,11 @@ public class BackendLoadingTests : Folks.TestCase { yield store.load_backends (); - store.enabled_backends.foreach ((i) => + foreach (var backend in store.enabled_backends.values) { - var backend = (Backend) i; - assert (backends_expected.contains (backend.name)); backends_expected.remove (backend.name); - }); + } assert (backends_expected.size == 0); } diff --git a/tests/telepathy/individual-retrieval.vala b/tests/telepathy/individual-retrieval.vala index 416ae7d..48bde9e 100644 --- a/tests/telepathy/individual-retrieval.vala +++ b/tests/telepathy/individual-retrieval.vala @@ -175,7 +175,7 @@ public class IndividualRetrievalTests : Folks.TestCase /* once we see a valid Tpf.PersonaStore, add our new personas */ var backend_store = BackendStore.dup (); - foreach (var backend in backend_store.enabled_backends) + foreach (var backend in backend_store.enabled_backends.values) { /* PersonaStores can be added after the backend is prepared */ backend.persona_store_added.connect ((store) => diff --git a/tests/tracker/add-persona.vala b/tests/tracker/add-persona.vala index e1c2cc0..40e173b 100644 --- a/tests/tracker/add-persona.vala +++ b/tests/tracker/add-persona.vala @@ -156,7 +156,7 @@ public class AddPersonaTests : Folks.TestCase yield this._aggregator.prepare (); this._pstore = null; - foreach (var backend in store.enabled_backends) + foreach (var backend in store.enabled_backends.values) { this._pstore = (Trf.PersonaStore) backend.persona_stores.get ("tracker"); diff --git a/tests/tracker/duplicated-emails.vala b/tests/tracker/duplicated-emails.vala index 6aa33f4..d7fbeff 100644 --- a/tests/tracker/duplicated-emails.vala +++ b/tests/tracker/duplicated-emails.vala @@ -83,7 +83,7 @@ public class DuplicatedEmailsTests : Folks.TestCase { yield this._aggregator.prepare (); this._pstore = null; - foreach (var backend in store.enabled_backends) + foreach (var backend in store.enabled_backends.values) { this._pstore = (Trf.PersonaStore) backend.persona_stores.get ("tracker"); diff --git a/tests/tracker/duplicated-phones.vala b/tests/tracker/duplicated-phones.vala index 24ff061..cbecc44 100644 --- a/tests/tracker/duplicated-phones.vala +++ b/tests/tracker/duplicated-phones.vala @@ -83,7 +83,7 @@ public class DuplicatedPhonesTests : Folks.TestCase { yield this._aggregator.prepare (); this._pstore = null; - foreach (var backend in store.enabled_backends) + foreach (var backend in store.enabled_backends.values) { this._pstore = (Trf.PersonaStore) backend.persona_stores.get ("tracker"); diff --git a/tests/tracker/link-personas-via-local-ids.vala b/tests/tracker/link-personas-via-local-ids.vala index 6280637..0046f49 100644 --- a/tests/tracker/link-personas-via-local-ids.vala +++ b/tests/tracker/link-personas-via-local-ids.vala @@ -135,7 +135,7 @@ public class LinkPersonasViaLocalIDsTests : Folks.TestCase yield this._aggregator.prepare (); PersonaStore pstore = null; - foreach (var backend in store.enabled_backends) + foreach (var backend in store.enabled_backends.values) { pstore = backend.persona_stores.get ("tracker"); if (pstore != null) diff --git a/tests/tracker/link-personas.vala b/tests/tracker/link-personas.vala index 8c19082..e966a52 100644 --- a/tests/tracker/link-personas.vala +++ b/tests/tracker/link-personas.vala @@ -139,7 +139,7 @@ public class LinkPersonasTests : Folks.TestCase yield this._aggregator.prepare (); PersonaStore pstore = null; - foreach (var backend in store.enabled_backends) + foreach (var backend in store.enabled_backends.values) { pstore = backend.persona_stores.get ("tracker"); if (pstore != null) diff --git a/tests/tracker/match-all.vala b/tests/tracker/match-all.vala index c9c8f35..7c8a4ac 100644 --- a/tests/tracker/match-all.vala +++ b/tests/tracker/match-all.vala @@ -109,7 +109,7 @@ public class MatchAllTests : Folks.TestCase { yield this._aggregator.prepare (); this._pstore = null; - foreach (var backend in store.enabled_backends) + foreach (var backend in store.enabled_backends.values) { this._pstore = (Trf.PersonaStore) backend.persona_stores.get ("tracker"); diff --git a/tests/tracker/match-email-addresses.vala b/tests/tracker/match-email-addresses.vala index 6b7727d..528be46 100644 --- a/tests/tracker/match-email-addresses.vala +++ b/tests/tracker/match-email-addresses.vala @@ -84,7 +84,7 @@ public class MatchEmailAddressesTests : Folks.TestCase { yield this._aggregator.prepare (); this._pstore = null; - foreach (var backend in store.enabled_backends) + foreach (var backend in store.enabled_backends.values) { this._pstore = (Trf.PersonaStore) backend.persona_stores.get ("tracker"); diff --git a/tests/tracker/match-im-addresses.vala b/tests/tracker/match-im-addresses.vala index 1e2ede2..c2b9784 100644 --- a/tests/tracker/match-im-addresses.vala +++ b/tests/tracker/match-im-addresses.vala @@ -84,7 +84,7 @@ public class MatchIMAddressesTests : Folks.TestCase { yield this._aggregator.prepare (); this._pstore = null; - foreach (var backend in store.enabled_backends) + foreach (var backend in store.enabled_backends.values) { this._pstore = (Trf.PersonaStore) backend.persona_stores.get ("tracker"); diff --git a/tests/tracker/match-known-emails.vala b/tests/tracker/match-known-emails.vala index f0068f8..31a7557 100644 --- a/tests/tracker/match-known-emails.vala +++ b/tests/tracker/match-known-emails.vala @@ -92,7 +92,7 @@ public class MatchKnownEmailsTests : Folks.TestCase { yield this._aggregator.prepare (); this._pstore = null; - foreach (var backend in store.enabled_backends) + foreach (var backend in store.enabled_backends.values) { this._pstore = (Trf.PersonaStore) backend.persona_stores.get ("tracker"); diff --git a/tests/tracker/match-name.vala b/tests/tracker/match-name.vala index f7b67d2..702488b 100644 --- a/tests/tracker/match-name.vala +++ b/tests/tracker/match-name.vala @@ -147,7 +147,7 @@ public class MatchNameTests : Folks.TestCase { yield this._aggregator.prepare (); this._pstore = null; - foreach (var backend in store.enabled_backends) + foreach (var backend in store.enabled_backends.values) { this._pstore = (Trf.PersonaStore) backend.persona_stores.get ("tracker"); diff --git a/tests/tracker/match-phone-number.vala b/tests/tracker/match-phone-number.vala index 2b718ba..2bd0d07 100644 --- a/tests/tracker/match-phone-number.vala +++ b/tests/tracker/match-phone-number.vala @@ -86,7 +86,7 @@ public class MatchPhoneNumberTests : Folks.TestCase { yield this._aggregator.prepare (); this._pstore = null; - foreach (var backend in store.enabled_backends) + foreach (var backend in store.enabled_backends.values) { this._pstore = (Trf.PersonaStore) backend.persona_stores.get ("tracker"); diff --git a/tests/tracker/remove-persona.vala b/tests/tracker/remove-persona.vala index 2ca4484..0214dae 100644 --- a/tests/tracker/remove-persona.vala +++ b/tests/tracker/remove-persona.vala @@ -90,7 +90,7 @@ public class RemovePersonaTests : Folks.TestCase yield this._aggregator.prepare (); this._pstore = null; - foreach (var backend in store.enabled_backends) + foreach (var backend in store.enabled_backends.values) { this._pstore = backend.persona_stores.get ("tracker"); if (this._pstore != null) diff --git a/tests/tracker/set-duplicate-email.vala b/tests/tracker/set-duplicate-email.vala index 7ff76a9..fe3a51e 100644 --- a/tests/tracker/set-duplicate-email.vala +++ b/tests/tracker/set-duplicate-email.vala @@ -84,7 +84,7 @@ public class SetDuplicateEmailTests : Folks.TestCase { yield this._aggregator.prepare (); this._pstore = null; - foreach (var backend in store.enabled_backends) + foreach (var backend in store.enabled_backends.values) { this._pstore = (Trf.PersonaStore) backend.persona_stores.get ("tracker"); |