[PATCH] [RFC] socket activation code for xserver
Łukasz Stelmach
l.stelmach at samsung.com
Thu Nov 28 00:54:33 PST 2013
It was <2013-11-27 śro 11:37>, when Hans de Goede wrote:
> Hi Łukasz,
>
> So I was taking a closer look at the libxtrans patch today,
> and I noticed that it does not build. It adds a check for
> trans->flags&TRANS_RECEIVED inside the main loop in
> MakeAllCOTSServerListeners. But TRANS_RECEIVED does not get
> defined anywhere.
It definitely is
http://article.gmane.org/gmane.comp.freedesktop.xorg.devel/37694
http://thread.gmane.org/gmane.comp.freedesktop.xorg.devel/36092/focus=37694
>
> I've added a define for it to Xtransint.h, which fixes the
> build issue. But other then fixing the BuildIssue this
> makes little sense, since the flag is never being set.
It is, in the patch above.
>
> So an alternative approach to fixing the build issue, would
> to just remove the addition of the check from your patch.
>
> Which has left me wondering why did you add this check in
> the first place? What was it supposed to do, and if we
> drop the check, do we need some other code to achieve
> the same result?
>
> While looking into this, I've polished the patch a but
> up, addressing some of the review comments from its
> earlier posting, and improving error reporting. I've
> attached my cleaned-up version.
Could you please make sure you are working with the v4 version of the
patches as available here:
http://thread.gmane.org/gmane.comp.freedesktop.xorg.devel/36092/focus=37693
Kind regards,
--
Łukasz Stelmach
Samsung R&D Institute Poland
Samsung Electronics
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 489 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20131128/da648907/attachment-0001.pgp>
More information about the xorg-devel
mailing list