From: Nicolas Goaziou <mail@nicolasgoaziou.fr>
To: Org-mode <emacs-orgmode@gnu.org>
Subject: Re: [RFC] [PATCH] Warn about unexpanded macros on export
Date: Fri, 19 Sep 2014 21:29:28 +0200 [thread overview]
Message-ID: <87bnqbv27b.fsf@nicolasgoaziou.fr> (raw)
In-Reply-To: <87k34zqv9r.fsf@gmail.com> (Aaron Ecay's message of "Fri, 19 Sep 2014 15:12:48 -0400")
Hello,
Aaron Ecay <aaronecay@gmail.com> writes:
> Currently, if a macro is not defined, it will silently produce an empty
> string while exporting. This situation could arise for example if a
> macro name is acidentally mistyped. I think it’s desirable to warn the
> user in this case. The attached patch introduces a function to do so,
> and shows how it would be integrated in the latex backend. This raises
> several questions:
>
> 1. Should the warning be a “message” (allows the export process to
> continue) or a “user-error” (stops the export process)? Or, should
> this be configurable?
Certainly not a message, due to asynchronous export.
> 2. Since this is a feature that many backends will want to use, should
> we introduce a new “abstract” backend from which other backends can
> inherit, which incorporates this feature, and others like it in the
> future? The idea would be similar to prog-mode in emacs, the major
> mode from which programming-language modes can derive. The
> alternative is adding the (macro . org-export-macro-warn) entry
> manually to all the relevant backends, and relying on future backend
> authors to do the same.
>
> 3. Should this even be implemented as part of the backend’s
> translate-alist, or at a lower level?
Don't bother with export back-ends, they never get to see macros, which
are expanded very early in the export process. This explains, for
example, why there is no `macro' translator.
You have two options. Either report an error, as you suggested, or
insert an obnoxious message in the output, e.g., "UNKNOWN MACRO", à la
"DEFINITION NOT FOUND." for footnote definitions. In any case, this
should happen in "org-macro.el", not in the export framework.
Regards,
--
Nicolas Goaziou
next prev parent reply other threads:[~2014-09-19 19:29 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-19 19:12 [RFC] [PATCH] Warn about unexpanded macros on export Aaron Ecay
2014-09-19 19:29 ` Nicolas Goaziou [this message]
2014-09-23 3:25 ` Aaron Ecay
2014-09-23 16:59 ` Grant Rettke
2014-09-23 17:18 ` Aaron Ecay
2014-09-23 17:29 ` Jacob Gerlach
2014-09-23 21:10 ` Nicolas Goaziou
2014-09-23 20:24 ` Nicolas Goaziou
2014-09-28 3:53 ` Aaron Ecay
2014-09-28 6:59 ` Nicolas Goaziou
2014-10-12 15:48 ` Aaron Ecay
2014-09-23 20:26 ` Rasmus
2014-09-28 4:00 ` Aaron Ecay
2014-09-28 7:03 ` Nicolas Goaziou
2014-10-12 15:49 ` Aaron Ecay
2014-10-12 17:19 ` Nicolas Goaziou
2015-03-12 2:55 ` Jacob Gerlach
2015-03-15 8:44 ` Nicolas Goaziou
2015-03-17 16:20 ` Jacob Gerlach
2015-03-17 22:38 ` Nicolas Goaziou
2015-03-18 20:25 ` Jacob Gerlach
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=87bnqbv27b.fsf@nicolasgoaziou.fr \
--to=mail@nicolasgoaziou.fr \
--cc=emacs-orgmode@gnu.org \
/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.