Re: Windows 64 port

[Paul Eggert]

On 02/29/2012 08:03 PM, Eli Zaretskii wrote:
> There's nothing bogus about using the right data type

But the code in question is already using a correct data type.
That is, for the diffs marked "Not needed for Windows 64" in
<http://lists.gnu.org/archive/html/emacs-devel/2012-02/msg00762.html>,
Emacs is already using data types that work,
both on mainstream platforms and on Windows 64.

>  or declare a variable or an argument `const' when it is.

Suppose someone proposed lots of patches like this:
                                          
-  size_t len = strlen (string);                                                
+  size_t const len = strlen (string);

on the grounds that the XYZ Corp. compiler warns whenever
code fails to use 'const' at every opportunity.
That would be bogus -- that's not the Emacs
programming style, and we shouldn't slavishly alter
mainstream code merely to pacify a third-party compiler
that prefers a different style.

> I asked several times to do the same for texinfo.tex when you sync with
> gnulib, and you declined.

Those patches were automatically merged from an already-debugged
and well-modularized upstream, and it's trivial to see which parts
affect texinfo.tex.

In contrast, here we're dealing with a handwritten patch that is
tickling core Emacs internals in tricky, low-level ways, a patch that
almost surely still contains bugs, partly because it is trying to do
too much at once.  It's entirely appropriate to spend more time with
such a patch, to see which parts are really needed for Windows 64, and
to isolate the changes as much as possible to the Windows-specific
code base.