ges: Keep ges-gerror categories simple.
authorThibault Saunier <thibault.saunier@collabora.com>
Fri, 22 Mar 2013 20:39:04 +0000 (17:39 -0300)
committerThibault Saunier <thibault.saunier@collabora.com>
Fri, 22 Mar 2013 20:39:04 +0000 (17:39 -0300)
docs/libs/ges-sections.txt
ges/ges-asset.c
ges/ges-base-xml-formatter.c
ges/ges-gerror.h
ges/ges-uri-asset.c

index 3a76b7cd55d3125b4cf2b4db5208f7eec7d374e8..0f64f48afa0b31cf8b0e2b47cc8dc108c6c9cdc8 100644 (file)
@@ -24,10 +24,8 @@ GESAssetLoadingReturn
 <SECTION>
 <FILE>ges-gerror</FILE>
 <TITLE>GES GErrors</TITLE>
-GES_ASSET_ERROR
-GES_FORMATTER_ERROR
-GESAssetError
-GESFormatterError
+GES_ERROR
+GESError
 </SECTION>
 
 <SECTION>
index 359661dc1d91cfc98745cf94f9c4a37c67c48558..6be3e3faf2f1de09878e866f97c8a56c9561e68f 100644 (file)
@@ -171,7 +171,7 @@ _check_and_update_parameters (GType * extractable_type, const gchar * id,
         g_type_name (old_type));
 
     if (error && *error == NULL)
-      g_set_error (error, GES_ASSET_ERROR, GES_ASSET_WRONG_ID,
+      g_set_error (error, GES_ERROR, GES_ERROR_ASSET_WRONG_ID,
           "Wrong ID, can not find any extractable_type");
     return NULL;
   }
@@ -182,7 +182,7 @@ _check_and_update_parameters (GType * extractable_type, const gchar * id,
 
     g_free (real_id);
     if (error && *error == NULL)
-      g_set_error (error, GES_ASSET_ERROR, GES_ASSET_WRONG_ID, "Wrong ID");
+      g_set_error (error, GES_ERROR, GES_ERROR_ASSET_WRONG_ID, "Wrong ID");
 
     return NULL;
   }
@@ -224,7 +224,7 @@ async_initable_init_async (GAsyncInitable * initable, gint io_priority,
     case GES_ASSET_LOADING_ERROR:
     {
       if (error == NULL)
-        g_set_error (&error, GES_ASSET_ERROR, GES_ASSET_ERROR_LOADING,
+        g_set_error (&error, GES_ERROR, GES_ERROR_ASSET_LOADING,
             "Could not start loading asset");
 
       /* FIXME Define error code */
index a821bd2c9398024ffbcaf9d55369c6848f234646..81b82f56d918d15020b14e8430071b5bd0df676b 100644 (file)
@@ -688,7 +688,7 @@ ges_base_xml_formatter_add_clip (GESBaseXmlFormatter * self,
 
   entry = g_hash_table_lookup (priv->layers, GINT_TO_POINTER (layer_prio));
   if (entry == NULL) {
-    g_set_error (error, GES_FORMATTER_ERROR, GES_FORMATTER_WRONG_INPUT_FILE,
+    g_set_error (error, GES_ERROR, GES_ERROR_FORMATTER_MALFORMED_INPUT_FILE,
         "We got a Clip in a layer"
         " that does not exist, something is wrong either in the project file or"
         " in %s", g_type_name (G_OBJECT_TYPE (self)));
index 4052a53c19514a5491094aaedc77723cfbc3fb7e..1fce4237b734ce28a85b4d90b478db525378f357 100644 (file)
 G_BEGIN_DECLS
 
 /**
- * GES_ASSET_ERROR:
+ * GES_ERROR:
  *
- * An error happend using an asset
+ * An error happend in GES
  */
-#define GES_ASSET_ERROR g_quark_from_static_string("GES_ASSET_ERROR")
+#define GES_ERROR g_quark_from_static_string("GES_ERROR")
 
 /**
- * GES_FORMATTER_ERROR:
- *
- * An error happend using a formatter
- */
-#define GES_FORMATTER_ERROR g_quark_from_static_string("GES_FORMATTER_ERROR")
-
-/**
- * GESAssetError:
- * @GES_ASSET_WRONG_ID: The ID passed is malformed
- * @GES_ASSET_ERROR_LOADING: An error happened while loading the asset
- */
-typedef enum
-{
-  GES_ASSET_WRONG_ID,
-  GES_ASSET_ERROR_LOADING
-} GESAssetError;
-
-/**
- * GESFormatterError:
- * @GES_FORMATTER_WRONG_INPUT_FILE: The formatted files was malformed
+ * GESError:
+ * @GES_ERROR_ASSET_WRONG_ID: The ID passed is malformed
+ * @GES_ERROR_ASSET_LOADING: An error happened while loading the asset
+ * @GES_ERROR_FORMATTER_MALFORMED_INPUT_FILE: The formatted files was malformed
  */
 typedef enum
 {
-  GES_FORMATTER_WRONG_INPUT_FILE,
-} GESFormatterError;
+  GES_ERROR_ASSET_WRONG_ID,
+  GES_ERROR_ASSET_LOADING,
+  GES_ERROR_FORMATTER_MALFORMED_INPUT_FILE,
+} GESError;
 
 G_END_DECLS
 #endif /* __GES_ERROR_H__ */
index 4a2a790ba0bfe358dd45ceb0ac288978b1471e24..1f739c4f3b0a21e9e0ffdfffc692d9b327583671 100644 (file)
@@ -448,8 +448,8 @@ ges_uri_clip_asset_request_sync (const gchar * uri, GError ** error)
   if (asset)
     return asset;
 
-  if (lerror && lerror->domain == GES_ASSET_ERROR &&
-      lerror->code == GES_ASSET_WRONG_ID) {
+  if (lerror && lerror->domain == GES_ERROR &&
+      lerror->code == GES_ERROR_ASSET_WRONG_ID) {
     g_propagate_error (error, lerror);
 
     return NULL;