all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Michael Heerdegen <michael_heerdegen@web.de>
To: Philipp Stephani <p.stephani2@gmail.com>
Cc: 29857@debbugs.gnu.org
Subject: bug#29857: 27.0.50; error: "Loading `nil': old-style backquotes detected!"
Date: Sat, 30 Dec 2017 23:33:55 +0100	[thread overview]
Message-ID: <87vagn7smk.fsf@web.de> (raw)
In-Reply-To: <CAArVCkTgngo-CAU1bBAwfYUTz6=SfYkvQ93wAf+C4OSgWZhWSQ@mail.gmail.com> (Philipp Stephani's message of "Sat, 30 Dec 2017 14:34:09 +0000")

Hi Philipp,

thanks for working on this.

>  > It would be reasonable and easy to remove the "Loading `nil'" part in
>  > the case where no file is being loaded.
>
>  Yes, I think that would be good.
>
> OK, I've sent a patch to do that:
> https://lists.gnu.org/archive/html/emacs-devel/2017-12/msg00901.html

Yes, that should be ok, I think.

>  > Otherwise I'd accept introducing a variable to control whether
>  > oldstyle backquotes should raise an error or get interpreted as
>  > newstyle.
>
>  That would be optimal for my case.
>
> I've sent a patch for this as well:
> https://lists.gnu.org/archive/html/emacs-devel/2017-12/msg00902.html

Thanks.  Looks like it does what I want.

> Then I guess el-search should document the new behavior. Previously
> (up to Emacs 26) (read "(,@ x)") returns (\,@ x), now (with the new
> variable) it returns ((\,@ x)).  However, if el-search uses `read` for
> both the buffer text and the search pattern, the interpretations
> should still match (within one version of Emacs).

Yes, it does.  I think there is no problem with this new behavior (well,
unless you either try to make use of old-style backquotes, or need the
semantics of old-style backquotes to transform old code that makes use
of old-style backquotes - I don't need to support these cases).


Thanks,

Michael.





  reply	other threads:[~2017-12-30 22:33 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-26 14:38 bug#29857: 27.0.50; error: "Loading `nil': old-style backquotes detected!" Michael Heerdegen
2017-12-29 20:34 ` Philipp Stephani
2017-12-29 20:59   ` [PATCH 1/2] Add new variable to force new-style backquote interpretation Philipp Stephani
2017-12-29 20:59     ` [PATCH 2/2] Improve error message for old-style backquotes Philipp Stephani
2017-12-30  6:19     ` [PATCH 1/2] Add new variable to force new-style backquote interpretation Paul Eggert
2017-12-30  9:02       ` Philipp Stephani
2017-12-30 23:07         ` Paul Eggert
2017-12-30 23:29           ` Philipp Stephani
2017-12-31  0:40             ` Paul Eggert
2018-01-07 13:21               ` Philipp Stephani
2017-12-31  5:08             ` Stefan Monnier
2017-12-30 14:00   ` bug#29857: 27.0.50; error: "Loading `nil': old-style backquotes detected!" Michael Heerdegen
2017-12-30 14:34     ` Philipp Stephani
2017-12-30 22:33       ` Michael Heerdegen [this message]
2018-01-07 13:26         ` Philipp Stephani

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

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

  git send-email \
    --in-reply-to=87vagn7smk.fsf@web.de \
    --to=michael_heerdegen@web.de \
    --cc=29857@debbugs.gnu.org \
    --cc=p.stephani2@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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.