changeset
[gaim-migrate @ 15587]
Sat, 11 Feb 2006 07:23:29 +0000
- author
- Richard Laager <rlaager@pidgin.im>
- date
- Sat, 11 Feb 2006 07:23:29 +0000
- changeset 13224
- 485977c61b47
- parent 13223
- a6e949c9d73e
- child 13225
- b82e04c17537
[gaim-migrate @ 15587]
While tracking down a formatting bug that occurred when switching protocols in a contact-aware convo, SimGuy noticed that imhtml_clear_formatting didn't clear imhtml->edit.fontface. We both noticed that those values should be g_free()d before clearing them. I noticed that formatting wasn't clearing properly when I was testing a plain old AIM conversation and just hit Ctrl-R.
| src/gtkimhtml.c | file | annotate | diff | comparison | revisions |
--- a/src/gtkimhtml.c Sat Feb 11 05:43:58 2006 +0000 +++ b/src/gtkimhtml.c Sat Feb 11 07:23:29 2006 +0000 @@ -3738,20 +3738,29 @@ gtk_text_buffer_remove_tag_by_name(imhtml->text_buffer, "ITALICS", &start, &end); gtk_text_buffer_remove_tag_by_name(imhtml->text_buffer, "UNDERLINE", &start, &end); gtk_text_buffer_remove_tag_by_name(imhtml->text_buffer, "STRIKE", &start, &end); - remove_font_size(imhtml, &start, &end, TRUE); - remove_font_face(imhtml, &start, &end, TRUE); - remove_font_forecolor(imhtml, &start, &end, TRUE); - remove_font_backcolor(imhtml, &start, &end, TRUE); - remove_font_background(imhtml, &start, &end, TRUE); - remove_font_link(imhtml, &start, &end, TRUE); + remove_font_size(imhtml, &start, &end, FALSE); + remove_font_face(imhtml, &start, &end, FALSE); + remove_font_forecolor(imhtml, &start, &end, FALSE); + remove_font_backcolor(imhtml, &start, &end, FALSE); + remove_font_background(imhtml, &start, &end, FALSE); + remove_font_link(imhtml, &start, &end, FALSE); imhtml->edit.bold = 0; imhtml->edit.italic = 0; imhtml->edit.underline = 0; imhtml->edit.strike = 0; imhtml->edit.fontsize = 0; + + g_free(imhtml->edit.fontface); + imhtml->edit.fontface = NULL; + + g_free(imhtml->edit.forecolor); imhtml->edit.forecolor = NULL; + + g_free(imhtml->edit.backcolor); imhtml->edit.backcolor = NULL; + + g_free(imhtml->edit.background); imhtml->edit.background = NULL; }
