<!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-12-20 at 08:56 -0500, Adam Jackson wrote:<BR>
<BLOCKQUOTE TYPE=CITE>
    <TT><FONT COLOR="#1a1a1a">The ones I pushed hadn't been nacked, I didn't think?&nbsp; Regardless, yeah,</FONT></TT><BR>
    <TT><FONT COLOR="#1a1a1a">revert.</FONT></TT><BR>
    <BR>
</BLOCKQUOTE>
Correct. A number of similar patches were posted within a 6 day period with various reviewers responding to various patches. That leaves one patch for which I posted a revert and would appreciate a review. <A HREF="http://lists.x.org/archives/xorg-devel/2010-December/017299.html">http://lists.x.org/archives/xorg-devel/2010-December/017299.html</A><BR>
<BR>
My main concern is introducing 2 ways of exporting/hiding symbols that are far apart (Makefile and source code). It's just error prone. Other libraries use _X_EXPORT and _X_HIDDEN and we should stick to this method, even if a &quot;one liner in the makefile&quot; appears to be a short term gain.<BR>
<BR>
Thanks<BR>
<BR>
Gaetan
</BODY>
</HTML>