Subject: Re: PATCH: build psiconv like other libs
From: Frodo Looijaard (frodol@dds.nl)
Date: Sun Jan 28 2001 - 11:15:57 CST
Tom Briggs wrote:
>
> Frodo Looijaard wrote:
> > I just have one question: where do you link in the `compat' library?
>
> At this point it's not being linked in because as far as I'm aware it
> isn't needed. Then again, the only platforms I've compiled it on are Win32
> and Linux, so I haven't exactly tested the majority.
> Haven't we had a dozen conversations about non-standard functions before?
> Is it really preferable to have to do a big dance around some compat library
> for the sake of a one (or more, I dunno) functions that I'm sure could be
> replaced with XP code?
There are two reasons why I don't want to replace it with XP-code:
* I personally hate to have to remember which functions I should
prefix with whatever to make them XP-functions, and which ones
I can leave alone. That is why a configure-like solution is so
much better: it links in these function under their own name,
so you can still use strdup instead of having to remember
to use UT_strdup.
* More importantly, I really, really want to leave the code in
the psiconv/psiconv directory exactly the same as in the
lib/psiconv directory of the complete psiconv package. If we
start to make changes between them now, keeping them both
synchronised is going to be hell.
The second reason is also why I am not very happy with your snprintf-
patch to error.c; I will try to change it to something better.
Thanks,
Frodo
-- Frodo Looijaard <frodol@dds.nl> PGP key and more: http://huizen.dds.nl/~frodol Defenestration n. (formal or joc.): The act of removing Windows from your computer in disgust, usually followed by the installation of Linux or some other Unix-like operating system.
This archive was generated by hypermail 2b25 : Sun Jan 28 2001 - 11:16:49 CST