unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Dan Nicolaescu <dann@ics.uci.edu>
To: YAMAMOTO Mitsuharu <mituharu@math.s.chiba-u.ac.jp>
Cc: Chong Yidong <cyd@stupidchicken.com>,
	Stefan Monnier <monnier@iro.umontreal.ca>,
	emacs-devel@gnu.org
Subject: Re: Carbon port and multi-tty
Date: Wed, 07 May 2008 18:03:40 -0700	[thread overview]
Message-ID: <200805080103.m4813eWH020955@sallyv1.ics.uci.edu> (raw)
In-Reply-To: <wlabj1ir1w.wl%mituharu@math.s.chiba-u.ac.jp> (YAMAMOTO Mitsuharu's message of "Thu, 08 May 2008 09:21:31 +0900")

YAMAMOTO Mitsuharu <mituharu@math.s.chiba-u.ac.jp> writes:

  > >>>>> On Wed, 07 May 2008 00:15:52 -0700, Dan Nicolaescu <dann@ics.uci.edu> said:
  > 
  > > YAMAMOTO Mitsuharu <mituharu@math.s.chiba-u.ac.jp> writes:
  > >> >>>>> On Tue, 06 May 2008 22:55:12 -0700, Dan Nicolaescu
  > >> <dann@ics.uci.edu> said:
  > >> 
  > >> >> The bug introduced by this change is mentioned here: >>
  > >> http://lists.gnu.org/archive/html/emacs-devel/2007-12/msg01069.html
  > >> 
  > >> > OK, a 1 line change (in a ~1000 lines patch) that you seemed to
  > >> have > known about for a while.
  > >> 
  > >> 1-line change can introduce a nasty bug as actually seen in this
  > >> case.  And this could be easily avoided by not changing the part
  > >> that is unrelated to multi-tty.
  > 
  > > And it would have been easily fixed, like all other bugs in emacs,
  > > had the platform not been declared unmaintained the moment the patch
  > > came out.  Not saying that without the patch the code would not even
  > > compile.  Not mentioning that you could have simply fixed it without
  > > causing all this racket.
  > 
  > Again, I'm saying about mixing unrelated changes, not this particular
  > single-line bug.  Please don't understate the problem.

You keep repeating the same thing over and over.  We heard you the first
time, and nobody agreed.

We have all realized by now that you are making a mountain out of a mole hill.

This code has been in CVS for a year, you have made a choice to neither
undo it, nor move a single finger to make it completely functional.  
The fact that you have known about a one line bug for 4 months and chose
not to fix it, but instead create all this commotion did not win you any
friends.

You have already been told by the 2 head Emacs maintainers: "If you're disturbed by
the remaining problems on Carbon, please try working on a fix instead of
complaining." "If you can do better, show us.  Otherwise, please stop complaining"

PLEASE STOP WASTING OUR TIME.

  > And about this ChangeLog entry:
  > 
  > 2008-05-07  Dan Nicolaescu  <dann@ics.uci.edu>
  > 
  > 	* macfns.c (Fx_create_frame): Make a copy of frame parameters
  > 	because the original parameters are in pure storage now.
  > 
  > The primary purpose of the copying is not for the parameters allocated
  > in the pure storage, but to prepare for frame parameter clearing in
  > x_get_arg.

This is the same ChangeLog used by Martin Rudalics to fix a similar
problem that was discovered by Juanma Barranquero that manifested itself
by trying to write to pure storage.




  reply	other threads:[~2008-05-08  1:03 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-06 16:55 Carbon port and multi-tty Chong Yidong
2008-05-06 17:49 ` Ted Zlatanov
2008-05-06 22:08 ` YAMAMOTO Mitsuharu
2008-05-07  1:58   ` Stefan Monnier
2008-05-07  2:36     ` YAMAMOTO Mitsuharu
2008-05-07  2:37       ` YAMAMOTO Mitsuharu
2008-05-07  3:13       ` Stefan Monnier
2008-05-07  3:18         ` YAMAMOTO Mitsuharu
2008-05-07  3:52           ` Nick Roberts
2008-05-07  4:01             ` YAMAMOTO Mitsuharu
2008-05-07  4:26               ` Stefan Monnier
2008-05-07  6:00             ` David Kastrup
2008-05-07  8:50             ` Eli Zaretskii
2008-05-07  4:28           ` Dan Nicolaescu
2008-05-07  5:00             ` YAMAMOTO Mitsuharu
2008-05-07  5:55               ` Dan Nicolaescu
2008-05-07  6:10                 ` YAMAMOTO Mitsuharu
2008-05-07  6:40                   ` David Kastrup
2008-05-07  6:47                     ` YAMAMOTO Mitsuharu
2008-05-07  7:56                       ` Stephen J. Turnbull
2008-05-07  8:42                         ` David Kastrup
2008-05-07  9:41                           ` Eli Zaretskii
2008-05-07 10:05                             ` David Kastrup
2008-05-07 10:26                               ` Eli Zaretskii
2008-05-07 10:51                                 ` David Kastrup
2008-05-07 13:40                                   ` Eli Zaretskii
2008-05-07 13:53                                     ` David Kastrup
2008-05-08  8:44                                 ` Richard M Stallman
2008-05-08  8:56                                   ` David Kastrup
2008-05-08 10:01                                     ` Eli Zaretskii
2008-05-08 10:11                                       ` David Kastrup
2008-05-08  9:53                                   ` Eli Zaretskii
2008-05-09 11:13                                     ` Richard M Stallman
2008-05-07 10:07                           ` YAMAMOTO Mitsuharu
2008-05-07 10:16                             ` David Kastrup
2008-05-07 10:32                               ` YAMAMOTO Mitsuharu
2008-05-07  7:15                   ` Dan Nicolaescu
2008-05-08  0:21                     ` YAMAMOTO Mitsuharu
2008-05-08  1:03                       ` Dan Nicolaescu [this message]
2008-05-08  1:25                         ` YAMAMOTO Mitsuharu
2008-05-08  2:07                       ` Stefan Monnier

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200805080103.m4813eWH020955@sallyv1.ics.uci.edu \
    --to=dann@ics.uci.edu \
    --cc=cyd@stupidchicken.com \
    --cc=emacs-devel@gnu.org \
    --cc=mituharu@math.s.chiba-u.ac.jp \
    --cc=monnier@iro.umontreal.ca \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).