Skip to content
This repository has been archived by the owner on Jan 16, 2023. It is now read-only.

Commit

Permalink
Strictly adhere to ANSI-C standard
Browse files Browse the repository at this point in the history
  • Loading branch information
Bert Münnich committed Oct 13, 2011
1 parent a09b20c commit 4383a65
Show file tree
Hide file tree
Showing 8 changed files with 26 additions and 35 deletions.
4 changes: 2 additions & 2 deletions Makefile
@@ -1,7 +1,7 @@
VERSION = git-20111012
VERSION = git-20111013

CC = gcc
CFLAGS = -Wall -pedantic -O2
CFLAGS = -ansi -Wall -pedantic -O2
LDFLAGS =
LIBS = -lX11 -lImlib2

Expand Down
2 changes: 1 addition & 1 deletion config.c
Expand Up @@ -9,7 +9,7 @@
#define PUT_MACRO(m) \
printf(" -D%s=%s", #m, QUOTE(m))

inline int puts_if(const char *s, int c) {
int puts_if(const char *s, int c) {
return c ? printf(" %s", s) : 0;
}

Expand Down
5 changes: 2 additions & 3 deletions image.c
Expand Up @@ -43,7 +43,6 @@ enum { MIN_GIF_DELAY = 50 };
float zoom_min;
float zoom_max;

static inline
bool zoomdiff(float z1, float z2) {
const float mindelta = 0.001;
return (z1 - z2 > mindelta) || (z1 - z2 < mindelta);
Expand Down Expand Up @@ -311,11 +310,11 @@ bool img_load(img_t *img, const fileinfo_t *file) {
(void) fmt;

#if EXIF_SUPPORT
if (streq(fmt, "jpeg"))
if (STREQ(fmt, "jpeg"))
exif_auto_orientate(file);
#endif
#if GIF_SUPPORT
if (streq(fmt, "gif"))
if (STREQ(fmt, "gif"))
img_load_gif(img, file);
#endif

Expand Down
6 changes: 3 additions & 3 deletions main.c
Expand Up @@ -145,7 +145,7 @@ void set_timeout(timeout_f handler, int time, bool overwrite) {
if (timeouts[i].handler == handler) {
if (!timeouts[i].active || overwrite) {
gettimeofday(&timeouts[i].when, 0);
tv_add_msec(&timeouts[i].when, time);
TV_ADD_MSEC(&timeouts[i].when, time);
timeouts[i].active = true;
}
return;
Expand All @@ -171,7 +171,7 @@ bool check_timeouts(struct timeval *t) {
gettimeofday(&now, 0);
while (i < ARRLEN(timeouts)) {
if (timeouts[i].active) {
tdiff = tv_diff(&timeouts[i].when, &now);
tdiff = TV_DIFF(&timeouts[i].when, &now);
if (tdiff <= 0) {
timeouts[i].active = false;
if (timeouts[i].handler != NULL)
Expand All @@ -184,7 +184,7 @@ bool check_timeouts(struct timeval *t) {
i++;
}
if (tmin > 0 && t != NULL)
tv_set_msec(t, tmin);
TV_SET_MSEC(t, tmin);
return tmin > 0;
}

Expand Down
2 changes: 1 addition & 1 deletion options.c
Expand Up @@ -140,5 +140,5 @@ void parse_options(int argc, char **argv) {
_options.filenames = argv + optind;
_options.filecnt = argc - optind;
_options.from_stdin = _options.filecnt == 1 &&
streq(_options.filenames[0], "-");
STREQ(_options.filenames[0], "-");
}
2 changes: 1 addition & 1 deletion thumbs.c
Expand Up @@ -258,7 +258,7 @@ bool tns_load(tns_t *tns, int n, const fileinfo_t *file,
(void) fmt;

#if EXIF_SUPPORT
if (!cache_hit && streq(fmt, "jpeg"))
if (!cache_hit && STREQ(fmt, "jpeg"))
exif_auto_orientate(file);
#endif

Expand Down
2 changes: 1 addition & 1 deletion util.c
Expand Up @@ -272,7 +272,7 @@ char* r_readdir(r_dir_t *rdir) {

while (true) {
if (rdir->dir != NULL && (dentry = readdir(rdir->dir)) != NULL) {
if (streq(dentry->d_name, ".") || streq(dentry->d_name, ".."))
if (STREQ(dentry->d_name, ".") || STREQ(dentry->d_name, ".."))
continue;

len = strlen(rdir->name) + strlen(dentry->d_name) + 2;
Expand Down
38 changes: 15 additions & 23 deletions util.h
Expand Up @@ -36,6 +36,21 @@

#define ARRLEN(a) (sizeof(a) / sizeof((a)[0]))

#define STREQ(s1,s2) (strcmp((s1), (s2)) == 0)

#define TV_DIFF(t1,t2) (((t1)->tv_sec - (t2)->tv_sec ) * 1000 + \
((t1)->tv_usec - (t2)->tv_usec) / 1000)

#define TV_SET_MSEC(tv,t) { \
(tv)->tv_sec = (t) / 1000; \
(tv)->tv_usec = (t) % 1000 * 1000; \
}

#define TV_ADD_MSEC(tv,t) { \
(tv)->tv_sec = (t) / 1000; \
(tv)->tv_usec = (t) % 1000 * 1000; \
}

typedef struct {
DIR *dir;
char *name;
Expand All @@ -46,29 +61,6 @@ typedef struct {
int stlen;
} r_dir_t;

static inline
bool streq(const char *a, const char *b) {
return strcmp(a, b) == 0;
}

static inline
long tv_diff(const struct timeval *t1, const struct timeval *t2) {
return (t1->tv_sec - t2->tv_sec) * 1000 +
(t1->tv_usec - t2->tv_usec) / 1000;
}

static inline
void tv_set_msec(struct timeval *t, int msec) {
t->tv_sec = msec / 1000;
t->tv_usec = msec % 1000 * 1000;
}

static inline
void tv_add_msec(struct timeval *t, int msec) {
t->tv_sec += msec / 1000;
t->tv_usec += msec % 1000 * 1000;
}

void* s_malloc(size_t);
void* s_realloc(void*, size_t);
char* s_strdup(char*);
Expand Down

0 comments on commit 4383a65

Please sign in to comment.