Re: printing.el v6.6.2

[Kai Großjohann]

Vinicius Jose Latorre <address@hidden> writes:

> The pr-temp-dir has the following setting:
>
> (defcustom pr-temp-dir
>   (pr-dosify-path
>    ;; hacked from `temporary-file-directory' variable in files.el
>    (file-name-as-directory
>     (or (getenv "TMPDIR") (getenv "TMP") (getenv "TEMP")
>       (cond (ps-windows-system "c:/temp")
>             ((memq system-type '(vax-vms axp-vms)) "SYS$SCRATCH:")
>             (t "/tmp")
>             ))))
>   "*Specify a directory for temporary files during printing."
>   :type '(directory :tag "Temporary Directory")
>   :group 'printing)
>
> So, by default, pr-temp-dir and temporary-file-directory has the same value.

I think it is better to set it to 

    (if (boundp 'temporary-file-directory)
        temporary-file-directory
      (SOME-OTHER-CODE-HERE))

This seems cleaner -- if the temporary-file-directory logic is
improved, your code will profit from it.  If the user changes
temporary-file-directory, your code will use the new directory
automatically.

You can use (symbol-value 'temporary-file-directory) to avoid the
byte-compiler warning, if you want that.

What do you think?

kai
-- 
Simplification good!  Oversimplification bad!  (Larry Wall)