<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<br>
<div class="moz-cite-prefix">On 14/01/2014 22:05, Richard Smith
wrote:<br>
</div>
<blockquote cite="mid:7406757645903024425@gmail297201516"
type="cite">
<div>Well:</div>
<div>1) the context is an "incompatible with C++98" diagnostic</div>
<div>2) we really don't need to worry about theoretical future C17
or OpenMP constructs now; we can easily change our diagnostics
if we ever support those things</div>
</blockquote>
<br>
Sorry, I didn't mention -- the diag clarification change was split
out from the C11 attributes extension I'd promised to help Renato
with for his 'vectorize' attribute.<br>
<br>
Having split this part out as an unrelated change may have
inadvertently lost some of the context as to why it was useful :-)<br>
<br>
<blockquote cite="mid:7406757645903024425@gmail297201516"
type="cite">
<div>3) we have the '[[' in the snippet</div>
<div>4) a %select is *better* than a single fixed string if it
makes the diagnostic clearer</div>
<div><br>
</div>
<div>I'm fine with "'[[...]]' attributes" or similar, although it
seems a little redundant given (3). "C++ attributes" or "C++11
attributes" work for me.</div>
</blockquote>
<br>
"C++11 attributes" works in this scenario if you want to go ahead
and change it, but let's keep options open to go back on that,
because it looks a little silly alongside the C11 version:<br>
<br>
<code>def ext_cxx11_attribute_in_c : ExtWarn<</code><code><br>
</code><code> "generalized attributes are a non-standard C
extension">, InGroup<CXXAttributes>;</code><br>
<br>
<br>
I'll throw up the full patch in a bit.<br>
<br>
Alp.<br>
<br>
<br>
<pre class="moz-signature" cols="72">--
<a class="moz-txt-link-freetext" href="http://www.nuanti.com">http://www.nuanti.com</a>
the browser experts
</pre>
</body>
</html>