Changeset dde1b4d for logging.c


Ignore:
Timestamp:
Jul 15, 2011, 11:40:47 PM (10 years ago)
Author:
David Benjamin <davidben@mit.edu>
Branches:
master, release-1.8, release-1.9
Children:
c6332f5
Parents:
30d0cf7
git-author:
David Benjamin <davidben@mit.edu> (07/12/11 11:58:02)
git-committer:
David Benjamin <davidben@mit.edu> (07/15/11 23:40:47)
Message:
Use g_build_filename instead of g_strdup_printf to build paths

Saves us a duplicate slash in a few places, although most of the time it
doesn't do anything. Eh, may as well use it and be more explicit or
something.

Also change the one existing call to g_build_path to g_build_filename
instead. It's much less of a pain to use.
File:
1 edited

Legend:

Unmodified
Added
Removed
  • logging.c

    r6829afc rdde1b4d  
    224224    while (cc != NULL) {
    225225      temp = short_zuser(cc->data);
    226       filename = g_strdup_printf("%s/%s", logpath, temp);
     226      filename = g_build_filename(logpath, temp, NULL);
    227227      owl_log_append(m, filename);
    228228
     
    248248  }
    249249
    250   filename = g_strdup_printf("%s/%s", logpath, to);
     250  filename = g_build_filename(logpath, to, NULL);
    251251  owl_log_append(m, filename);
    252252  g_free(to);
    253253  g_free(filename);
    254254
    255   filename = g_strdup_printf("%s/all", logpath);
     255  filename = g_build_filename(logpath, "all", NULL);
    256256  owl_log_append(m, filename);
    257257  g_free(logpath);
     
    285285  /* expand ~ in path names */
    286286  logpath = owl_util_makepath(owl_global_get_logpath(&g));
    287   filename = g_strdup_printf("%s/%s", logpath, tobuff);
     287  filename = g_build_filename(logpath, tobuff, NULL);
    288288  msgbuf = g_string_new("");
    289289  g_string_printf(msgbuf, "ERROR (owl): %s\n%s\n", tobuff, text);
     
    294294  g_string_free(msgbuf, TRUE);
    295295
    296   filename = g_strdup_printf("%s/all", logpath);
     296  filename = g_build_filename(logpath, "all", NULL);
    297297  g_free(logpath);
    298298  msgbuf = g_string_new("");
     
    393393  if (personal) {
    394394    logpath = owl_util_makepath(owl_global_get_logpath(&g));
    395     filename = g_strdup_printf("%s/%s", logpath, from);
    396     allfilename = g_strdup_printf("%s/all", logpath);
     395    filename = g_build_filename(logpath, from, NULL);
     396    allfilename = g_build_filename(logpath, "all", NULL);
    397397    owl_log_append(m, allfilename);
    398398    g_free(allfilename);
    399399  } else {
    400400    logpath = owl_util_makepath(owl_global_get_classlogpath(&g));
    401     filename = g_strdup_printf("%s/%s", logpath, from);
     401    filename = g_build_filename(logpath, from, NULL);
    402402  }
    403403
     
    415415      temp = short_zuser(cc->data);
    416416      if (strcasecmp(temp, frombuff) != 0) {
    417         filename = g_strdup_printf("%s/%s", logpath, temp);
     417        filename = g_build_filename(logpath, temp, NULL);
    418418        owl_log_append(m, filename);
    419419        g_free(filename);
Note: See TracChangeset for help on using the changeset viewer.