unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: martin rudalics <rudalics@gmx.at>
To: Thierry Volpiatto <thierry.volpiatto@gmail.com>
Cc: Chong Yidong <cyd@stupidchicken.com>, emacs-devel@gnu.org
Subject: Re: bug in 23.2.92 with anything
Date: Mon, 17 Jan 2011 18:48:07 +0100	[thread overview]
Message-ID: <4D3480D7.1070303@gmx.at> (raw)
In-Reply-To: <87tyh72103.fsf@gmail.com>

 > Here what i changed:
 >
 > ,----
 > |   if (NILP (norecord))
 > |     {
 > |       ++window_select_count;
 > |       XSETFASTINT (w->use_time, window_select_count);
 > |     }
 > |
 > |   /* if (EQ (window, selected_window)) */
 > |   /*   return window; */
 > |   if (EQ (window, selected_window))
 > |     {
 > |      inhibit_point_swap = 0;
 > |      return window;
 > |     }
 > |
 > `----
 >
 > I have applied these changes on emacs-23 branch after this commit:
 >
 > ,----
 > | commit d0c73d59998f4686dbb2a0d7b05e16d7161fe75d
 > | Author: Stefan Monnier <monnier@iro.umontreal.ca>
 > | Date:   Sun Jan 16 10:40:47 2011 -0500
 > |
 > |     * image.c (syms_of_image): Don't access XSYMBOL's internals directly.
 > `----
 >
 > and it seem to work fine :-)
 > Thanks.

So this should hopefully fix my initial proposal.  Could you also try
the other patch I sent you?  You would have to revert to the state of
the repository for that purpose.

martin



  reply	other threads:[~2011-01-17 17:48 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-15 19:05 bug in 23.2.92 with anything Thierry Volpiatto
2011-01-15 19:47 ` Chong Yidong
2011-01-15 20:27   ` Thierry Volpiatto
2011-01-15 20:05 ` martin rudalics
2011-01-15 21:00   ` Thierry Volpiatto
2011-01-15 21:35   ` Chong Yidong
2011-01-15 21:54     ` martin rudalics
2011-01-15 22:33       ` Thierry Volpiatto
2011-01-16  0:04         ` martin rudalics
2011-01-16  7:16           ` Thierry Volpiatto
2011-01-16 10:37             ` martin rudalics
2011-01-17  9:49               ` Thierry Volpiatto
2011-01-17 10:28                 ` martin rudalics
2011-01-17 10:37                   ` Thierry Volpiatto
2011-01-17 10:47                     ` martin rudalics
2011-01-17 13:09                       ` Thierry Volpiatto
2011-01-17 10:45                   ` Thierry Volpiatto
2011-01-17 12:00                   ` Thierry Volpiatto
2011-01-17 17:48                     ` martin rudalics [this message]
2011-01-17 18:09                       ` Thierry Volpiatto
2011-01-18  7:47                         ` martin rudalics
2011-01-18  8:55                           ` Thierry Volpiatto
2011-01-18  9:12                             ` martin rudalics
2011-01-18 10:12                               ` Thierry Volpiatto
2011-01-18 12:02                                 ` Simon Leinen
2011-01-22  2:45                                   ` Chong Yidong
2011-01-18 15:11                             ` Eli Zaretskii
2011-01-18 15:50                               ` Thierry Volpiatto
2011-01-18 15:52                                 ` Eli Zaretskii
2011-01-18 18:52                                   ` Thierry Volpiatto
2011-01-20  1:17                                   ` David De La Harpe Golden
2011-01-20  7:37                                     ` Thierry Volpiatto
2011-01-20  8:25                                       ` Eli Zaretskii
2011-01-20 11:07                                         ` Stephen J. Turnbull
2011-01-16  4:26   ` Stefan Monnier
2011-01-16  9:30     ` Eli Zaretskii
2011-01-16 11:35       ` martin rudalics
2011-01-16 13:48         ` Eli Zaretskii
2011-01-16 17:51           ` martin rudalics
2011-01-16 15:16         ` Stefan Monnier
2011-01-16 17:51           ` martin rudalics
2011-01-16 22:48             ` Eli Zaretskii
2011-01-16 23:23               ` martin rudalics
2011-01-17 12:52                 ` Eli Zaretskii
2011-01-17 17:48                   ` martin rudalics
2011-01-16 10:35     ` martin rudalics
2011-01-17 10:09       ` Thierry Volpiatto
2011-01-17 10:43         ` martin rudalics

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=4D3480D7.1070303@gmx.at \
    --to=rudalics@gmx.at \
    --cc=cyd@stupidchicken.com \
    --cc=emacs-devel@gnu.org \
    --cc=thierry.volpiatto@gmail.com \
    /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).