Changeset 0071c88


Ignore:
Timestamp:
Aug 1, 2011, 11:28:43 PM (13 years ago)
Author:
David Benjamin <davidben@mit.edu>
Children:
2e42098
Parents:
9078f69
git-author:
David Benjamin <davidben@mit.edu> (08/01/11 02:03:32)
git-committer:
David Benjamin <davidben@mit.edu> (08/01/11 23:28:43)
Message:
Restore correct semantics of message 'time' attribute

This rewrites part of 4ebbfbc5360fa004637dd101f5a0c833cdccd60a. We can't
replace every instance of ctime with a user-formatted time, as the time
attribute is not user-formatted. It is (unfortunately) the API for perl
to override the timestamp and owl_perlconfig_hashref2message expects a
particular format for strptime. We should not then flip the values
around once they reach C. (Especially not a locale-dependent one.)

Rename *_to_timestr functions to owl_util_format_* so it is clear the
function should only be used for user-formatted times.
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • functions.c

    rb9517cf r0071c88  
    12291229  now = time(NULL);
    12301230
    1231   tmpbuff = owl_util_time_to_timestr(localtime(&now));
     1231  tmpbuff = owl_util_format_time(localtime(&now));
    12321232  fprintf(file, "[%d -  %s - %lds]: ",
    12331233          (int) getpid(), tmpbuff, now - owl_global_get_starttime(&g));
     
    13791379  owl_fmtext fm, attrfm;
    13801380  const owl_view *v;
     1381  char *time;
    13811382#ifdef HAVE_LIBZEPHYR
    13821383  const ZNotice_t *n;
     
    14091410  }
    14101411
    1411   owl_fmtext_appendf_normal(&fm, "  Time      : %s\n", owl_message_get_timestr(m));
     1412  time = owl_message_format_time(m);
     1413  owl_fmtext_appendf_normal(&fm, "  Time      : %s\n", time);
     1414  g_free(time);
    14121415
    14131416  if (!owl_message_is_type_admin(m)) {
     
    17821785  owl_fmtext_append_normal(&fm, "\n");
    17831786
    1784   tmpbuff = owl_util_time_to_timestr(localtime(&start));
     1787  tmpbuff = owl_util_format_time(localtime(&start));
    17851788  owl_fmtext_appendf_normal(&fm, "  Startup Time: %s\n", tmpbuff);
    17861789  g_free(tmpbuff);
     
    30783081      idle=owl_buddy_get_idle_time(b);
    30793082      if (idle!=0) {
    3080         timestr=owl_util_minutes_to_timestr(idle);
     3083        timestr=owl_util_format_minutes(idle);
    30813084      } else {
    30823085        timestr=g_strdup("");
     
    34073410
    34083411  now = time(NULL);
    3409   date = owl_util_time_to_timestr(localtime(&now));
     3412  date = owl_util_format_time(localtime(&now));
    34103413
    34113414  buff = g_strdup_printf("%s %s", date, string);
  • message.c

    rb9517cf r0071c88  
    4343  /* save the time */
    4444  m->time = time(NULL);
    45   m->timestr = owl_util_time_to_timestr(localtime(&m->time));
     45  m->timestr = g_strdup(ctime(&m->time));
     46  m->timestr[strlen(m->timestr)-1] = '\0';
    4647
    4748  m->fmtext = NULL;
     
    344345  if (m->timestr) return(m->timestr);
    345346  return("");
     347}
     348
     349CALLER_OWN char *owl_message_format_time(const owl_message *m)
     350{
     351  return owl_util_format_time(localtime(&m->time));
    346352}
    347353
     
    793799  if (m->timestr) g_free(m->timestr);
    794800  m->time = n->z_time.tv_sec;
    795   m->timestr = owl_util_time_to_timestr(localtime(&m->time));
     801  m->timestr = g_strdup(ctime(&m->time));
     802  m->timestr[strlen(m->timestr)-1] = '\0';
    796803
    797804  /* set other info */
  • util.c

    rf271129 r0071c88  
    268268
    269269/* caller must free the return */
    270 CALLER_OWN char *owl_util_minutes_to_timestr(int in)
     270CALLER_OWN char *owl_util_format_minutes(int in)
    271271{
    272272  int days, hours;
     
    289289}
    290290
    291 CALLER_OWN char *owl_util_time_to_timestr(const struct tm *time)
     291CALLER_OWN char *owl_util_format_time(const struct tm *time)
    292292{
    293293  /* 32 chosen for first attempt because timestr will end up being
Note: See TracChangeset for help on using the changeset viewer.