Changeset: f1417d82bbef for MonetDB URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=f1417d82bbef Modified Files: gdk/gdk_atoms.c Branch: Oct2014 Log Message:
Simplify code. diffs (42 lines): diff --git a/gdk/gdk_atoms.c b/gdk/gdk_atoms.c --- a/gdk/gdk_atoms.c +++ b/gdk/gdk_atoms.c @@ -328,10 +328,10 @@ ATOMformat(int t, const void *p, char ** int sz = 0; return (*tostr) (buf, &sz, p); } - *buf = GDKmalloc(4); + *buf = GDKstrdup("nil"); if (*buf == NULL) return -1; - return snprintf(*buf, 4, "nil"); + return 3; /* strlen(*buf) */ } ptr @@ -360,16 +360,15 @@ ATOMdup(int t, const void *p) * as 'dst' and/or a *len==0 is valid; the conversion function will * then alloc some region for you. */ -#define atommem(TYPE, size) \ - do { \ - if (!*dst) { \ - *dst = (TYPE *) GDKmalloc(*len = (size)); \ - } else if (*len < (int) (size)) { \ - GDKfree(*dst); \ - *dst = (TYPE *) GDKmalloc(*len = (size)); \ - } \ - if (!*dst) \ - return -1; \ +#define atommem(TYPE, size) \ + do { \ + if (*dst == NULL || *len < (int) (size)) { \ + GDKfree(*dst); \ + *len = (size); \ + *dst = (TYPE *) GDKmalloc(*len); \ + if (*dst == NULL) \ + return -1; \ + } \ } while (0) #define atomtostr(TYPE, FMT, FMTCAST) \ _______________________________________________ checkin-list mailing list checkin-list@monetdb.org https://www.monetdb.org/mailman/listinfo/checkin-list