diff options
author | Dan Fandrich <dan@coneharvesters.com> | 2010-02-04 04:04:56 +0100 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2010-02-04 04:04:56 +0100 |
commit | 1821d188ca674b42bf0f384b0c2332ff95701bba (patch) | |
tree | 7666fd803fadc230f3d2bb7df9a908aff0a231b1 | |
parent | f111b67100d17e0dab904160a0ab18430a70b9ba (diff) |
diff: more compiler compat fixes. no code changes
Signed-off-by: Dan Fandrich <dan@coneharvesters.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
-rw-r--r-- | Makefile.flags | 1 | ||||
-rw-r--r-- | editors/diff.c | 17 |
2 files changed, 10 insertions, 8 deletions
diff --git a/Makefile.flags b/Makefile.flags index 032f6c135..60bb888d8 100644 --- a/Makefile.flags +++ b/Makefile.flags @@ -40,6 +40,7 @@ CFLAGS += $(call cc-option,-Werror,) ## Classic *(off_t*)(void*)ptr does not work, ## and I am unwilling to do crazy gcc specific ({ void *ppp = ...; }) ## stuff in macros. This would obfuscate the code too much. +## Maybe try __attribute__((__may_alias__))? #CFLAGS += $(call cc-ifversion, -eq, 0404, -fno-strict-aliasing) endif # gcc 3.x emits bogus "old style proto" warning on find.c:alloc_action() diff --git a/editors/diff.c b/editors/diff.c index a8ffcdd8d..07594e8d8 100644 --- a/editors/diff.c +++ b/editors/diff.c @@ -471,7 +471,7 @@ start: ix[i][j] = nfile[i][j].offset; } - /* lenght of prefix and suffix is calculated */ + /* length of prefix and suffix is calculated */ for (; pref < nlen[0] && pref < nlen[1] && nfile[0][pref + 1].value == nfile[1][pref + 1].value; pref++); @@ -501,7 +501,7 @@ start: free(nfile[1]); class = xmalloc((slen[0] + 1) * sizeof(class[0])); - for (int i = 1; i <= slen[0]; i++) /* Unsorting */ + for (i = 1; i <= slen[0]; i++) /* Unsorting */ class[sfile[0][i].serial] = sfile[0][i].value; free(nfile[0]); #else @@ -565,7 +565,7 @@ static bool diff(FILE* fp[2], char *file[2]) FILE_and_pos_t ft[2]; typedef struct { int a, b; } vec_t[2]; vec_t *vec = NULL; - int i = 1, idx = -1; + int i = 1, j, k, idx = -1; bool anychange = false; int *J; @@ -608,8 +608,8 @@ static bool diff(FILE* fp[2], char *file[2]) break; } - for (int j = 0; j < 2; j++) - for (int k = v[j].a; k < v[j].b; k++) + for (j = 0; j < 2; j++) + for (k = v[j].a; k < v[j].b; k++) nonempty |= (ix[j][k+1] - ix[j][k] != 1); vec = xrealloc_vector(vec, 6, ++idx); @@ -624,6 +624,7 @@ static bool diff(FILE* fp[2], char *file[2]) if (idx < 0 || ((option_mask32 & FLAG(B)) && !nonempty)) goto cont; if (!(option_mask32 & FLAG(q))) { + int lowa; vec_t span, *cvp = vec; if (!anychange) { @@ -633,7 +634,7 @@ static bool diff(FILE* fp[2], char *file[2]) } printf("@@"); - for (int j = 0; j < 2; j++) { + for (j = 0; j < 2; j++) { int a = span[j].a = MAX(1, (*cvp)[j].a - opt_U_context); int b = span[j].b = MIN(nlen[j], vec[idx][j].b + opt_U_context); @@ -647,12 +648,12 @@ static bool diff(FILE* fp[2], char *file[2]) * Output changes in "unified" diff format--the old and new lines * are printed together. */ - for (int lowa = span[0].a; ; lowa = (*cvp++)[0].b + 1) { + for (lowa = span[0].a; ; lowa = (*cvp++)[0].b + 1) { bool end = cvp > &vec[idx]; fetch(&ft[0], ix[0], lowa, end ? span[0].b : (*cvp)[0].a - 1, ' '); if (end) break; - for (int j = 0; j < 2; j++) + for (j = 0; j < 2; j++) fetch(&ft[j], ix[j], (*cvp)[j].a, (*cvp)[j].b, j ? '+' : '-'); } } |