[Xcb] About next release of xcb/util

[Julien Danjou]

Barton C Massey <bart at cs.pdx.edu> writes:

> I'd buy Ian's earlier argument that there's much less likely
> to be a bug in the second version in the case of multiple
> fields; the compiler remembers to reorder them for you.

I let you that.

> Ian points out that maybe putting it somewhere weird is good
> to encourage people to not use sync when they only need
> flush.  I think there are cases where sync is needed, but
> the fact that I can't name any offhand that I know for sure
> is an argument on Ian's side. :-)

Me neither FWIW.

>> Clearly not.
>
> Why?

Because the gain is null, so the overhead useless.

> Maybe the wm-side icccm and ewmh bits should be split from
> the client-side bits and made into their own library?

Why? Don't see the point.

> Or we could just split out the whole thing and make most
> apps link against it, but that seems wrong somehow.
>
> Anyway, it will hardly contain "almost everything".  By my
> count, we'd be splitting out a minimum of five new
> libraries: event, property, reply, image and renderutil.
> That seems to me to be a good first cut.  We can always do
> more later if we're still unhappy.

I'm sorry, but I'm getting a bit lost and not sure we're talking about
the same things.
xcb-util repository is already building 10 shared library (.so files).

We need to sum up our decision because I'm not sure it's clear what we
are advising Arnaud to do.

What we go is:
* Repository: xcb/util, building:
  - xcb-atom.so
  - xcb-aux.so
  - xcb-event.so
  - xcb-ewmh.so
  - xcb-icccm.so
  - xcb-image.so
  - xcb-keysyms.so
  - xcb-property.so
  - xcb-renderutil.so
  - xcb-reply.so

What I propose:
* Repository: xcb/util, building:
  - xcb-util.so (merging:
                 - xcb-atom.so
                 - xcb-aux.so
                 - xcb-icccm.so
                 - xcb-ewmh.so
                 - xcb-keysyms.so
                 - Some part of the following, but most of the code
                   should be removed because it's craps:
                   - xcb-event.so
                   - xcb-reply.so
                   - xcb-property.so
* Repository: xcb/image, building:
  - xcb-image.so
* Repository: xcb/renderutil, building:
  - xcb-renderutil.so

Based on the fact that most of the .so composing the future xcb-util are
too small and too essential to be distributed on their own.
Please amend my proposal and insult me if you disagree. :)

-- 
Julien Danjou
// ᐰ <julien at danjou.info>   http://julien.danjou.info
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/xcb/attachments/20100323/ef8a302e/attachment.pgp>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/xcb/attachments/20100323/ef8a302e/attachment-0001.pgp>