Re: crash in display, triggered by calendar

[Richard Stallman]

    > PS. may I suggest that you add some defvars in diary-lib.el to avoid
    > compiler warnings?

    Hmmm... I'm not sure a good idea in all the cases.  Some of the
    variables (diary-entries-list, or entry in sexp forms) never exist at
    the top level.  Ed, what do you think?

What does "exist at top level" mean for a variable?  Do you mean
that they don't have any global bindings?

It is ok to defvar variables that don't have global bindings.

It is also ok to write a defvar that gives them global bindings of
nil; If those global bindings are never used, they don't do any harm.
This way, the defvar can have a doc string.  Those doc strings will be
useful.

The one somewhat ugly aspect is that some of these variables, such as
number, date, entry, and d-file, have very short and nondescript names
not tied to the diary package.  It is somewhat unclean to be using
those names in this way.

What are those variables used for?  Can we rename them?
Can we rename displayed-month to diary-month, etc?
Aside from date, none of them seems to be documented
in either manual.