Mercurial > pidgin
comparison src/gtkrequest.c @ 12600:e856f985a0b9
[gaim-migrate @ 14934]
Enable the extra warnings regardless of --enable-debug.
Enable FORTIFY_SOURCE regardless of --enable-debug, adding a --disable-fortify flag to configure.
Enable (well, stop disabling) the missing initializer warnings.
This leads to warnings with: GValue v = {0,}; that must be worked around.
Basically, instead of:
GValue v = {0,};
...
g_value_init(&v, G_TYPE_FOO); /* or other use of the GValue */
We'd need to do:
GValue v;
...
v.g_type = 0;
g_value_init(&v, G_TYPE_FOO); /* or other use of the GValue */
Fix several cases of missing initializers. I don't think any of these are bugs, but having this warning seems like a good idea. It might prevent us from making a mistake in the future.
While I was fixing missing initializers, I optimized substitute_simple_word in plugins/spellchk.c, in the same way as I did substitute_word before. Yes, I'm bad for committing these together.
Added a --enable-fatal-asserts flag to configure. As the name implies, this makes g_return_... guards fatal. This is a useful flag to run on a debug copy of Gaim. It will make it very clear if your changes have triggered one of these guards. It's also useful in detecting g_return_... abuse, which helps prevent crashes if Gaim is compiled with G_DISABLE_ASSERT defined.
committer: Tailor Script <tailor@pidgin.im>
author | Richard Laager <rlaager@wiktel.com> |
---|---|
date | Wed, 21 Dec 2005 18:36:19 +0000 |
parents | 255e6912607b |
children | 8765dee8f517 |
comparison
equal
deleted
inserted
replaced
12599:e94c33909aa6 | 12600:e856f985a0b9 |
---|---|
761 #ifdef NEW_STYLE_COMPLETION | 761 #ifdef NEW_STYLE_COMPLETION |
762 static gboolean screenname_completion_match_func(GtkEntryCompletion *completion, | 762 static gboolean screenname_completion_match_func(GtkEntryCompletion *completion, |
763 const gchar *key, GtkTreeIter *iter, gpointer user_data) | 763 const gchar *key, GtkTreeIter *iter, gpointer user_data) |
764 { | 764 { |
765 GtkTreeModel *model; | 765 GtkTreeModel *model; |
766 GValue val1 = { 0, }; | 766 GValue val1; |
767 GValue val2 = { 0, }; | 767 GValue val2; |
768 const char *tmp; | 768 const char *tmp; |
769 | 769 |
770 model = gtk_entry_completion_get_model (completion); | 770 model = gtk_entry_completion_get_model (completion); |
771 | 771 |
772 val1.g_type = 0; | |
772 gtk_tree_model_get_value(model, iter, 2, &val1); | 773 gtk_tree_model_get_value(model, iter, 2, &val1); |
773 tmp = g_value_get_string(&val1); | 774 tmp = g_value_get_string(&val1); |
774 if (tmp != NULL && gaim_str_has_prefix(tmp, key)) | 775 if (tmp != NULL && gaim_str_has_prefix(tmp, key)) |
775 { | 776 { |
776 g_value_unset(&val1); | 777 g_value_unset(&val1); |
777 return TRUE; | 778 return TRUE; |
778 } | 779 } |
779 g_value_unset(&val1); | 780 g_value_unset(&val1); |
780 | 781 |
782 val2.g_type = 0; | |
781 gtk_tree_model_get_value(model, iter, 3, &val2); | 783 gtk_tree_model_get_value(model, iter, 3, &val2); |
782 tmp = g_value_get_string(&val2); | 784 tmp = g_value_get_string(&val2); |
783 if (tmp != NULL && gaim_str_has_prefix(tmp, key)) | 785 if (tmp != NULL && gaim_str_has_prefix(tmp, key)) |
784 { | 786 { |
785 g_value_unset(&val2); | 787 g_value_unset(&val2); |
791 } | 793 } |
792 | 794 |
793 static gboolean screenname_completion_match_selected_cb(GtkEntryCompletion *completion, | 795 static gboolean screenname_completion_match_selected_cb(GtkEntryCompletion *completion, |
794 GtkTreeModel *model, GtkTreeIter *iter, gpointer *user_data) | 796 GtkTreeModel *model, GtkTreeIter *iter, gpointer *user_data) |
795 { | 797 { |
796 GValue val = { 0, }; | 798 GValue val; |
797 GaimRequestField *screen_field = user_data[1]; | 799 GaimRequestField *screen_field = user_data[1]; |
798 GList *fields = screen_field->group->fields; | 800 GList *fields = screen_field->group->fields; |
799 GaimAccount *account; | 801 GaimAccount *account; |
800 | 802 |
803 val.g_type = 0; | |
801 gtk_tree_model_get_value(model, iter, 1, &val); | 804 gtk_tree_model_get_value(model, iter, 1, &val); |
802 gtk_entry_set_text(GTK_ENTRY(user_data[0]), g_value_get_string(&val)); | 805 gtk_entry_set_text(GTK_ENTRY(user_data[0]), g_value_get_string(&val)); |
803 g_value_unset(&val); | 806 g_value_unset(&val); |
804 | 807 |
805 gtk_tree_model_get_value(model, iter, 4, &val); | 808 gtk_tree_model_get_value(model, iter, 4, &val); |