[epiphany/wip/sync: 74/86] synchronizable: Remove GError parameter from _to_bso() and _from_bso()



commit 588d7258a4ecbe688d947f41b16a7170ad738d39
Author: Gabriel Ivascu <ivascu gabriel59 gmail com>
Date:   Tue Mar 21 17:17:48 2017 +0200

    synchronizable: Remove GError parameter from _to_bso() and _from_bso()

 src/bookmarks/ephy-bookmark.c  |    3 +-
 src/sync/ephy-synchronizable.c |   12 +++------
 src/sync/ephy-synchronizable.h |   47 ++++++++++++++++++---------------------
 3 files changed, 27 insertions(+), 35 deletions(-)
---
diff --git a/src/bookmarks/ephy-bookmark.c b/src/bookmarks/ephy-bookmark.c
index 198118a..51ecf12 100644
--- a/src/bookmarks/ephy-bookmark.c
+++ b/src/bookmarks/ephy-bookmark.c
@@ -346,8 +346,7 @@ ephy_bookmark_synchronizable_set_is_uploaded (EphySynchronizable *synchronizable
 }
 
 static char *
-ephy_bookmark_synchronizable_to_bso (EphySynchronizable  *synchronizable,
-                                     GError             **error)
+ephy_bookmark_synchronizable_to_bso (EphySynchronizable *synchronizable)
 {
   return NULL;
 }
diff --git a/src/sync/ephy-synchronizable.c b/src/sync/ephy-synchronizable.c
index 735cfc3..75936cf 100644
--- a/src/sync/ephy-synchronizable.c
+++ b/src/sync/ephy-synchronizable.c
@@ -164,7 +164,6 @@ ephy_synchronizable_set_is_uploaded (EphySynchronizable *synchronizable,
 /**
  * ephy_synchronizable_to_bso:
  * @synchronizable: an #EphySynchronizable
- * @error: return location for a #GError, or %NULL
  *
  * Converts an #EphySynchronizable into its string representation
  * of a Basic Storage Object from the client's point of view
@@ -175,22 +174,20 @@ ephy_synchronizable_set_is_uploaded (EphySynchronizable *synchronizable,
  * Return value: (transfer full): @synchronizable's BSO's string representation
  **/
 char *
-ephy_synchronizable_to_bso (EphySynchronizable  *synchronizable,
-                            GError             **error)
+ephy_synchronizable_to_bso (EphySynchronizable *synchronizable)
 {
   EphySynchronizableInterface *iface;
 
   g_return_val_if_fail (EPHY_IS_SYNCHRONIZABLE (synchronizable), NULL);
 
   iface = EPHY_SYNCHRONIZABLE_GET_IFACE (synchronizable);
-  return iface->to_bso (synchronizable, error);
+  return iface->to_bso (synchronizable);
 }
 
 /**
  * ephy_synchronizable_from_bso:
  * @bso: an #JsonObject holding the JSON representation of a Basic Storage Object
  * @gtype: the #GType of object to construct
- * @error: return location for a #GError, or %NULL
  *
  * Converts a JSON representation of a Basic Storage Object
  * from the server's point of view (i.e. the %modified field is present)
@@ -205,9 +202,8 @@ ephy_synchronizable_to_bso (EphySynchronizable  *synchronizable,
  *  Return value: (transfer full): a #GObject or %NULL
  **/
 GObject *
-ephy_synchronizable_from_bso (JsonObject  *bso,
-                              GType        gtype,
-                              GError     **error)
+ephy_synchronizable_from_bso (JsonObject *bso,
+                              GType       gtype)
 {
   g_return_val_if_fail (bso, NULL);
 
diff --git a/src/sync/ephy-synchronizable.h b/src/sync/ephy-synchronizable.h
index 4a4a4dd..241c978 100644
--- a/src/sync/ephy-synchronizable.h
+++ b/src/sync/ephy-synchronizable.h
@@ -36,37 +36,34 @@ G_DECLARE_INTERFACE (EphySynchronizable, ephy_synchronizable, EPHY, SYNCHRONIZAB
 struct _EphySynchronizableInterface {
   GTypeInterface parent_iface;
 
-  const char * (*get_id)                  (EphySynchronizable  *synchronizable);
-  void         (*set_id)                  (EphySynchronizable  *synchronizable,
-                                           const char          *id);
+  const char * (*get_id)                  (EphySynchronizable *synchronizable);
+  void         (*set_id)                  (EphySynchronizable *synchronizable,
+                                           const char         *id);
 
-  double       (*get_modification_time)   (EphySynchronizable  *synchronizable);
-  void         (*set_modification_time)   (EphySynchronizable  *synchronizable,
-                                           double               modified);
+  double       (*get_modification_time)   (EphySynchronizable *synchronizable);
+  void         (*set_modification_time)   (EphySynchronizable *synchronizable,
+                                           double              modified);
 
-  gboolean     (*is_uploaded)             (EphySynchronizable  *synchronizable);
-  void         (*set_is_uploaded)         (EphySynchronizable  *synchronizable,
-                                           gboolean             uploaded);
+  gboolean     (*is_uploaded)             (EphySynchronizable *synchronizable);
+  void         (*set_is_uploaded)         (EphySynchronizable *synchronizable,
+                                           gboolean            uploaded);
 
-  char *       (*to_bso)                  (EphySynchronizable  *synchronizable,
-                                           GError             **error);
+  char *       (*to_bso)                  (EphySynchronizable *synchronizable);
 };
 
-const char          *ephy_synchronizable_get_id                 (EphySynchronizable  *synchronizable);
-void                 ephy_synchronizable_set_id                 (EphySynchronizable  *synchronizable,
-                                                                 const char          *id);
-double               ephy_synchronizable_get_modification_time  (EphySynchronizable  *synchronizable);
-void                 ephy_synchronizable_set_modification_time  (EphySynchronizable  *synchronizable,
-                                                                 double               modified);
-gboolean             ephy_synchronizable_is_uploaded            (EphySynchronizable  *synchronizable);
-void                 ephy_synchronizable_set_is_uploaded        (EphySynchronizable  *synchronizable,
-                                                                 gboolean             uploaded);
-char                *ephy_synchronizable_to_bso                 (EphySynchronizable  *synchronizable,
-                                                                 GError             **error);
+const char          *ephy_synchronizable_get_id                 (EphySynchronizable *synchronizable);
+void                 ephy_synchronizable_set_id                 (EphySynchronizable *synchronizable,
+                                                                 const char         *id);
+double               ephy_synchronizable_get_modification_time  (EphySynchronizable *synchronizable);
+void                 ephy_synchronizable_set_modification_time  (EphySynchronizable *synchronizable,
+                                                                 double              modified);
+gboolean             ephy_synchronizable_is_uploaded            (EphySynchronizable *synchronizable);
+void                 ephy_synchronizable_set_is_uploaded        (EphySynchronizable *synchronizable,
+                                                                 gboolean            uploaded);
+char                *ephy_synchronizable_to_bso                 (EphySynchronizable *synchronizable);
 /* This can't be an interface method because we lack the EphySynchronizable object.
  * Think of it as more of an utility function. */
-GObject             *ephy_synchronizable_from_bso   (JsonObject  *bso,
-                                                     GType        gtype,
-                                                     GError     **error);
+GObject             *ephy_synchronizable_from_bso   (JsonObject *bso,
+                                                     GType       gtype);
 
 G_END_DECLS


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]