From 8c5957537c63e2a8b9a937fe99db509b153df6c6 Mon Sep 17 00:00:00 2001 From: Marvin W Date: Wed, 22 Jan 2020 00:09:06 +0100 Subject: Fix some compiler warnings --- main/src/emojichooser.c | 20 ++++++++++---------- .../conversation_summary/chat_state_populator.vala | 4 ++-- .../ui/conversation_summary/content_populator.vala | 8 ++++---- .../conversation_item_skeleton.vala | 2 +- .../date_separator_populator.vala | 2 +- 5 files changed, 18 insertions(+), 18 deletions(-) (limited to 'main') diff --git a/main/src/emojichooser.c b/main/src/emojichooser.c index 35d3bcff..5ffc15c3 100644 --- a/main/src/emojichooser.c +++ b/main/src/emojichooser.c @@ -236,7 +236,7 @@ static gboolean has_variations (GVariant *emoji_data) { GVariant *codes; - int i; + gsize i; gboolean has_variations; has_variations = FALSE; @@ -366,7 +366,7 @@ add_emoji (GtkWidget *box, GVariant *codes; char text[64]; char *p = text; - int i; + gsize i; PangoLayout *layout; PangoRectangle rect; @@ -451,21 +451,21 @@ populate_emoji_chooser (gpointer data) g_variant_get_child (item, 1, "&s", &name); - if (strcmp (name, chooser->body.first) == 0) + if (g_strcmp0 (name, chooser->body.first) == 0) chooser->box = chooser->body.box; - else if (strcmp (name, chooser->nature.first) == 0) + else if (g_strcmp0 (name, chooser->nature.first) == 0) chooser->box = chooser->nature.box; - else if (strcmp (name, chooser->food.first) == 0) + else if (g_strcmp0 (name, chooser->food.first) == 0) chooser->box = chooser->food.box; - else if (strcmp (name, chooser->travel.first) == 0) + else if (g_strcmp0 (name, chooser->travel.first) == 0) chooser->box = chooser->travel.box; - else if (strcmp (name, chooser->activities.first) == 0) + else if (g_strcmp0 (name, chooser->activities.first) == 0) chooser->box = chooser->activities.box; - else if (strcmp (name, chooser->objects.first) == 0) + else if (g_strcmp0 (name, chooser->objects.first) == 0) chooser->box = chooser->objects.box; - else if (strcmp (name, chooser->symbols.first) == 0) + else if (g_strcmp0 (name, chooser->symbols.first) == 0) chooser->box = chooser->symbols.box; - else if (strcmp (name, chooser->flags.first) == 0) + else if (g_strcmp0 (name, chooser->flags.first) == 0) chooser->box = chooser->flags.box; add_emoji (chooser->box, FALSE, item, 0, chooser); diff --git a/main/src/ui/conversation_summary/chat_state_populator.vala b/main/src/ui/conversation_summary/chat_state_populator.vala index 5db2a1f6..291f3537 100644 --- a/main/src/ui/conversation_summary/chat_state_populator.vala +++ b/main/src/ui/conversation_summary/chat_state_populator.vala @@ -89,9 +89,9 @@ private enum StateType { } private class MetaChatStateItem : Plugins.MetaConversationItem { - public override Jid? jid { get; set; } + public override Jid jid { get; set; } public override bool dim { get; set; default=true; } - public override DateTime? sort_time { get; set; default=new DateTime.now_utc().add_years(10); } + public override DateTime sort_time { get; set; default=new DateTime.now_utc().add_years(10); } public override bool can_merge { get; set; default=false; } public override bool requires_avatar { get; set; default=false; } diff --git a/main/src/ui/conversation_summary/content_populator.vala b/main/src/ui/conversation_summary/content_populator.vala index cc014a04..368f2a67 100644 --- a/main/src/ui/conversation_summary/content_populator.vala +++ b/main/src/ui/conversation_summary/content_populator.vala @@ -67,10 +67,10 @@ public class ContentProvider : ContentItemCollection, Object { } public class ContentMetaItem : Plugins.MetaConversationItem { - public override Jid? jid { get; set; } - public override DateTime? sort_time { get; set; } - public override DateTime? display_time { get; set; } - public override Encryption? encryption { get; set; } + public override Jid jid { get; set; } + public override DateTime sort_time { get; set; } + public override DateTime display_time { get; set; } + public override Encryption encryption { get; set; } public ContentItem content_item; private ContentItemWidgetFactory widget_factory; diff --git a/main/src/ui/conversation_summary/conversation_item_skeleton.vala b/main/src/ui/conversation_summary/conversation_item_skeleton.vala index 28551b01..46669863 100644 --- a/main/src/ui/conversation_summary/conversation_item_skeleton.vala +++ b/main/src/ui/conversation_summary/conversation_item_skeleton.vala @@ -121,7 +121,7 @@ public class ItemMetaDataHeader : Box { update_name_label(); name_label.style_updated.connect(update_name_label); - if (item.encryption != null && item.encryption != Encryption.NONE) { + if (item.encryption != Encryption.NONE) { encryption_image.visible = true; encryption_image.set_from_icon_name("dino-changes-prevent-symbolic", ICON_SIZE_HEADER); } diff --git a/main/src/ui/conversation_summary/date_separator_populator.vala b/main/src/ui/conversation_summary/date_separator_populator.vala index a5c1a887..f1ad17ef 100644 --- a/main/src/ui/conversation_summary/date_separator_populator.vala +++ b/main/src/ui/conversation_summary/date_separator_populator.vala @@ -52,7 +52,7 @@ class DateSeparatorPopulator : Plugins.ConversationItemPopulator, Plugins.Conver } public class MetaDateItem : Plugins.MetaConversationItem { - public override DateTime? sort_time { get; set; } + public override DateTime sort_time { get; set; } public override bool can_merge { get; set; default=false; } public override bool requires_avatar { get; set; default=false; } -- cgit v1.2.3-54-g00ecf