Changeset 0864a52


Ignore:
Timestamp:
2015-05-28T05:26:30Z (5 years ago)
Author:
dequis <dx@…>
Branches:
master
Children:
b38f655
Parents:
faeb521
git-author:
dequis <dx@…> (08-05-15 04:16:37)
git-committer:
dequis <dx@…> (28-05-15 05:26:30)
Message:

imcb_notify_email: change parameters to take a format string

Saves some messing with g_strdup_printf for the callers, and
flags/sent_at weren't used anyway.

Also check if the mail_notifications setting is enabled

Location:
protocols
Files:
6 edited

Legend:

Unmodified
Added
Removed
  • protocols/bee.h

    rfaeb521 r0864a52  
    156156G_MODULE_EXPORT void imcb_buddy_msg(struct im_connection *ic, const char *handle, const char *msg, guint32 flags,
    157157                                    time_t sent_at);
    158 G_MODULE_EXPORT void imcb_notify_email(struct im_connection *ic, const char *handle, char *msg, guint32 flags,
    159                                        time_t sent_at);
     158G_MODULE_EXPORT void imcb_notify_email(struct im_connection *ic, char *format, ...) G_GNUC_PRINTF(2, 3);
    160159
    161160/* bee_chat.c */
  • protocols/bee_user.c

    rfaeb521 r0864a52  
    271271}
    272272
    273 void imcb_notify_email(struct im_connection *ic, const char *handle, char *msg, uint32_t flags, time_t sent_at)
    274 {
     273void imcb_notify_email(struct im_connection *ic, char *format, ...)
     274{
     275        const char *handle;
     276        va_list params;
     277        char *msg;
     278
     279        if (!set_getbool(&ic->acc->set, "mail_notifications")) {
     280                return;
     281        }
     282
     283        va_start(params, format);
     284        msg = g_strdup_vprintf(format, params);
     285        va_end(params);
     286
     287        /* up to the protocol to set_add this if they want to use this */
     288        handle = set_getstr(&ic->acc->set, "notify_handle");
     289
    275290        if (handle != NULL) {
    276                 imcb_buddy_msg(ic, handle, msg, flags, sent_at);
     291                imcb_buddy_msg(ic, handle, msg, 0, 0);
    277292        } else {
    278293                imcb_log(ic, "%s", msg);
    279294        }
     295
     296        g_free(msg);
    280297}
    281298
  • protocols/jabber/iq.c

    rfaeb521 r0864a52  
    810810                char *subject = "<no subject>";
    811811                char *sender = "<no sender>";
    812                 char *msg = NULL;
    813812                guint64 t_time;
    814813
     
    828827                }
    829828
    830                 msg = g_strdup_printf("New mail from %s: %s", sender, subject);
    831 
    832                 imcb_notify_email(ic, set_getstr(&ic->acc->set, "notify_handle"), msg, 0, 0);
     829                imcb_notify_email(ic, "New mail from %s: %s", sender, subject);
    833830
    834831                c = c->next;
    835                 g_free(msg);
    836832        }
    837833
  • protocols/msn/ns.c

    rfaeb521 r0864a52  
    394394
    395395                                        if (inbox && folders) {
    396                                                 char *msg = g_strdup_printf(
     396                                                imcb_notify_email(ic,
    397397                                                        "INBOX contains %s new messages, plus %s messages in other folders.", inbox,
    398398                                                        folders);
    399                                                 imcb_notify_email(ic, set_getstr(&ic->acc->set, "notify_handle"), msg, 0, 0);
    400                                                 g_free(msg);
    401399                                        }
    402400
     
    410408
    411409                                        if (from && fromname) {
    412                                                 char *msg = g_strdup_printf("Received an e-mail message from %s <%s>.",
    413                                                                             fromname, from);
    414                                                 imcb_notify_email(ic, set_getstr(&ic->acc->set, "notify_handle"), msg, 0, 0);
    415                                                 g_free(msg);
     410                                                imcb_notify_email(ic, "Received an e-mail message from %s <%s>.", fromname, from);
    416411                                        }
    417412
  • protocols/purple/purple.c

    rfaeb521 r0864a52  
    12611261{
    12621262        struct im_connection *ic = purple_ic_by_gc(gc);
    1263         char *msg = g_strdup_printf("Received e-mail from %s for %s: %s <%s>", from, to, subject, url);
    1264 
    1265         imcb_notify_email(ic, set_getstr(&ic->acc->set, "notify_handle"), msg, 0, 0);
    1266         g_free(msg);
     1263
     1264        imcb_notify_email(ic, "Received e-mail from %s for %s: %s <%s>", from, to, subject, url);
    12671265
    12681266        return NULL;
  • protocols/yahoo/yahoo.c

    rfaeb521 r0864a52  
    959959{
    960960        struct im_connection *ic = byahoo_get_ic_by_id(id);
    961         char *msg;
    962 
    963         if (set_getbool(&ic->acc->set, "mail_notifications")) {
    964                 if (from && subj) {
    965                         msg = g_strdup_printf("Received e-mail message from %s with subject `%s'", from, subj);
    966                 } else if (cnt > 0) {
    967                         msg = g_strdup_printf("Received %d new e-mails", cnt);
    968                 }
    969 
    970                 imcb_notify_email(ic, set_getstr(&ic->acc->set, "notify_handle"), msg, 0, 0);
    971                 g_free(msg);
     961
     962        if (from && subj) {
     963                imcb_notify_email(ic, "Received e-mail message from %s with subject `%s'", from, subj);
     964        } else if (cnt > 0) {
     965                imcb_notify_email(ic, "Received %d new e-mails", cnt);
    972966        }
    973967}
Note: See TracChangeset for help on using the changeset viewer.