unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [Emacs-diffs] trunk r117941: Default to stack objects on non-GNU/Linux, non-DOS_NT platforms.
@ 2014-09-25  8:15 Eli Zaretskii
  2014-09-25  9:51 ` Dmitry Antipov
                   ` (2 more replies)
  0 siblings, 3 replies; 13+ messages in thread
From: Eli Zaretskii @ 2014-09-25  8:15 UTC (permalink / raw)
  To: Paul Eggert; +Cc: emacs-devel

I don't understand this commit.

  . It was done in complete silence, without prior discussions wrt
    whether to make this the default.  (Yes, I suggested that, but no
    one replied, and we never discussed it.)

  . It arbitrarily excludes the native MS-Windows builds from this
    feature, for no good reasons: the 64-bit Windows build has no
    problems with it, and the problem discovered in the 32-bit build
    has a simple solution.  If you (Paul) didn't want to implement
    that solution yourself, you could have asked _before_ turning
    this on.

    In general, every feature that exists only on some platforms is a
    Bad Thing, as it introduces maintenance problems and in particular
    makes people who work on different platforms unable to usefully
    compare what they see and solve problems reported by others.

  . It does include the 32-bit Cygwin-w32 build which could
    potentially be affected by the same problem as the native Windows
    32-bit builds.

In sum, I don't understand these sneaky practices, and I wish they'd
stopped.  How many times do you have to be told to revert your commits
before you understand that this is not how we do things around here?



^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2014-09-25 20:56 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-09-25  8:15 [Emacs-diffs] trunk r117941: Default to stack objects on non-GNU/Linux, non-DOS_NT platforms Eli Zaretskii
2014-09-25  9:51 ` Dmitry Antipov
2014-09-25 10:14   ` Eli Zaretskii
2014-09-25 12:52   ` Stefan Monnier
2014-09-25 16:12     ` Dmitry Antipov
2014-09-25 18:12       ` Stefan Monnier
2014-09-25 19:08         ` Paul Eggert
2014-09-25  9:52 ` Eli Zaretskii
2014-09-25 16:06 ` Paul Eggert
2014-09-25 16:48   ` Eli Zaretskii
2014-09-25 18:46     ` Paul Eggert
2014-09-25 19:16       ` Eli Zaretskii
2014-09-25 20:56         ` Stefan Monnier

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).