From: Cedric D. <ced...@fr...> - 2005-09-21 18:10:25
|
Jaakko Heinonen wrote: > > case 'n': > > - case 'N': > [...] > > case 'y': > > - case 'Y': > This changes the behavior. :) But it's probably OK. I don't think so. Just above we have switch(tolower(getch())) { The tolower() call should guarantee we won't ever meet uppercase 'N' and 'Y'. :) A few other things: * Some functions (especially change_field() for the field length) rely on a 'limit' parameter when calling ui_readline() which in turn calls abook_readline() with this 'limit' parameter... which is not used at all. * And the attached patch fixes a memory leak. Regards, --=20 C=E9dric |