Shouldn't be a problem.  util.binary now checks an entire string, not just the
first 4K.

Unfortunately this got reverted again by Bryan's #4b2c266bf059.

In main, resolving.

> This should IMHO be backed out

Done in crew-stable and crew by 6ed371423d34, thanks.

I asked about that when I submitted the patch ;)

However I believe the other cleanups should stay,
so no backout, just revert to previous behaviour:

http://www.selenic.com/pipermail/mercurial-devel/2008-April/005648.html

changeset:   6247:7f4257b5cbfc
parent:      6242:a375ffc2aa1b
user:        Christian Ebert <blacktrash@gmx.net>
date:        Thu Mar 13 10:42:46 2008 +0100
summary:     win32text: use util.binary to detect \0

This should IMHO be backed out. See for example
usersguide/tutorials/j2ee-tut/fm/J2EE-NBTutorial.pdf in hg.netbeans.org/main as
of 397a011dde00; this should be treated as a binary file, and in Hg 0.9.5 it
was. It contains CRLF, and also NUL - but not in the first 4096 bytes. Therefore
checking out the file on a Windows box with the decode/encode hooks active
prints a warning. In this case can be worked around by running dos2unix on the
file, which seems to still make it viewable as PDF.