bpl21 and sending/aborting mail

From: Steve Kunitzer (fester@festerhead.com)
Date: 05/19/02


There is an extraneous 'Message Sent!' in modify.c, line 234 that will:
1.  Give a player 2 'Message Sent!' notices when sending mail.
2.  Give a player a 'Message aborted.' and a 'Message sent!' when
aborting a mail message.

Was:
  if (PLR_FLAGGED(d->character, PLR_MAILING)) {
    if (action == STRINGADD_SAVE && *d->str) {
      store_mail(d->mail_to, GET_IDNUM(d->character), *d->str);
      write_to_output(d, "Message sent!\r\n");
    } else
      write_to_output(d, "Mail aborted.\r\n");
      free(*d->str);
      free(d->str);
      //write_to_output(d, "Message sent!\r\n");
  }

Is:
  if (PLR_FLAGGED(d->character, PLR_MAILING)) {
    if (action == STRINGADD_SAVE && *d->str) {
      store_mail(d->mail_to, GET_IDNUM(d->character), *d->str);
      write_to_output(d, "Message sent!\r\n");
    } else
      write_to_output(d, "Mail aborted.\r\n");
      free(*d->str);
      free(d->str);
      //write_to_output(d, "Message sent!\r\n");
  }

Good? Bad? Punt?

Steve Kunitzer

--
   +---------------------------------------------------------------+
   | FAQ: http://qsilver.queensu.ca/~fletchra/Circle/list-faq.html |
   | Archives: http://post.queensu.ca/listserv/wwwarch/circle.html |
   | Newbie List:  http://groups.yahoo.com/group/circle-newbies/   |
   +---------------------------------------------------------------+



This archive was generated by hypermail 2b30 : 06/25/03 PDT