Subject: Re: More psiconv woes
From: Frodo Looijaard (frodol@dds.nl)
Date: Sun Jan 21 2001 - 13:19:34 CST
Hubert Figuiere wrote:
>
> >
> > > Well, it depends on what Frodo wants. The options are:
> > >
> >> 1) use UT_strdup in our version of psiconv, and thus have them forked
> >> (if only slightly), assuming he doesn't want to include our util code.
> >>
> >> 2) use something other than strdup in the main tree of psiconv.
> >>
> >
> >If Frodo really cared about portability he would write his own
> >"strdup()" function and use it instead.
>
> He can probably borrow the implementation we have in AbiWord :-)
> But this scares me as this would duplicate code. Perhaps finding a
> way to combine both, ie use its own strdup if none is available, but
> use UT_strdup if we compile psiconv within AbiWord :-))
I *do* care about portability. If I didn't, I wouldn't use autoconf at
all. And I *do* know about strdup: it is tested using the autoconf
AC_REPLACE_FUNCS(strdup) function. And, what's more, if autoconf does
not detect it, it is linked in using the version in the compat
subdirectory.
So I see the following options:
* Perhaps autoconf does not detect your lack of strdup correctly? If so,
I would like to see its output on your machine.
* Perhaps I forgot to include compat.h in a few source files? If so,
you might get warnings that strdup is undeclared, but most C compilers
won't really care until linktime, and then it should be picked up
through libcompat.la.
* Perhaps you just saw strdup used and did not test whether I took
care of it properly?
Please explain what your problem is, so I can fix it properly.
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 21 2001 - 13:19:53 CST