all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Re: [Emacs-diffs] emacs/src fileio.c
       [not found] <E1MtNFk-0002Gn-Dt@cvs.savannah.gnu.org>
@ 2009-10-01 23:23 ` Juanma Barranquero
  2009-10-02  5:54   ` Michael Albinus
  0 siblings, 1 reply; 3+ messages in thread
From: Juanma Barranquero @ 2009-10-01 23:23 UTC (permalink / raw)
  To: Michael Albinus; +Cc: Emacs developers

On Thu, Oct 1, 2009 at 17:07, Michael Albinus <Michael.Albinus@gmx.de> wrote:

> Modified files:
>        src            : fileio.c
>
> Log message:
>        * fileio.c (Fdelete_directory_internal): Renamed from
>        Fdelete_directory.  It is not a command anymore.  It has no file
>        name handler.

I've changed w32fns.c:Fsystem_move_file_to_trash to use
Fdelete_directory_internal instead of Fdelete_directory so Emacs can
bootstrap again on Windows.

But Fdelete_directory_internal is now skipping the file name handler,
so it is likely not safe?

    Juanma




^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Emacs-diffs] emacs/src fileio.c
  2009-10-01 23:23 ` [Emacs-diffs] emacs/src fileio.c Juanma Barranquero
@ 2009-10-02  5:54   ` Michael Albinus
  2009-10-02 12:27     ` Juanma Barranquero
  0 siblings, 1 reply; 3+ messages in thread
From: Michael Albinus @ 2009-10-02  5:54 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: Emacs developers

Juanma Barranquero <lekktu@gmail.com> writes:

> I've changed w32fns.c:Fsystem_move_file_to_trash to use
> Fdelete_directory_internal instead of Fdelete_directory so Emacs can
> bootstrap again on Windows.
>
> But Fdelete_directory_internal is now skipping the file name handler,
> so it is likely not safe?

I believe, we should rather use

  intern ("delete-directory")

Qdelete_directory_internal could be removed from lisp.h then.

Best regards, Michael.

>     Juanma




^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Emacs-diffs] emacs/src fileio.c
  2009-10-02  5:54   ` Michael Albinus
@ 2009-10-02 12:27     ` Juanma Barranquero
  0 siblings, 0 replies; 3+ messages in thread
From: Juanma Barranquero @ 2009-10-02 12:27 UTC (permalink / raw)
  To: Michael Albinus; +Cc: Emacs developers

On Fri, Oct 2, 2009 at 07:54, Michael Albinus <michael.albinus@gmx.de> wrote:

> I believe, we should rather use
>
>  intern ("delete-directory")
>
> Qdelete_directory_internal could be removed from lisp.h then.

OK, thanks for fixing it.

    Juanma




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-10-02 12:27 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <E1MtNFk-0002Gn-Dt@cvs.savannah.gnu.org>
2009-10-01 23:23 ` [Emacs-diffs] emacs/src fileio.c Juanma Barranquero
2009-10-02  5:54   ` Michael Albinus
2009-10-02 12:27     ` Juanma Barranquero

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.