Re: printing.el v6.6.2

[Vinicius Jose Latorre]

Kai Großjohann <address@hidden> writes:

> 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.

Yes, I'll implement this on next version.


Thanks,


Vinicius