small bugfixes and a major one (freeze when deleting an appointment's note, thanks Jan for reporting it)

This commit is contained in:
Frederic Culot 2008-12-15 20:02:00 +00:00
parent 8220580918
commit b1d3178bba
12 changed files with 76 additions and 56 deletions

View File

@ -1,3 +1,18 @@
2008-12-15 Frederic Culot <frederic@culot.org>
* src/todo.c (todo_delete)
* src/day.c (day_edit_item): fixed a bug that prevented from
editing an item
* src/keys.c: generic-escape renamed into generic-cancel
* src/utils.c (getstring): make the escape key work again to
cancel editing
* src/apoint.c (apoint_delete_bynum)
* src/day.c (day_erase_item): fixed a possible freeze when
deleting an appointment's note
2008-12-14 Frederic Culot <frederic@culot.org> 2008-12-14 Frederic Culot <frederic@culot.org>
* === Released 2.4_beta === * === Released 2.4_beta ===
@ -9,11 +24,11 @@
* src/recur.c (foreach_date_dump): exception dates are now * src/recur.c (foreach_date_dump): exception dates are now
properly taken into account properly taken into account
* src/apoint.c: * src/apoint.c
* src/calendar.c: * src/calendar.c
* src/custom.c: * src/custom.c
* src/event.c: * src/event.c
* src/io.c: * src/io.c
* src/recur.c: make use of error handling macros * src/recur.c: make use of error handling macros
* src/utils.c (status_bar): do not show 'credits' key binding * src/utils.c (status_bar): do not show 'credits' key binding

View File

@ -8,7 +8,7 @@ msgid ""
msgstr "" msgstr ""
"Project-Id-Version: calcurse 2.4_beta\n" "Project-Id-Version: calcurse 2.4_beta\n"
"Report-Msgid-Bugs-To: frederic@culot.org\n" "Report-Msgid-Bugs-To: frederic@culot.org\n"
"POT-Creation-Date: 2008-12-14 16:48+0100\n" "POT-Creation-Date: 2008-12-15 20:53+0100\n"
"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n" "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n" "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
"Language-Team: LANGUAGE <LL@li.org>\n" "Language-Team: LANGUAGE <LL@li.org>\n"
@ -45,19 +45,19 @@ msgstr ""
msgid "Do you really want to delete this item ?" msgid "Do you really want to delete this item ?"
msgstr "" msgstr ""
#: src/apoint.c:276 #: src/apoint.c:274
msgid "no such type" msgid "no such type"
msgstr "" msgstr ""
#: src/apoint.c:385 src/recur.c:247 #: src/apoint.c:383 src/recur.c:247
msgid "date error in appointment" msgid "date error in appointment"
msgstr "" msgstr ""
#: src/apoint.c:406 src/recur.c:856 src/recur.c:880 src/recur.c:916 #: src/apoint.c:404 src/recur.c:856 src/recur.c:880 src/recur.c:916
msgid "item not found" msgid "item not found"
msgstr "" msgstr ""
#: src/apoint.c:450 src/apoint.c:610 #: src/apoint.c:451 src/apoint.c:611
msgid "no such appointment" msgid "no such appointment"
msgstr "" msgstr ""

View File

@ -1,4 +1,4 @@
# $calcurse: de.po,v 1.21 2008/12/14 15:54:50 culot Exp $ # $calcurse: de.po,v 1.22 2008/12/15 20:02:00 culot Exp $
# #
# German translations for calcurse package. # German translations for calcurse package.
# Copyright (C) 2006-2007 Michael Schulz <bloodshower@web.de> # Copyright (C) 2006-2007 Michael Schulz <bloodshower@web.de>
@ -9,7 +9,7 @@ msgid ""
msgstr "" msgstr ""
"Project-Id-Version: calcurse 1.9\n" "Project-Id-Version: calcurse 1.9\n"
"Report-Msgid-Bugs-To: frederic@culot.org\n" "Report-Msgid-Bugs-To: frederic@culot.org\n"
"POT-Creation-Date: 2008-12-14 16:48+0100\n" "POT-Creation-Date: 2008-12-15 20:53+0100\n"
"PO-Revision-Date: 2007-09-02 18:06+0200\n" "PO-Revision-Date: 2007-09-02 18:06+0200\n"
"Last-Translator: Michael\n" "Last-Translator: Michael\n"
"Language-Team: Michael Schulz <bloodshower@freenet.de>\n" "Language-Team: Michael Schulz <bloodshower@freenet.de>\n"
@ -52,20 +52,20 @@ msgstr "[EINGABE]-Taste um fortzufahren"
msgid "Do you really want to delete this item ?" msgid "Do you really want to delete this item ?"
msgstr "Möchten Sie wirklich diesen Eintrag löschen ?" msgstr "Möchten Sie wirklich diesen Eintrag löschen ?"
#: src/apoint.c:276 #: src/apoint.c:274
msgid "no such type" msgid "no such type"
msgstr "" msgstr ""
#: src/apoint.c:385 src/recur.c:247 #: src/apoint.c:383 src/recur.c:247
#, fuzzy #, fuzzy
msgid "date error in appointment" msgid "date error in appointment"
msgstr "nächster Termin :\n" msgstr "nächster Termin :\n"
#: src/apoint.c:406 src/recur.c:856 src/recur.c:880 src/recur.c:916 #: src/apoint.c:404 src/recur.c:856 src/recur.c:880 src/recur.c:916
msgid "item not found" msgid "item not found"
msgstr "" msgstr ""
#: src/apoint.c:450 src/apoint.c:610 #: src/apoint.c:451 src/apoint.c:611
#, fuzzy #, fuzzy
msgid "no such appointment" msgid "no such appointment"
msgstr "nächster Termin :\n" msgstr "nächster Termin :\n"

View File

@ -1,4 +1,4 @@
# $calcurse: en.po,v 1.16 2008/12/14 15:54:50 culot Exp $ # $calcurse: en.po,v 1.17 2008/12/15 20:02:00 culot Exp $
# #
# English/GB translation of calcurse. # English/GB translation of calcurse.
# Copyright (C) 2006 Copyright (c) Frederic Culot <frederic@culot.org> # Copyright (C) 2006 Copyright (c) Frederic Culot <frederic@culot.org>
@ -11,7 +11,7 @@ msgid ""
msgstr "" msgstr ""
"Project-Id-Version: calcurse 1.4\n" "Project-Id-Version: calcurse 1.4\n"
"Report-Msgid-Bugs-To: frederic@culot.org\n" "Report-Msgid-Bugs-To: frederic@culot.org\n"
"POT-Creation-Date: 2008-12-14 16:48+0100\n" "POT-Creation-Date: 2008-12-15 20:53+0100\n"
"PO-Revision-Date: 2006-07-03 00:05+0100\n" "PO-Revision-Date: 2006-07-03 00:05+0100\n"
"Last-Translator: Neil Williams <linux@codehelp.co.uk>\n" "Last-Translator: Neil Williams <linux@codehelp.co.uk>\n"
"Language-Team: English/GB <en_GB@li.org>\n" "Language-Team: English/GB <en_GB@li.org>\n"
@ -49,20 +49,20 @@ msgstr "Press [Enter] to continue"
msgid "Do you really want to delete this item ?" msgid "Do you really want to delete this item ?"
msgstr "Do you really want to delete this item ?" msgstr "Do you really want to delete this item ?"
#: src/apoint.c:276 #: src/apoint.c:274
msgid "no such type" msgid "no such type"
msgstr "" msgstr ""
#: src/apoint.c:385 src/recur.c:247 #: src/apoint.c:383 src/recur.c:247
#, fuzzy #, fuzzy
msgid "date error in appointment" msgid "date error in appointment"
msgstr "Appointment :" msgstr "Appointment :"
#: src/apoint.c:406 src/recur.c:856 src/recur.c:880 src/recur.c:916 #: src/apoint.c:404 src/recur.c:856 src/recur.c:880 src/recur.c:916
msgid "item not found" msgid "item not found"
msgstr "" msgstr ""
#: src/apoint.c:450 src/apoint.c:610 #: src/apoint.c:451 src/apoint.c:611
#, fuzzy #, fuzzy
msgid "no such appointment" msgid "no such appointment"
msgstr "Appointment :" msgstr "Appointment :"

View File

@ -1,4 +1,4 @@
# $calcurse: es.po,v 1.17 2008/12/14 15:54:50 culot Exp $ # $calcurse: es.po,v 1.18 2008/12/15 20:02:00 culot Exp $
# #
# Spanish translations for calcurse package. # Spanish translations for calcurse package.
# Copyright (c) Jose Lopez <testinground@gmail.com> 2006 Free Software Foundation, Inc. # Copyright (c) Jose Lopez <testinground@gmail.com> 2006 Free Software Foundation, Inc.
@ -9,7 +9,7 @@ msgid ""
msgstr "" msgstr ""
"Project-Id-Version: Calcurse 1.8\n" "Project-Id-Version: Calcurse 1.8\n"
"Report-Msgid-Bugs-To: frederic@culot.org\n" "Report-Msgid-Bugs-To: frederic@culot.org\n"
"POT-Creation-Date: 2008-12-14 16:48+0100\n" "POT-Creation-Date: 2008-12-15 20:53+0100\n"
"PO-Revision-Date: 2006-09-10 08:06+0200\n" "PO-Revision-Date: 2006-09-10 08:06+0200\n"
"Last-Translator: Jose Lopez <testinground@gmail.com>\n" "Last-Translator: Jose Lopez <testinground@gmail.com>\n"
"Language-Team: Jose Lopez <testinground@gmail.com>\n" "Language-Team: Jose Lopez <testinground@gmail.com>\n"
@ -50,20 +50,20 @@ msgstr "Pulsa [INTRO] para continuar"
msgid "Do you really want to delete this item ?" msgid "Do you really want to delete this item ?"
msgstr "¿Seguro que quieres borrar este elemento?" msgstr "¿Seguro que quieres borrar este elemento?"
#: src/apoint.c:276 #: src/apoint.c:274
msgid "no such type" msgid "no such type"
msgstr "" msgstr ""
#: src/apoint.c:385 src/recur.c:247 #: src/apoint.c:383 src/recur.c:247
#, fuzzy #, fuzzy
msgid "date error in appointment" msgid "date error in appointment"
msgstr "Proxima cita:\n" msgstr "Proxima cita:\n"
#: src/apoint.c:406 src/recur.c:856 src/recur.c:880 src/recur.c:916 #: src/apoint.c:404 src/recur.c:856 src/recur.c:880 src/recur.c:916
msgid "item not found" msgid "item not found"
msgstr "" msgstr ""
#: src/apoint.c:450 src/apoint.c:610 #: src/apoint.c:451 src/apoint.c:611
#, fuzzy #, fuzzy
msgid "no such appointment" msgid "no such appointment"
msgstr "Proxima cita:\n" msgstr "Proxima cita:\n"

View File

@ -1,4 +1,4 @@
# $calcurse: fr.po,v 1.27 2008/12/14 15:54:51 culot Exp $ # $calcurse: fr.po,v 1.28 2008/12/15 20:02:00 culot Exp $
# #
# French translations for calcurse package. # French translations for calcurse package.
# Copyright (C) 2004-2008 Frederic Culot <frederic@culot.org> # Copyright (C) 2004-2008 Frederic Culot <frederic@culot.org>
@ -9,7 +9,7 @@ msgid ""
msgstr "" msgstr ""
"Project-Id-Version: calcurse 2.1\n" "Project-Id-Version: calcurse 2.1\n"
"Report-Msgid-Bugs-To: frederic@culot.org\n" "Report-Msgid-Bugs-To: frederic@culot.org\n"
"POT-Creation-Date: 2008-12-14 16:48+0100\n" "POT-Creation-Date: 2008-12-15 20:53+0100\n"
"PO-Revision-Date: 2008-09-23 19:21+0200\n" "PO-Revision-Date: 2008-09-23 19:21+0200\n"
"Last-Translator: Frederic Culot <frederic@culot.org>\n" "Last-Translator: Frederic Culot <frederic@culot.org>\n"
"Language-Team: French <calcurse-i18n@culot.org>\n" "Language-Team: French <calcurse-i18n@culot.org>\n"
@ -50,21 +50,21 @@ msgstr "Appuyez sur [ENTREE] pour continuer"
msgid "Do you really want to delete this item ?" msgid "Do you really want to delete this item ?"
msgstr "Voulez-vous vraiment effacer cet élement ?" msgstr "Voulez-vous vraiment effacer cet élement ?"
#: src/apoint.c:276 #: src/apoint.c:274
msgid "no such type" msgid "no such type"
msgstr "" msgstr ""
#: src/apoint.c:385 src/recur.c:247 #: src/apoint.c:383 src/recur.c:247
#, fuzzy #, fuzzy
msgid "date error in appointment" msgid "date error in appointment"
msgstr "prochain rendez-vous:\n" msgstr "prochain rendez-vous:\n"
#: src/apoint.c:406 src/recur.c:856 src/recur.c:880 src/recur.c:916 #: src/apoint.c:404 src/recur.c:856 src/recur.c:880 src/recur.c:916
#, fuzzy #, fuzzy
msgid "item not found" msgid "item not found"
msgstr "Fichier note non trouvé\n" msgstr "Fichier note non trouvé\n"
#: src/apoint.c:450 src/apoint.c:610 #: src/apoint.c:451 src/apoint.c:611
#, fuzzy #, fuzzy
msgid "no such appointment" msgid "no such appointment"
msgstr "prochain rendez-vous:\n" msgstr "prochain rendez-vous:\n"

View File

@ -1,4 +1,4 @@
# $calcurse: nl.po,v 1.15 2008/12/14 15:54:51 culot Exp $ # $calcurse: nl.po,v 1.16 2008/12/15 20:02:00 culot Exp $
# #
# Dutch translations for calcurse package. # Dutch translations for calcurse package.
# Copyright (C) 2007-2008 Jeremy Roon <p.roon@chello.nl> # Copyright (C) 2007-2008 Jeremy Roon <p.roon@chello.nl>
@ -9,7 +9,7 @@ msgid ""
msgstr "" msgstr ""
"Project-Id-Version: calcurse 2.1\n" "Project-Id-Version: calcurse 2.1\n"
"Report-Msgid-Bugs-To: frederic@culot.org\n" "Report-Msgid-Bugs-To: frederic@culot.org\n"
"POT-Creation-Date: 2008-12-14 16:48+0100\n" "POT-Creation-Date: 2008-12-15 20:53+0100\n"
"PO-Revision-Date: 2008-10-14 20:54+0200\n" "PO-Revision-Date: 2008-10-14 20:54+0200\n"
"Last-Translator: Jeremy Roon\n" "Last-Translator: Jeremy Roon\n"
"Language-Team: Jeremy Roon <p.roon@chello.nl>\n" "Language-Team: Jeremy Roon <p.roon@chello.nl>\n"
@ -48,21 +48,21 @@ msgstr "Druk op [Enter] om door te gaan)"
msgid "Do you really want to delete this item ?" msgid "Do you really want to delete this item ?"
msgstr "Wilt u werkelijk dit item wissen ?" msgstr "Wilt u werkelijk dit item wissen ?"
#: src/apoint.c:276 #: src/apoint.c:274
msgid "no such type" msgid "no such type"
msgstr "" msgstr ""
#: src/apoint.c:385 src/recur.c:247 #: src/apoint.c:383 src/recur.c:247
#, fuzzy #, fuzzy
msgid "date error in appointment" msgid "date error in appointment"
msgstr "Volgende afspraak :\n" msgstr "Volgende afspraak :\n"
#: src/apoint.c:406 src/recur.c:856 src/recur.c:880 src/recur.c:916 #: src/apoint.c:404 src/recur.c:856 src/recur.c:880 src/recur.c:916
#, fuzzy #, fuzzy
msgid "item not found" msgid "item not found"
msgstr "Noot-bestand niet gevonden\n" msgstr "Noot-bestand niet gevonden\n"
#: src/apoint.c:450 src/apoint.c:610 #: src/apoint.c:451 src/apoint.c:611
#, fuzzy #, fuzzy
msgid "no such appointment" msgid "no such appointment"
msgstr "Volgende afspraak :\n" msgstr "Volgende afspraak :\n"

View File

@ -1,4 +1,4 @@
/* $calcurse: apoint.c,v 1.27 2008/12/14 15:54:51 culot Exp $ */ /* $calcurse: apoint.c,v 1.28 2008/12/15 20:02:00 culot Exp $ */
/* /*
* Calcurse - text-based organizer * Calcurse - text-based organizer
@ -269,9 +269,7 @@ apoint_delete (conf_t *conf, unsigned *nb_events, unsigned *nb_apoints)
to_be_removed = 3; to_be_removed = 3;
} }
else if (deleted_item_type == 0) else if (deleted_item_type == 0)
{ return;
to_be_removed = 0;
}
else else
EXIT (_("no such type")); EXIT (_("no such type"));
/* NOTREACHED */ /* NOTREACHED */
@ -425,7 +423,10 @@ apoint_delete_bynum (long start, unsigned num, erase_flag_e flag)
if (n == num) if (n == num)
{ {
if (flag == ERASE_FORCE_ONLY_NOTE) if (flag == ERASE_FORCE_ONLY_NOTE)
erase_note (&i->note, flag); {
erase_note (&i->note, flag);
pthread_mutex_unlock (&(alist_p->mutex));
}
else else
{ {
if (notify_bar ()) if (notify_bar ())

View File

@ -1,4 +1,4 @@
/* $calcurse: day.c,v 1.41 2008/12/12 20:44:50 culot Exp $ */ /* $calcurse: day.c,v 1.42 2008/12/15 20:02:00 culot Exp $ */
/* /*
* Calcurse - text-based organizer * Calcurse - text-based organizer
@ -728,7 +728,7 @@ day_edit_item (conf_t *conf)
p = day_get_item (item_num); p = day_get_item (item_num);
date = calendar_get_slctd_day_sec (); date = calendar_get_slctd_day_sec ();
ch = 0; ch = -1;
switch (p->type) switch (p->type)
{ {
case RECUR_EVNT: case RECUR_EVNT:
@ -819,9 +819,10 @@ day_erase_item (long date, int item_number, erase_flag_e flag)
"Delete (i)tem or just its (n)ote ?"); "Delete (i)tem or just its (n)ote ?");
char *note_choice = _("[i/n] "); char *note_choice = _("[i/n] ");
char *erase_choice = _("[a/o] "); char *erase_choice = _("[a/o] ");
int ch = 0, ans; int ch, ans;
unsigned delete_whole; unsigned delete_whole;
ch = -1;
p = day_get_item (item_number); p = day_get_item (item_number);
if (flag == ERASE_DONT_FORCE) if (flag == ERASE_DONT_FORCE)
{ {
@ -877,7 +878,10 @@ day_erase_item (long date, int item_number, erase_flag_e flag)
recur_apoint_erase (date, p->appt_pos, delete_whole, flag); recur_apoint_erase (date, p->appt_pos, delete_whole, flag);
} }
} }
return (p->type); if (flag == ERASE_FORCE_ONLY_NOTE)
return 0;
else
return (p->type);
} }
/* Returns a structure containing the selected item. */ /* Returns a structure containing the selected item. */

View File

@ -1,4 +1,4 @@
/* $calcurse: keys.c,v 1.9 2008/12/12 20:44:50 culot Exp $ */ /* $calcurse: keys.c,v 1.10 2008/12/15 20:02:00 culot Exp $ */
/* /*
* Calcurse - text-based organizer * Calcurse - text-based organizer
@ -49,7 +49,7 @@ static struct key_str_s *keys[NBKEYS];
static keys_e actions[MAXKEYVAL]; static keys_e actions[MAXKEYVAL];
static struct keydef_s keydef[NBKEYS] = { static struct keydef_s keydef[NBKEYS] = {
{"generic-escape", "ESC"}, {"generic-cancel", "ESC"},
{"generic-select", "SPC"}, {"generic-select", "SPC"},
{"generic-credits", "@"}, {"generic-credits", "@"},
{"generic-help", "?"}, {"generic-help", "?"},

View File

@ -1,4 +1,4 @@
/* $calcurse: todo.c,v 1.27 2008/12/12 20:44:50 culot Exp $ */ /* $calcurse: todo.c,v 1.28 2008/12/15 20:02:00 culot Exp $ */
/* /*
* Calcurse - text-based organizer * Calcurse - text-based organizer
@ -239,7 +239,7 @@ todo_delete (conf_t *conf)
"Delete (t)odo or just its (n)ote ?"); "Delete (t)odo or just its (n)ote ?");
char *erase_choice = _("[t/n] "); char *erase_choice = _("[t/n] ");
bool go_for_todo_del = false; bool go_for_todo_del = false;
int answer = 0, has_note; int answer, has_note;
if (conf->confirm_delete) if (conf->confirm_delete)
{ {
@ -264,7 +264,7 @@ todo_delete (conf_t *conf)
return; return;
} }
answer = 0; answer = -1;
has_note = (todo_get_item (hilt)->note != NULL) ? 1 : 0; has_note = (todo_get_item (hilt)->note != NULL) ? 1 : 0;
if (has_note == 0) if (has_note == 0)
answer = 't'; answer = 't';

View File

@ -1,4 +1,4 @@
/* $calcurse: utils.c,v 1.59 2008/12/14 15:54:51 culot Exp $ */ /* $calcurse: utils.c,v 1.60 2008/12/15 20:02:00 culot Exp $ */
/* /*
* Calcurse - text-based organizer * Calcurse - text-based organizer
@ -329,7 +329,7 @@ getstring (WINDOW *win, char *str, int l, int x, int y)
newpos++; newpos++;
break; break;
case KEY_GENERIC_CANCEL: /* cancel editing */ case ESCAPE: /* cancel editing */
return (GETSTRING_ESC); return (GETSTRING_ESC);
break; break;