<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div><blockquote type="cite" class=""><div class="">On Jan 23, 2015, at 12:20 PM, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com" class="">rafael.espindola@gmail.com</a>> wrote:</div><div class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote">On 23 January 2015 at 14:34, John McCall <span dir="ltr" class=""><<a href="mailto:rjmccall@apple.com" target="_blank" class="">rjmccall@apple.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class=""><span class=""><blockquote type="cite" class=""><div class="">On Jan 23, 2015, at 11:04 AM, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com" target="_blank" class="">rafael.espindola@gmail.com</a>> wrote:</div><div class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word" class=""><div class="">How is it not a viable fight? Is the section attribute coming from a completely different place? Or are you suggesting that it is never viable to tell people that they ought to fix their code, no matter how unnecessarily perverse it is? A section should be an intrinsic part of an definition, saying that you can’t define the same thing in multiple inconsistent ways is not even slightly unreasonable.</div></div></blockquote><div class=""><br class=""></div><div class="">The bug first got reported to us while trying to build glibc. The bug Richard noticed was fixed in gcc because it was breaking the linux kernel. If anyone thinks it is productive to try to get them to change, go for it.</div></div></div></div></div></blockquote><div class=""><br class=""></div></span><div class="">Sorry, do these open-source projects no longer accept patches? Adding section attributes after a definition does not seem defensible to me, and I would guess that the declarations are actually in the same file, just in the wrong order.</div></div></div></blockquote><div class=""><br class=""></div><div class="">Maybe, but I have better things to do than being a message boy between your opinion on gnu extensions and their use of them.</div></div></div></div></div></blockquote><div><br class=""></div>Rafael, you are taking this very personally and apparently trying to pick a fight. I’m sorry, but putting visibility aside, this is not the right fix, and this code should be considered ill-formed. We should not allow declarations following a definition to add section attributes, and we should not allow redeclarations to add asm attributes at all.</div><div><br class=""></div><div>John.</div></body></html>