From: Chun-wei Fan Date: Tue, 20 Mar 2012 05:19:11 +0000 (+0800) Subject: glib/gqsort.c: Fix C99ism/GCCism X-Git-Tag: 2.32.0~4 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b08b3012235b089cc3c8d015c47f703da01a4d81;p=platform%2Fupstream%2Fglib.git glib/gqsort.c: Fix C99ism/GCCism -There were a number of variables that were declared in the middle of the block, so move these declarations to the start of the block -There was a use of mempcpy, but it is a GCC extension, so use memcpy since we didn't care about the return value of the call to mempcpy. https://bugzilla.gnome.org/show_bug.cgi?id=672095 --- diff --git a/glib/gqsort.c b/glib/gqsort.c index cd32181..fc699ea 100644 --- a/glib/gqsort.c +++ b/glib/gqsort.c @@ -178,7 +178,7 @@ msort_with_tmp (const struct msort_param *p, void *b, size_t n) } else { - mempcpy (tmp, b2, s); + memcpy (tmp, b2, s); tmp += s; b2 += s; --n2; @@ -226,6 +226,8 @@ msort_r (void *b, size_t n, size_t s, GCompareDataFunc cmp, void *arg) void **tp = (void **) (p.t + n * sizeof (void *)); void **t = tp; void *tmp_storage = (void *) (tp + n); + char *kp; + size_t i; while ((void *) t < tmp_storage) { @@ -238,8 +240,6 @@ msort_r (void *b, size_t n, size_t s, GCompareDataFunc cmp, void *arg) /* tp[0] .. tp[n - 1] is now sorted, copy around entries of the original array. Knuth vol. 3 (2nd ed.) exercise 5.2-10. */ - char *kp; - size_t i; for (i = 0, ip = (char *) b; i < n; i++, ip += s) if ((kp = tp[i]) != ip) {