Re: [evolution-patches] [Calendar] New Appointment - summary validation



On Wed, 2005-12-14 at 13:57 +0530, Johnny Jacob wrote:
> +               comp = comp_editor_get_current_comp (editor);
> +               e_cal_component_get_summary (comp, &text);
> +               g_object_unref (comp); 
This must come inside prompt_to_save_changes. Perhaps the
prompt_to_save_changes does more than what it should do which is
confusing, it prompts for the dialog and also saves/discards the changes
depending on the response. It should be renamed.

thanks, Chenthill.



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]