diff options
| author | Federico Mena Quintero <federico@helixcode.com> | 2000-12-09 05:09:04 +0800 |
|---|---|---|
| committer | Federico Mena Quintero <federico@src.gnome.org> | 2000-12-09 05:09:04 +0800 |
| commit | 4e47d1142a431f2c64437a7e47fc892a43c17d16 (patch) | |
| tree | 0b83423b2a97d03a35aee348bce1374c7c01dcce /calendar/gui/event-editor.c | |
| parent | 3df2257e016793b56ab81107ef61beecf2ced330 (diff) | |
| download | gsoc2013-evolution-4e47d1142a431f2c64437a7e47fc892a43c17d16.tar.gz gsoc2013-evolution-4e47d1142a431f2c64437a7e47fc892a43c17d16.tar.zst gsoc2013-evolution-4e47d1142a431f2c64437a7e47fc892a43c17d16.zip | |
Free the strings we get from the editables.
2000-12-08 Federico Mena Quintero <federico@helixcode.com>
* gui/event-editor.c (dialog_to_comp_object): Free the strings we
get from the editables.
* gui/dialogs/task-editor.c (dialog_to_comp_object): Likewise.
This sucks; this code should be shared between the two dialogs.
svn path=/trunk/; revision=6868
Diffstat (limited to 'calendar/gui/event-editor.c')
| -rw-r--r-- | calendar/gui/event-editor.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/calendar/gui/event-editor.c b/calendar/gui/event-editor.c index 0d98d75d24..8939e9bb64 100644 --- a/calendar/gui/event-editor.c +++ b/calendar/gui/event-editor.c @@ -2150,7 +2150,7 @@ dialog_to_comp_object (EventEditor *ee, CalComponent *comp) /* Summary */ str = e_dialog_editable_get (priv->general_summary); - if (strlen (str) == 0) + if (!str || strlen (str) == 0) cal_component_set_summary (comp, NULL); else { CalComponentText text; @@ -2161,10 +2161,13 @@ dialog_to_comp_object (EventEditor *ee, CalComponent *comp) cal_component_set_summary (comp, &text); } + if (str) + g_free (str); + /* Description */ str = e_dialog_editable_get (priv->description); - if (strlen (str) == 0) + if (!str || strlen (str) == 0) cal_component_set_description_list (comp, NULL); else { GSList l; @@ -2177,7 +2180,9 @@ dialog_to_comp_object (EventEditor *ee, CalComponent *comp) cal_component_set_description_list (comp, &l); } - /* FIXME: Do we need to free str?? */ + + if (!str) + g_free (str); /* Dates */ |