<div dir="ltr">Hi,<div><br></div><div>I was looking at your diff I don't understand the -Wno-missing-field-initializers vs -Wno-extra. If the compiler doesn't support the former why add the -Wno-extra instead of just not adding -Wno-missing... ? Why are you disabling -Wextra in that case?</div>
<div><br></div><div>The linker check seems wrong to me. The config.log output for this part is:</div><div><br></div><div><div>configure:16930: checking whether C compiler accepts -fstack-protector</div><div>configure:16949: gcc -std=gnu99 -c -g -O2 -pthread -Wall -Werror -Wno-error=unused-result  -fstack-protector  conftest.c >&5</div>
<div>configure:16949: $? = 0</div><div>configure:16957: result: yes</div><div>configure:16960: checking whether the linker accepts -fstack-protector</div><div>configure:16979: gcc -std=gnu99 -o conftest -g -O2 -pthread -Wall -Werror -Wno-error=unused-result    -fstack-protector conftest.c   >&5</div>
<div>configure:16979: $? = 0</div><div>configure:16988: result: yes</div></div><div><br></div><div>So both checks are doing the same. Can you share the config.log?</div><div><br></div><div>f.-</div><div><br></div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Thu, Jan 9, 2014 at 6:07 PM, Nils Goroll <span dir="ltr"><<a href="mailto:slink@schokola.de" target="_blank">slink@schokola.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
d9492da540430103b9dcd6844eb59b<u></u>508c061d66 effectively breaks builds with gcc 3.4.<br>
</blockquote>
<br>
... only when developer warnings are enabled.<br>
<br>
Patch attached.<br>
<br>
This also fixes linker issues if the compiler supports -fstack-protector but not the linker and a "the address of ‘buf’ will never be NULL" on Solaris / gcc 4.3.3<span class="HOEnZb"><font color="#888888"><br>
<br>
Nils<br>
</font></span><br>_______________________________________________<br>
varnish-dev mailing list<br>
<a href="mailto:varnish-dev@varnish-cache.org">varnish-dev@varnish-cache.org</a><br>
<a href="https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev" target="_blank">https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev</a><br></blockquote></div><br></div>