unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Juri Linkov <juri@jurta.org>
Cc: David Abrahams <dave@boost-consulting.com>
Subject: Re: bug in save-some-buffers or diff.el?
Date: 22 Oct 2003 17:35:14 +0300	[thread overview]
Message-ID: <87ismh49y5.fsf@mail.jurta.org> (raw)
In-Reply-To: <uznftehtm.fsf@boost-consulting.com>

David Abrahams <dave@boost-consulting.com> writes:
> Juri Linkov <juri@jurta.org> writes:
> > David Abrahams <dave@boost-consulting.com> writes:
> >> Here's my backtrace:
> >> Debugger entered--Lisp error: (wrong-type-argument stringp nil)
> >>   expand-file-name(nil)
> >>   find-file-noselect(nil)
> >>   diff-parse-differences(732 nil)
> >
> > This bug is already fixed and yesterday checked in to CVS (files
> > compile.el and diff.el).  This bug was caused by setting the variable
> > `compile-auto-highlight' to t.  Now it is nil by default.
> > However, even if you set it back to t, it will not cause the same bug,
> > because function `diff-parse-differences' now is changed not to
> > call the `find-file-noselect' at all.
> 
> Well, the problem was repro'd in a version of emacs built yesterday
> at approx. 5pm EST, and neither of those two files has changed since
> I did my checkout.  I appear to have both changes, as well.
> 
> Hmm, maybe I need to do 
> 
>   nmake recompile && nmake install 
> 
> in the lisp directory?  It surprises me that 
> 
>    nmake && nmake install
> 
> from the nt directory is insufficient after updating my local CVS
> image.

Your backtrace indicates that you use older version of diff.el, because
newest version have no `find-file-noselect' function call at all.
I can't tell how to recompile .el files under Windows, but it's clear
that your .elc files are older than fixed .el files.

BTW, I'm sure that current behavior of `load' is wrong.  It always
tries to load .elc file first, even if it's older than .el.  But often
it causes too much trouble after updating .el files.  There are many
reasons why this happens: user forgets to recompile updated .el files,
something goes wrong in the make process, etc.  Yes, `load' reports
a warning, but usually this warning goes unnoticed, which results in
broken functionality.  Instead, `load' should load the newest of .el
or .elc files, and report the same warning as a simple reminder to
recompile updated .el files to execute them faster.

-- 
http://www.jurta.org/emacs/

  parent reply	other threads:[~2003-10-22 14:35 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-10-22  0:07 bug in save-some-buffers or diff.el? David Abrahams
2003-10-22  8:38 ` Juri Linkov
2003-10-22  9:35   ` David Abrahams
2003-10-22 12:52     ` Stefan Monnier
2003-10-22 15:22       ` David Abrahams
2003-10-22 14:35     ` Juri Linkov [this message]
2003-10-22 16:28       ` David Abrahams
2003-10-22 16:34       ` Kevin Rodgers
2003-10-22 17:52         ` Eli Zaretskii
2003-10-23  4:53           ` Juri Linkov
2003-10-23  6:06             ` Eli Zaretskii
2003-10-23  8:00               ` Juri Linkov
2003-10-24 14:52                 ` Eli Zaretskii
2003-10-23 18:38       ` Richard Stallman
2003-10-23 19:47         ` David Abrahams
2003-10-24 10:41           ` Richard Stallman
2003-10-27 17:44             ` emacshear (was: bug in save-some-buffers or diff.el?) Juri Linkov

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=87ismh49y5.fsf@mail.jurta.org \
    --to=juri@jurta.org \
    --cc=dave@boost-consulting.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).