[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