Re: print-esp.c patch

["Gisle Vanem" <giva () bgnett no>]

"Guy Harris" <gharris () sonic net> said:
> Is it GPLed?  The comment just speaks of a "COPYING.DJ" file.
>
> I'll use the BSD "strsep.c" instead, just to be sure there isn't a
> license clash.

It's GPL'ed, but please add the BSD version instead. (remove the 
<sys/cdefs.h> at top).

> Is the "t" necessary for text on any DOS/Windows platforms?  Or is it
> always the case that "r" and "w" give you text mode?

I'm not sure what the standard says. Testing on MSVC/MingW/djgpp
and Watcom shows that "r" and "w" implies text-mode. But IMHO better 
to append "t" to be sure.

> If by "the configure process" you mean the process performed by the
> configure script, that's done with AC_REPLACE_FUNCS; I've added it to
> the list of string functions in the AC_REPLACE_FUNCS in configure.in.
>
> I've checked your changes in, along with an update to FILES to include
> missing/strsep.c.
 
Okay, thanks.
 

-
This is the TCPDUMP workers list. It is archived at
http://www.tcpdump.org/lists/workers/index.html
To unsubscribe use mailto:tcpdump-workers-request () tcpdump org?body=unsubscribe