[PATCH 0/5] Be careful about lstat()-vs-readlink() · Linus Torvalds <torvalds@linux-foundation.org> · 2016-06-15
[PATCH 1/5] Add generic 'strbuf_readlink()' helper function · Linus Torvalds <torvalds@linux-foundation.org> · 2016-06-15
[PATCH 2/5] Make 'ce_compare_link()' use the new 'strbuf_readlink()' · Linus Torvalds <torvalds@linux-foundation.org> · 2016-06-15
[PATCH 3/5] Make 'index_path()' use 'strbuf_readlink()' · Linus Torvalds <torvalds@linux-foundation.org> · 2016-06-15
[PATCH 4/5] Make 'diff_populate_filespec()' use the new 'strbuf_readlink()' · Linus Torvalds <torvalds@linux-foundation.org> · 2016-06-15
[PATCH 5/5] Make 'prepare_temp_file()' ignore st_size for symlinks · Linus Torvalds <torvalds@linux-foundation.org> · 2016-06-15
[PATCH 7/5] builtin-blame.c: use strbuf_readlink() · Junio C Hamano <hidden> · 2016-06-15
[PATCH 6/5] make_absolute_path(): check bounds when seeing an overlong symlink · Junio C Hamano <hidden> · 2016-06-15
[PATCH 8/5] combine-diff.c: use strbuf_readlink() · Junio C Hamano <hidden> · 2016-06-15
Re: [PATCH 8/5] combine-diff.c: use strbuf_readlink() · Linus Torvalds <torvalds@linux-foundation.org> · 2016-06-15
Re: [PATCH 4/5] Make 'diff_populate_filespec()' use the new 'strbuf_readlink()' · Junio C Hamano <hidden> · 2016-06-15
Re: [PATCH 4/5] Make 'diff_populate_filespec()' use the new 'strbuf_readlink()' · Mark Burton <hidden> · 2016-06-15
Re: [PATCH 4/5] Make 'diff_populate_filespec()' use the new 'strbuf_readlink()' · Linus Torvalds <torvalds@linux-foundation.org> · 2016-06-15
Re: [PATCH 4/5] Make 'diff_populate_filespec()' use the new 'strbuf_readlink()' · René Scharfe <hidden> · 2016-06-15
Re: [PATCH 4/5] Make 'diff_populate_filespec()' use the new 'strbuf_readlink()' · Linus Torvalds <torvalds@linux-foundation.org> · 2016-06-15
Re: [PATCH 4/5] Make 'diff_populate_filespec()' use the new 'strbuf_readlink()' · Olivier Galibert <hidden> · 2016-06-15
Re: [PATCH 4/5] Make 'diff_populate_filespec()' use the new 'strbuf_readlink()' · René Scharfe <hidden> · 2016-06-15
Re: [PATCH 4/5] Make 'diff_populate_filespec()' use the new 'strbuf_readlink()' · René Scharfe <hidden> · 2016-06-15
[PATCH] diff.c: fix pointer type warning · René Scharfe <hidden> · 2016-06-15
Re: [PATCH 3/5] Make 'index_path()' use 'strbuf_readlink()' · Junio C Hamano <hidden> · 2016-06-15
Re: [PATCH 1/5] Add generic 'strbuf_readlink()' helper function · Jay Soffian <hidden> · 2016-06-15
Re: [PATCH 1/5] Add generic 'strbuf_readlink()' helper function · Linus Torvalds <torvalds@linux-foundation.org> · 2016-06-15
[PATCH] strbuf_readlink semantics update. · Pierre Habouzit <hidden> · 2016-08-13
Re: [PATCH] strbuf_readlink semantics update. · Pierre Habouzit <hidden> · 2016-06-15
Re: [PATCH] strbuf_readlink semantics update. · Linus Torvalds <torvalds@linux-foundation.org> · 2016-06-15
Re: [PATCH] strbuf_readlink semantics update. · Pierre Habouzit <hidden> · 2016-06-15
Re: [PATCH] strbuf_readlink semantics update. · René Scharfe <hidden> · 2016-06-15

Re: [PATCH 4/5] Make 'diff_populate_filespec()' use the new 'strbuf_readlink()'

From: Mark Burton <hidden>
Date: 2016-06-15 22:45:47

Howdy folks,

When I compile this latest version of diff.c on a i686 dual-core Pentium box
I see:

diff.c: In function ‘diff_populate_filespec’:
diff.c:1781: warning: passing argument 2 of ‘strbuf_detach’ from incompatible pointer type

The same code compiles without warning on a x86_64 AMD box. Both
machines are running stock Ubuntu 8.04.

Does it need a cast on some architectures?

Cheers,

Mark

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help