<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
  <META NAME="GENERATOR" CONTENT="GtkHTML/3.26.0">
</HEAD>
<BODY>
On Mon, 2010-11-01 at 16:34 -0700, Jeremy Huddleston wrote:<BR>
<BLOCKQUOTE TYPE=CITE>
    <TT><FONT COLOR="#1a1a1a">So... basically what I said originally... ? ;)</FONT></TT><BR>
    <BR>
</BLOCKQUOTE>
End result is the same. I failed to recognize this&nbsp; level of indirection:<BR>
<BR>
<BLOCKQUOTE>
<PRE>
<TT><FONT COLOR="#1a1a1a">XORG_CWARNFLAGS would be updated to call these two and set </FONT></TT>
<TT><FONT COLOR="#1a1a1a">CWARNFLAGS=&quot;$(CFLAGS_WARNINGS) $(CFLAGS_NO_STRICT_ALIASING)&quot;</FONT></TT>
</PRE>
</BLOCKQUOTE>
<BR>
This has nice advantages, and will help with the conversion so as long as<BR>
XORG_CWARNFLAGS is set back to it's original value eventually. The reason is that somewhere<BR>
in the future, either <TT><FONT COLOR="#1a1a1a">CFLAGS_WARNINGS</FONT></TT> or <TT><FONT COLOR="#1a1a1a">CFLAGS_NO_STRICT_ALIASING</FONT></TT> could get changed.<BR>
This would unfortunately change the value of XORG_CWARNFLAGS still used in previous versions<BR>
of modules or in modules outside the X.Org &quot;in the wild&quot;.<BR>
<BR>
By not touching XORG_CWARNFLAGS at all, we eliminate any risk, not to mention possible confusion.<BR>
<BR>
Once this point is settled, a patch can be submitted for review, along with a plan for converting modules.<BR>
<BR>
Thanks!<BR>
<BR>
Gaetan<BR>
<BR>
<BR>
</BODY>
</HTML>