summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenys Vlasenko <vda.linux@googlemail.com>2010-04-29 17:22:45 +0200
committerDenys Vlasenko <vda.linux@googlemail.com>2010-04-29 17:22:45 +0200
commite769b83bc129210a25198c7de73272d8632923fa (patch)
tree18b906ffba975c664e3448a72331a96148b06501
parent87327f43e4ea65d63ceab0985cf814e22940579a (diff)
time.c: fix unreadable code style: assignments within if(), misplaced comments
No logic changes Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r--libc/misc/time/time.c83
1 files changed, 48 insertions, 35 deletions
diff --git a/libc/misc/time/time.c b/libc/misc/time/time.c
index f80104ee1..758a737fd 100644
--- a/libc/misc/time/time.c
+++ b/libc/misc/time/time.c
@@ -1912,38 +1912,36 @@ void _time_tzset(int use_old_rules)
static char oldval[TZ_BUFLEN]; /* BSS-zero'd. */
#endif
+ /* Put this inside the lock to prevent the possibility of two different
+ * timezones being used in a threaded app. */
__UCLIBC_MUTEX_LOCK(_time_tzlock);
e = getenv(TZ); /* TZ env var always takes precedence. */
#if defined(__UCLIBC_HAS_TZ_FILE__) && !defined(__UCLIBC_HAS_TZ_FILE_READ_MANY__)
- /* Put this inside the lock to prevent the possiblity of two different
- * timezones being used in a threaded app. */
-
- if (e != NULL) {
- TZ_file_read = 0; /* Reset if the TZ env var is set. */
- } else if (TZ_file_read) {
+ if (e) {
+ /* Never use TZfile if TZ env var is set. */
+ TZ_file_read = 0;
+ }
+ if (TZ_file_read) {
+ /* We already parsed TZfile before, skip everything. */
goto FAST_DONE;
}
-#endif /* defined(__UCLIBC_HAS_TZ_FILE__) && !defined(__UCLIBC_HAS_TZ_FILE_READ_MANY__) */
+#endif
/* Warning!!! Since uClibc doesn't do lib locking, the following is
* potentially unsafe in a multi-threaded program since it is remotely
* possible that another thread could call setenv() for TZ and overwrite
* the string being parsed. So, don't do that... */
- if ((!e /* TZ env var not set... */
#ifdef __UCLIBC_HAS_TZ_FILE__
- && !(e = read_TZ_file(buf)) /* and no file or invalid file */
-#endif
- ) || !*e) { /* or set to empty string. */
-ILLEGAL: /* TODO: Clean up the following... */
-#ifdef __UCLIBC_HAS_TZ_CACHING__
- *oldval = 0; /* Set oldval to an empty string. */
+ if (!e)
+ e = read_TZ_file(buf);
#endif
- memset(_time_tzinfo, 0, 2*sizeof(rule_struct));
- strcpy(_time_tzinfo[0].tzname, UTC);
- goto DONE;
+ if (!e /* TZ env var not set and no TZfile (or bad TZfile) */
+ || !*e /* or set to empty string. */
+ ) {
+ goto ILLEGAL;
}
if (*e == ':') { /* Ignore leading ':'. */
@@ -1951,8 +1949,9 @@ ILLEGAL: /* TODO: Clean up the following... */
}
#ifdef __UCLIBC_HAS_TZ_CACHING__
- if (strcmp(e, oldval) == 0) { /* Same string as last time... */
- goto FAST_DONE; /* So nothing to do. */
+ if (strcmp(e, oldval) == 0) {
+ /* Same string as last time... nothing to do. */
+ goto FAST_DONE;
}
/* Make a copy of the TZ env string. It won't be nul-terminated if
* it is too long, but it that case it will be illegal and will be reset
@@ -1973,10 +1972,11 @@ LOOP:
s = new_rules[count].tzname;
n = 0;
while (*e
- && isascii(*e) /* SUSv3 requires char in portable char set. */
- && (isalpha(*e)
- || (c && (isalnum(*e) || (*e == '+') || (*e == '-'))))
- ) {
+ && isascii(*e) /* SUSv3 requires char in portable char set. */
+ && (isalpha(*e)
+ || (c && (isalnum(*e) || (*e == '+') || (*e == '-')))
+ )
+ ) {
*s++ = *e++;
if (++n > TZNAME_MAX) {
goto ILLEGAL;
@@ -1985,8 +1985,8 @@ LOOP:
*s = 0;
if ((n < 3) /* Check for minimum length. */
- || (c && (*e++ != c)) /* Match any quoting '<'. */
- ) {
+ || (c && (*e++ != c)) /* Match any quoting '<'. */
+ ) {
goto ILLEGAL;
}
@@ -2001,7 +2001,8 @@ LOOP:
}
++e;
- if (!(e = getoffset(e, &off))) {
+ e = getoffset(e, &off);
+ if (!e) {
goto ILLEGAL;
}
@@ -2038,7 +2039,8 @@ SKIP_OFFSET:
n = 365;
s = (char *) RULE;
- if ((c = *e++) == 'M') {
+ c = *e++;
+ if (c == 'M') {
n = 12;
} else if (c == 'J') {
s += 8;
@@ -2048,26 +2050,31 @@ SKIP_OFFSET:
s += 6;
}
- *(p = &new_rules[count].rule_type) = c;
+ p = &new_rules[count].rule_type;
+ *p = c;
if (c != 'M') {
p -= 2;
}
do {
++s;
- if (!(e = getnumber(e, &f))
- || (((unsigned int)(f - s[1])) > n)
- || (*s && (*e++ != *s))
- ) {
+ e = getnumber(e, &f);
+ if (!e
+ || ((unsigned int)(f - s[1]) > n)
+ || (*s && (*e++ != *s))
+ ) {
goto ILLEGAL;
}
*--p = f;
- } while ((n = *(s += 2)) > 0);
+ s += 2;
+ n = *s;
+ } while (n > 0);
off = 2 * 60 * 60; /* Default to 2:00:00 */
if (*e == '/') {
++e;
- if (!(e = getoffset(e, &off))) {
+ e = getoffset(e, &off);
+ if (!e) {
goto ILLEGAL;
}
}
@@ -2075,7 +2082,13 @@ SKIP_OFFSET:
} while (++count < 2);
if (*e) {
- goto ILLEGAL;
+ILLEGAL:
+#ifdef __UCLIBC_HAS_TZ_CACHING__
+ oldval[0] = 0; /* oldval = "" */
+#endif
+ memset(_time_tzinfo, 0, sizeof(_time_tzinfo));
+ strcpy(_time_tzinfo[0].tzname, UTC);
+ goto DONE;
}
}