eds: Minor API fixes in the EDS backend
authorPhilip Withnall <philip@tecnocode.co.uk>
Thu, 9 Aug 2012 23:11:20 +0000 (00:11 +0100)
committerPhilip Withnall <philip@tecnocode.co.uk>
Thu, 9 Aug 2012 23:26:02 +0000 (00:26 +0100)
These APIs, introduced in folks 0.7.1 (so this is an unstable API break,
but that’s OK), weren’t quite right in C. The SourceRegistry constructor
shouldn’t be exposed publicly as part of folks, and the PersonaStore
constructor had a stray extra parameter.

backends/eds/lib/edsf-persona-store.vala

index 4ea2ecb..2b727db 100644 (file)
@@ -31,7 +31,7 @@ extern const string BACKEND_NAME;
 /* The following function is needed in order to use the async SourceRegistry
  * constructor. FIXME: https://bugzilla.gnome.org/show_bug.cgi?id=659886 */
 [CCode (cname = "e_source_registry_new", cheader_filename = "libedataserver/libedataserver.h", finish_function = "e_source_registry_new_finish")]
-public extern static async E.SourceRegistry create_source_registry (GLib.Cancellable? cancellable = null) throws GLib.Error;
+internal extern static async E.SourceRegistry create_source_registry (GLib.Cancellable? cancellable = null) throws GLib.Error;
 
 /**
  * A persona store.
@@ -227,7 +227,7 @@ public class Edsf.PersonaStore : Folks.PersonaStore
    */
   [Deprecated (since = "0.7.2",
       replacement = "Edsf.PersonaStore.with_source_registry()")]
-  public PersonaStore (E.SourceRegistry r, E.Source s)
+  public PersonaStore (E.Source s)
     {
       string eds_uid = s.get_uid ();
       string eds_name = s.get_display_name ();