[PATCH] dix: make RegionInit legal C++

Keith Packard keithp at keithp.com
Thu Jan 8 14:45:03 PST 2015


Adam Jackson <ajax at redhat.com> writes:

> On Mon, 2015-01-05 at 13:54 -0800, Alan Coopersmith wrote:
>> Same code as http://patchwork.freedesktop.org/patch/39718/ but different
>> comments. My r-b from 
>> http://lists.x.org/archives/xorg-devel/2015-January/045096.html
>> applies to either set of comments.
>
> Tsk, that's what I get for not catching up fully on email.  I concede
> priority.

Your comments explain the actual problem though, so you win on that
front. I wasn't going to take the patch otherwise. Event still, making
life for c++ people easier by polluting our code with spurious casts
doesn't really get me excited.

-- 
-keith
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20150108/08890523/attachment-0001.sig>


More information about the xorg-devel mailing list