Bug 14834

Summary: liboldX ansification
Product: xorg Reporter: Paulo César Pereira de Andrade <pcpa>
Component: Lib/otherAssignee: Xorg Project Team <xorg-team>
Status: RESOLVED NOTABUG QA Contact: Xorg Project Team <xorg-team>
Severity: normal    
Priority: medium Keywords: patch
Version: git   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Attachments:
Description Flags
0001-Ansify-liboldX.patch none

Description Paulo César Pereira de Andrade 2008-03-05 09:57:50 UTC
Created attachment 14863 [details] [review]
0001-Ansify-liboldX.patch

Also added prototypes to X10.h.

I think there is a lot of code in different places
reimplementing the liboldX XAssoc features, i.e. a
per display, hash table of X resources.
Comment 1 Alan Coopersmith 2008-03-05 11:16:57 UTC
Any code written to use liboldX is likely not ANSI-compliant itself.   
X version 11 came out before ANSI/ISO C89 did.
Comment 2 Paulo César Pereira de Andrade 2008-03-05 11:33:56 UTC
(In reply to comment #1)
> Any code written to use liboldX is likely not ANSI-compliant itself.   
> X version 11 came out before ANSI/ISO C89 did.

  Most of the "ansification" patches I posted are basically cosmetic
changes, but some problems were also pointed. Cosmetic because it
should generate 100% binary compatible code, as the prototypes
are already ansified, and the patches just update the sources.

  BTW, I know in Mandriva there is nothing linked agains't liboldX.

  liboldX is a survivor, so many thing has been nuked over the time,
and this library is still here. I think it is almost as old as xedit :-)

Comment 3 David Nusinow 2008-03-05 16:53:55 UTC
Is there a good reason to not mark this WONTFIX?
Comment 4 Paulo César Pereira de Andrade 2008-03-05 17:36:09 UTC
(In reply to comment #3)
> Is there a good reason to not mark this WONTFIX?

  I think I never saw a bug that really deserved WONTFIX.

  If someone is recompiling some really old code, that also needs
liboldX, this person should be using an ansi compiler. Otherwise,
probably is also using liboldX from another source, and probably
with another name. And probably also compiling the entire X Server,
etc.

  But this is more of a cosmetic patch. If not droping it, then I
don't think there is a strong reason to also not modify it from K&R
to ansi, given that it doesn't use "promotable" function parameters.
Comment 5 Paulo César Pereira de Andrade 2008-07-23 16:08:16 UTC
  Patch doesn't correct any problems.

  The pseudo hash table (hashed by XID), and the
bezier polygons interfaces are not bad, and should
be significantly fast for sw only optimized
applications. But I feel no existing application
ever links with -loldX, at least this is the case
of Mandriva packages...
Comment 6 Alan Coopersmith 2008-07-23 20:05:08 UTC
liboldX is for X version 10 compatibility - i.e. programs using it would be 5 years older than even Linus's first public release of Linux, and would have managed to survive almost 21 years now without being ported to libX11 - it
seems highly unlikely any Linux distro needs it.   (Even with our extreme
backwards compatibility stance in Solaris, we don't bother shipping liboldX.)

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.