On Mon, 2010-12-20 at 08:56 -0500, Adam Jackson wrote:

> The ones I pushed hadn't been nacked, I didn't think?  Regardless,
> yeah,
> revert.
> 

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.
http://lists.x.org/archives/xorg-devel/2010-December/017299.html

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 "one liner in the makefile" appears to be a short term
gain.

Thanks

Gaetan

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to