diff options
author | Richard Hughes <richard@hughsie.com> | 2016-07-27 10:54:02 +0100 |
---|---|---|
committer | Richard Hughes <richard@hughsie.com> | 2016-07-27 10:54:02 +0100 |
commit | e5abe0fda12589d53612dae9aebe77f21ebfc2e9 (patch) | |
tree | 69fc328cf42bc780a1dfd0bbd6f36b9df8635765 | |
parent | c7fa3a21c0ca2719a397b191d742e88a45a265bd (diff) | |
download | appstream-glib-e5abe0fda12589d53612dae9aebe77f21ebfc2e9.tar.gz |
trivial: Remove AsReview.karma as it's implementation specific
-rw-r--r-- | libappstream-glib/as-review.c | 66 | ||||
-rw-r--r-- | libappstream-glib/as-review.h | 3 | ||||
-rw-r--r-- | libappstream-glib/as-self-test.c | 3 |
3 files changed, 1 insertions, 71 deletions
diff --git a/libappstream-glib/as-review.c b/libappstream-glib/as-review.c index ed739da..9488ff6 100644 --- a/libappstream-glib/as-review.c +++ b/libappstream-glib/as-review.c @@ -46,7 +46,6 @@ typedef struct gchar *summary; gchar *description; gchar *locale; - gint karma; gint priority; gint rating; gchar *version; @@ -59,7 +58,6 @@ typedef struct enum { PROP_0, PROP_ID, - PROP_KARMA, PROP_SUMMARY, PROP_DESCRIPTION, PROP_LOCALE, @@ -103,9 +101,6 @@ as_review_get_property (GObject *object, guint prop_id, AsReviewPrivate *priv = GET_PRIVATE (review); switch (prop_id) { - case PROP_KARMA: - g_value_set_int (value, priv->karma); - break; case PROP_ID: g_value_set_string (value, priv->id); break; @@ -149,9 +144,6 @@ as_review_set_property (GObject *object, guint prop_id, AsReview *review = AS_REVIEW (object); switch (prop_id) { - case PROP_KARMA: - as_review_set_karma (review, g_value_get_int (value)); - break; case PROP_ID: as_review_set_id (review, g_value_get_string (value)); break; @@ -198,16 +190,6 @@ as_review_class_init (AsReviewClass *klass) object_class->set_property = as_review_set_property; /** - * AsReview:karma: - * - * Since: 0.5.18 - **/ - pspec = g_param_spec_int ("karma", NULL, NULL, - G_MININT, G_MAXINT, 0, - G_PARAM_READWRITE | G_PARAM_CONSTRUCT); - g_object_class_install_property (object_class, PROP_KARMA, pspec); - - /** * AsReview:id: * * Since: 0.5.18 @@ -319,45 +301,6 @@ as_review_init (AsReview *review) } /** - * as_review_get_karma: - * @review: a #AsReview - * - * Gets the karma for the review, where positive numbers indicate - * more positive feedback for the review. - * - * Returns: the karma value, or 0 for unset. - * - * Since: 0.5.18 - **/ -gint -as_review_get_karma (AsReview *review) -{ - AsReviewPrivate *priv = GET_PRIVATE (review); - g_return_val_if_fail (AS_IS_REVIEW (review), 0); - return priv->karma; -} - -/** - * as_review_set_karma: - * @review: a #AsReview - * @karma: a karma value - * - * Sets the karma for the review, where positive numbers indicate - * more positive feedback for the review. - * - * Karma can be positive or negative, or 0 for unset. - * - * Since: 0.5.18 - **/ -void -as_review_set_karma (AsReview *review, gint karma) -{ - AsReviewPrivate *priv = GET_PRIVATE (review); - g_return_if_fail (AS_IS_REVIEW (review)); - priv->karma = karma; -} - -/** * as_review_get_priority: * @review: a #AsReview * @@ -838,8 +781,6 @@ as_review_equal (AsReview *review1, AsReview *review2) /* check for equality */ if (!g_date_time_equal (priv1->date, priv2->date)) return FALSE; - if (priv1->karma != priv2->karma) - return FALSE; if (priv1->priority != priv2->priority) return FALSE; if (priv1->rating != priv2->rating) @@ -882,10 +823,6 @@ as_review_node_insert (AsReview *review, GNode *parent, AsNodeContext *ctx) NULL); if (priv->id != NULL) as_node_add_attribute (n, "id", priv->id); - if (priv->karma != 0) { - g_autofree gchar *str = g_strdup_printf ("%i", priv->karma); - as_node_add_attribute (n, "karma", str); - } if (priv->priority != 0) { g_autofree gchar *str = g_strdup_printf ("%i", priv->priority); as_node_insert (n, "priority", str, @@ -963,9 +900,6 @@ as_review_node_parse (AsReview *review, GNode *node, const gchar *tmp; gint itmp; - itmp = as_node_get_attribute_as_int (node, "karma"); - if (itmp != G_MAXINT) - as_review_set_karma (review, itmp); itmp = as_node_get_attribute_as_int (node, "rating"); if (itmp != G_MAXINT) as_review_set_rating (review, itmp); diff --git a/libappstream-glib/as-review.h b/libappstream-glib/as-review.h index 35026bc..e19e9c8 100644 --- a/libappstream-glib/as-review.h +++ b/libappstream-glib/as-review.h @@ -69,7 +69,6 @@ typedef enum { AsReview *as_review_new (void); /* getters */ -gint as_review_get_karma (AsReview *review); gint as_review_get_priority (AsReview *review); const gchar *as_review_get_id (AsReview *review); const gchar *as_review_get_summary (AsReview *review); @@ -85,8 +84,6 @@ const gchar *as_review_get_metadata_item (AsReview *review, const gchar *key); /* setters */ -void as_review_set_karma (AsReview *review, - gint karma); void as_review_set_priority (AsReview *review, gint priority); void as_review_set_id (AsReview *review, diff --git a/libappstream-glib/as-self-test.c b/libappstream-glib/as-self-test.c index 87ec000..7ab1b33 100644 --- a/libappstream-glib/as-self-test.c +++ b/libappstream-glib/as-self-test.c @@ -1181,7 +1181,7 @@ as_test_review_func (void) AsNode *root; GString *xml; const gchar *src = - "<review date=\"2016-09-15\" rating=\"80\" karma=\"-1\" id=\"17\">\n" + "<review date=\"2016-09-15\" rating=\"80\" id=\"17\">\n" "<priority>5</priority>\n" "<summary>Hello world</summary>\n" "<description><p>Mighty Fine</p></description>\n" @@ -1212,7 +1212,6 @@ as_test_review_func (void) as_node_unref (root); /* verify */ - g_assert_cmpint (as_review_get_karma (review), ==, -1); g_assert_cmpint (as_review_get_priority (review), ==, 5); g_assert (as_review_get_date (review) != NULL); g_assert_cmpstr (as_review_get_id (review), ==, "17"); |