emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
From: Martyn Jago <martyn.jago@btinternet.com>
To: emacs-orgmode@gnu.org
Subject: Re: var `date' lacks a prefix warnings
Date: Fri, 27 Apr 2012 21:17:40 +0100	[thread overview]
Message-ID: <m21un9j5ej.fsf@btinternet.com> (raw)
In-Reply-To: 87mx5xvwfi.fsf@Rainer.invalid

Hi Achim

Achim Gratz <Stromeko@nexgo.de> writes:

> Martyn Jago writes:
>> Regarding the remaining Org-mode `lacks a prefix' warnings and
>> particularly in light of the recent message on emacs.devel [1] would it
>> not be sufficient to simply alias date and entry locally:
>>
>> (defvaralias 'org--date 'date)
>> (defvaralias 'org--entry 'entry)
>
> That doesn't work, AFAIK: defvaralias binds the symbol (it looks up
> what is behind the symbol and creates another symbol for it), but
> dynamic scoping does not (you look up what's behind the symbol each time
> you use it).  The problem is that when a symbol gets dynamically scoped
> it cannot be used as a lexical symbol after that, so any code that uses
> "date" and "entry" after the defvar doesn't get a lexcial binding.  You
> would need a language construct that would let you capture a symbol
> dynamically, internalize it (in essence it would be renaming that symbol
> in some scope) and then erase all traces of that symbol having been
> symbolically scoped so that it can be used lexically again.  I don't
> think anything like that currently exists.

I appreciate `defvaralias' doesn't work nicely with lexical variables,
but I am assuming somewhat that we are talking about dynamic variables
here?

If the global date / entry are dynamic then the dynamic `defvaralias'
alias will also be dynamic, and any prior use of the dynamic variable
will surely be a dynamic binding? I'm going to do some further testing
anyway, since I'm interested, but so far things appear to work.

>> p.s. I should point out that my personal view is that a serious well
>> maintained Emacs library such as Org-mode should have a zero warning
>> policy for official releases.
>
> Here's a problem that originates outside of our control.  At the very
> least, we can't fix it just in org.

I agree we can't fix the source of the problem but IF we can fix the
warning by removing `lack of prefixes' in Org-mode, then we have
demonstrated that we acknowledge the warning and have dealt with it.

In other words, if we can fix the problem within Org-mode we are not
propagating the problem. If we don't, then perhaps we didn't fix the
warning because we don't fix warnings.

However, it may indeed be impossible to fix as you pointed out!

Best, Martyn

      reply	other threads:[~2012-04-27 20:17 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-27 13:04 var `date' lacks a prefix warnings Martyn Jago
2012-04-27 14:45 ` Bastien
2012-04-27 20:35   ` Martyn Jago
2012-04-27 18:53 ` Achim Gratz
2012-04-27 20:17   ` Martyn Jago [this message]

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.orgmode.org/

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

  git send-email \
    --in-reply-to=m21un9j5ej.fsf@btinternet.com \
    --to=martyn.jago@btinternet.com \
    --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 public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.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).