Changeset 67b959c


Ignore:
Timestamp:
Jun 25, 2011, 10:01:22 AM (13 years ago)
Author:
Jason Gross <jgross@mit.edu>
Children:
caac19d
Parents:
b2bfe1f
git-author:
Jason Gross <jgross@mit.edu> (05/23/11 19:06:48)
git-committer:
Jason Gross <jgross@mit.edu> (06/25/11 10:01:22)
Message:
Added a delete-and-expunge command.

This command is analogous to the delete command, except that it also
expunges only the {current,given} message.  It also takes an optional
--quiet parameter, because I plan to use it internally and don't want
spew on success.  Perhaps there's a better way to accomplish this...

I couldn't figure out how the existing delete command is supposed to
work internally.  The logic behind the delete-and-expunge command is as
follows:
We always update both the message list and the display.  So model the
command that will do much of the work,
owl_function_delete_and_expunge_message, off of owl_function_expunge.
owl_function_delete_and_expunge_message takes one argument, which is the
position of the item in the message list.  It deletes it from the
message list with a new function
owl_messagelist_delete_and_expunge_element.  This calls delete on the
underlying owl_list, so the errors (if any) on an index that is too big
might be unhelpful.

owl_function_delete_and_expunge_cur and
owl_function_delete_and_expunge_by_id both get the index of the relevant
message (and a function owl_messagelist_get_index_by_id is added to
messagelist.c to help the latter).  They owl_function_error if no such
message exists, and conditionally owl_function_makemsg on success.
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • commands.c

    re6d7e4e r67b959c  
    693693              "current view.\n"),
    694694  OWLCMD_ALIAS("del", "delete"),
     695
     696  OWLCMD_ARGS("delete-and-expunge", owl_command_delete_and_expunge, OWL_CTX_INTERACTIVE,
     697              "delete a message",
     698              "delete-and-expunge [-id msgid] [-q | --quiet]",
     699              "If no message id is specified the current message is deleted.\n"
     700              "Otherwise the message with the given message id is deleted.\n"
     701              "If --quiet is specified, then there is no message displayed on\n"
     702              "success.\n"),
     703  OWLCMD_ALIAS("delx", "delete-and-expunge"),
    695704
    696705  OWLCMD_ARGS("undelete", owl_command_undelete, OWL_CTX_INTERACTIVE,
     
    23312340}
    23322341
     2342char *owl_command_delete_and_expunge(int argc, const char *const *argv, const char *buff)
     2343{
     2344  bool exclaim_success = true;
     2345
     2346  if (argc > 1 && (!strcmp(argv[1], "-q") || !strcmp(argv[1], "--quiet"))) {
     2347    exclaim_success = false;
     2348    argc--;
     2349    argv++;
     2350  } else if (!strcmp(argv[argc - 1], "-q") || !strcmp(argv[argc - 1], "--quiet")) {
     2351    exclaim_success = false;
     2352    argc--;
     2353  }
     2354
     2355  if (argc == 1) {
     2356    owl_function_delete_and_expunge_cur(exclaim_success);
     2357    return NULL;
     2358  }
     2359
     2360  if (argc == 3 && (!strcmp(argv[1], "-id") || !strcmp(argv[1], "--id"))) {
     2361    owl_function_delete_and_expunge_by_id(atoi(argv[2]), exclaim_success);
     2362    return NULL;
     2363  }
     2364
     2365  owl_function_makemsg("Unknown arguments to delete-and-expunge command");
     2366  return NULL;
     2367}
     2368
    23332369char *owl_command_undelete(int argc, const char *const *argv, const char *buff)
    23342370{
  • functions.c

    rb2bfe1f r67b959c  
    662662}
    663663
     664void owl_function_delete_and_expunge_message(int n)
     665{
     666  owl_messagelist *ml = owl_global_get_msglist(&g);
     667  owl_view *v = owl_global_get_current_view(&g);
     668  int lastmsgid = owl_function_get_curmsg_id(v);
     669
     670  /* delete and expunge the message */
     671  owl_messagelist_delete_and_expunge_element(ml, n);
     672
     673  owl_function_redisplay_to_nearest(lastmsgid, v);
     674}
     675
     676void owl_function_delete_and_expunge_cur(bool exclaim_success)
     677{
     678  int curmsg;
     679  const owl_view *v = owl_global_get_current_view(&g);
     680
     681  /* bail if there's no current message */
     682  if (owl_view_get_size(v) < 1) {
     683    owl_function_error("No current message to delete");
     684    return;
     685  }
     686
     687  /* delete the current message */
     688  curmsg = owl_global_get_curmsg(&g);
     689  owl_function_delete_and_expunge_message(curmsg);
     690  if (exclaim_success)
     691    owl_function_makemsg("Message deleted and expunged");
     692}
     693
    664694/* if move_after is 1, moves after the delete */
    665695void owl_function_deletecur(int move_after)
     
    16531683}
    16541684
     1685void owl_function_delete_and_expunge_by_id(int id, bool exclaim_success)
     1686{
     1687  const owl_messagelist *ml = owl_global_get_msglist(&g);
     1688  int msg = owl_messagelist_get_index_by_id(ml, id);
     1689  if (msg < 0) {
     1690    owl_function_error("No message with id %d: unable to delete", id);
     1691  } else {
     1692    owl_function_delete_and_expunge_message(msg);
     1693    if (exclaim_success)
     1694      owl_function_makemsg("Message deleted and expunged");
     1695  }
     1696}
     1697
    16551698/* note: this applies to global message list, not to view.
    16561699 * If flag is 1, deletes.  If flag is 0, undeletes. */
  • messagelist.c

    rfc8a87a r67b959c  
    2525}
    2626
    27 owl_message *owl_messagelist_get_by_id(const owl_messagelist *ml, int target_id)
     27int owl_messagelist_get_index_by_id(const owl_messagelist *ml, int target_id)
    2828{
    29   /* return the message with id == 'id'.  If it doesn't exist return NULL. */
     29  /* return the message index with id == 'id'.  If it doesn't exist return -1. */
    3030  int first, last, mid, msg_id;
    3131  owl_message *m;
     
    3838    msg_id = owl_message_get_id(m);
    3939    if (msg_id == target_id) {
    40       return(m);
     40      return mid;
    4141    } else if (msg_id < target_id) {
    4242      first = mid + 1;
     
    4545    }
    4646  }
    47   return(NULL);
     47  return -1;
     48}
     49
     50owl_message *owl_messagelist_get_by_id(const owl_messagelist *ml, int target_id)
     51{
     52  /* return the message with id == 'id'.  If it doesn't exist return NULL. */
     53  int n = owl_messagelist_get_index_by_id(ml, target_id);
     54  if (n < 0) return NULL;
     55  return ml->list->pdata[n];
    4856}
    4957
     
    6674  owl_message_unmark_delete(ml->list->pdata[n]);
    6775  return(0);
     76}
     77
     78void owl_messagelist_delete_and_expunge_element(owl_messagelist *ml, int n)
     79{
     80  owl_message_delete(g_ptr_array_remove_index(ml->list, n));
    6881}
    6982
Note: See TracChangeset for help on using the changeset viewer.