[Zlib-devel] zlib-1.2.3-cos2

[Cosmin Truta]

Hi,

There is a new zlib-1.2.3-cos2.tar.gz at the following address:

http://www.cs.toronto.edu/~cosmin/compr/zlib/zlib-1.2.3-cos2.tar.gz

I am recommending Mark to consider this when making the 1.2.4 beta,
instead of the the previous 1.2.3-cos1.

There is no distinction between 1.2.3-cos1 and 1.2.3-cos2 in the
changelog, because the reference is the public 1.2.3 release, and it's
easier this way to incorporate the notes in the 1.2.4.1 changelog.

Most importantly, I hope that the users of Visual C++ 8 will be happy
with the -cos2 changes ;) I reverted zconf.h back to the original found
in 1.2.3. I did not, however, roll back the following change:

- Swap the order of #include <stdio.h> and #include "zlib.h" in
  gzio.c, example.c and minigzip.c [Truta]

I think it's good, both for consistency reasons (to put stdio.h near
string.h and stdlib.h) and for practical reasons (in case there is or
there will ever be something in zconf.h to influence the standard
headers, as I intended to influence them by defining macros to turn off
the warnings).

**

I applied Enrico Weigelt's patches, including the pkgconfig patch, but
excluding the destdir patch and the treebuild patch. I find destdir
pointless (why not just run "make prefix=/my/prefix" as it is customary
with other software?) Treebuild is okay, it looks useful, but it should
be in the projects/treebuild directory, not in the top-level directory.

**

There is a stylistic change that I must explain:

- Rename zconf.in.h to zconf.h.in [Truta]

Most other open-source code that I have seen are having file.ext and
file.ext.in, not file.in.ext. Now, with the introduction of zlib.pc.in
in zlib, I think it makes sense to rename zconf.in.h to zconf.h.in. All
references to the former zconf.in.h are updated as well.

**

There is a change that doesn't have a changelog entry. Instead, I
updated the changelog in-place:

- Replace set_data_type() with a more accurate detect_data_type() in trees.c,
  according to the txtvsbin.txt document [Truta]

The old name was eval_data_type(), which makes sense to a statistician,
but in C hacker's slang, "eval" pertains to expression evaluation, and
"detect" more accurately expresses what it does.

The doc/txtvsbin.txt document is updated as well. It is a little
shorter, a typo has been corrected, etc. The initial changelog entry

- Add a document that explains the new text detection scheme to
  doc/txtvsbin.txt [Truta]

is good enough to include in zlib-1.2.4.1.

**

Enjoy!
Cosmin