Re: [Patch] 2.5.70-bk9 kick FAR out of the zlib

Richard B. Johnson (root@chaos.analogic.com)
Thu, 5 Jun 2003 16:17:52 -0400 (EDT)


On Thu, 5 Jun 2003, [iso-8859-1] Jörn Engel wrote:

> A while back:
>
> On Fri, 30 May 2003 14:38:07 -0700, Linus Torvalds wrote:
> > On Fri, 30 May 2003, Jörn Engel wrote:
> > >
> > > How about an all or nothing approach? If you really want to get rid
> > > of K&R, change indentation as well, rip out some of the rather
> > > tasteless macros (ZEXPORT, ZEXPORTVA, ZEXTERN, FAR, ...) and so on.
> >
> > I'd love to, but I suspect we lack the motivation to do so, and there
> > aren't any obvious upsides. Yes, the code is ugly, but it's also fairly
> > stable so people seldom need to look at it.
>
> Today was a lazy day and that is often motivation enough. The patch
> below removes FAR, the typedefs using FAR (Bytef and friends) and the
> function prototypes for zalloc and zfree that should have gone earlier
> already.
>
> Hope you like it.
>
> Jörn

[SNIPPED patch]

But you just removed the portability hooks. The current code worked
in DOS, on Windows, etc., as will as Linux. This means that if some-
body, as unlikely as it may seem, develops a better/quicker
version using M$ Visual C/C++, you can't get a patch. In particular,
FAR is your friend. A simple #define makes it disappear when you
are not using a segmented architecture, but allows the use of
large arrays when you are.

These kinds of things don't make the code 'pure'. It just prevents
future enhancements. Look in the 'C' header files and see all the
macros that disappear under the right conditions. Would you
justify getting rid of __P in those headers? If not, please don't
eliminate FAR.

Cheers,
Dick Johnson
Penguin : Linux version 2.4.20 on an i686 machine (797.90 BogoMips).
Why is the government concerned about the lunatic fringe? Think about it.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/