Why is defun not executed during load-file?

Why is defun not executed during load-file?

[Jean Louis]

I have this in the hyperscope.el and upon load-file the last function
is not executed, it does not generate the functions. But if I evaluate
it specifically, it generates functions.

Is there any reason for that?

(defun hyperscope-generate-the-add-function-by-hyperdocument-type (type-name id function-name)
  (eval `(defun ,(intern (symbol-name function-name)) ()
	   ,(format "Add new `%s' hyperdocument to Hyperscope." type-name)
	   (interactive)
	   (let* ((parent (hyperscope-select-set))
		  (prompt ,(format "New `%s' hyperdocument name: " type-name))
		  (name (read-from-minibuffer prompt)))
	     (hlink-add-generic name "" ,id parent nil)))))

(defun hyperscope-generate-add-functions-by-hyperdocument-types ()
  (let ((types (hyperscope-hyperdocument-types)))
    (while types
      (let* ((type (pop types))
	     (id (nth 0 type))
	     (type (nth 1 type))
	     (type-name type)
	     (type (downcase (string-replace " " "-" type)))
	     (function-name (intern (format "hyperscope-add-new-%s-hyperdocument" type))))
	(hyperscope-generate-the-add-function-by-hyperdocument-type type-name id function-name)))))

(hyperscope-generate-add-functions-by-hyperdocument-types)

Re: Why is defun not executed during load-file?

[Stefan Monnier via Users list for the GNU Emacs text editor]

Jean Louis [2021-05-30 12:02:25] wrote:
> I have this in the hyperscope.el and upon load-file the last function
> is not executed, it does not generate the functions. But if I evaluate
> it specifically, it generates functions.
>
> Is there any reason for that?
>
> (defun hyperscope-generate-the-add-function-by-hyperdocument-type (type-name id function-name)
>   (eval `(defun ,(intern (symbol-name function-name)) ()

My crystal ball says it's punishment for using `eval` and recommends
using macro(s) instead.  I wouldn't trust its judgment, but the
recommendation sounds useful anyway.

BTW, `intern + symbol-name` ends up a no-op if `function-name` is
already an interned symbol.


        Stefan

Re: Why is defun not executed during load-file?

[Jean Louis]

* Stefan Monnier via Users list for the GNU Emacs text editor <help-gnu-emacs@gnu.org> [2021-05-30 16:32]:
> Jean Louis [2021-05-30 12:02:25] wrote:
> > I have this in the hyperscope.el and upon load-file the last function
> > is not executed, it does not generate the functions. But if I evaluate
> > it specifically, it generates functions.
> >
> > Is there any reason for that?
> >
> > (defun hyperscope-generate-the-add-function-by-hyperdocument-type (type-name id function-name)
> >   (eval `(defun ,(intern (symbol-name function-name)) ()
> 
> My crystal ball says it's punishment for using `eval` and recommends
> using macro(s) instead.  I wouldn't trust its judgment, but the
> recommendation sounds useful anyway.
> 
> BTW, `intern + symbol-name` ends up a no-op if `function-name` is
> already an interned symbol.

I just did not find a way to make it working. Now I have put more care
and I got this to work:

(defmacro hyperscope-generate-the-add-function-by-hyperdocument-type (type-name id function-name)
  (list 'defun function-name '()
	(format "Add new `%s' hyperdocument to Hyperscope." type-name)
	'(interactive)
	'(let* ((parent (hyperscope-select-set))
		(prompt ,(format "New `%s' hyperdocument name: " type-name))
		(name (read-from-minibuffer prompt)))
	   (hlink-add-generic name "" ,id parent nil))))

(defun hyperscope-generate-add-functions-by-hyperdocument-types ()
  (let ((types (hyperscope-hyperdocument-types)))
    (while types
      (let* ((type (pop types))
	     (id (nth 0 type))
	     (type (nth 1 type))
	     (type-name type)
	     (type (downcase (string-replace " " "-" type)))
	     (function-name (format "hyperscope-add-new-%s-hyperdocument" type)))
	(insert "\n" type-name " " id " " function-name)
	(hyperscope-generate-the-add-function-by-hyperdocument-type type-name id function-name)))))

(hyperscope-generate-add-functions-by-hyperdocument-types)

Now, this does not use `eval' and generates functions when I evaluate
it.

But again it will not generate functions when I load the file.

Is that supposed to be so?

This is smallest example:

(defmacro my-generate-function (name)
  (list 'defun (intern name) '()
	"My function"
	'(interactive)
	'(message "hello")))

(my-generate-function "my-hello3")

and that example works well, it does generate function `my-hello3' but
my above doesn't.

On the other hand I have observed if I do just one time:

(fmakunbound 'hyperscope-add-new-markdown-hyperdocument)

then this function will never be generated again in the same
instance. I have to run another instance to be able to generate that
function.

Do you know why is that?


-- 
Jean

Take action in Free Software Foundation campaigns:
https://www.fsf.org/campaigns

Sign an open letter in support of Richard M. Stallman
https://stallmansupport.org/

Re: Why is defun not executed during load-file?

[Stefan Monnier]

> I just did not find a way to make it working. Now I have put more care
> and I got this to work:
>
> (defmacro hyperscope-generate-the-add-function-by-hyperdocument-type (type-name id function-name)
>   (list 'defun function-name '()
> 	(format "Add new `%s' hyperdocument to Hyperscope." type-name)
> 	'(interactive)
> 	'(let* ((parent (hyperscope-select-set))
> 		(prompt ,(format "New `%s' hyperdocument name: " type-name))
> 		(name (read-from-minibuffer prompt)))
> 	   (hlink-add-generic name "" ,id parent nil))))
>
> (defun hyperscope-generate-add-functions-by-hyperdocument-types ()
>   (let ((types (hyperscope-hyperdocument-types)))
>     (while types
>       (let* ((type (pop types))
> 	     (id (nth 0 type))
> 	     (type (nth 1 type))
> 	     (type-name type)
> 	     (type (downcase (string-replace " " "-" type)))
> 	     (function-name (format "hyperscope-add-new-%s-hyperdocument" type)))
> 	(insert "\n" type-name " " id " " function-name)
> 	(hyperscope-generate-the-add-function-by-hyperdocument-type type-name id function-name)))))
>
> (hyperscope-generate-add-functions-by-hyperdocument-types)

Try to `M-: macroexpand-all '(defun hyperscope-generate-add-functions-by-hyperdocument-types ...))
to see that your code doesn't do what you think:
`hyperscope-generate-add-functions-by-hyperdocument-types` needs to be
a macro.


        Stefan

Re: Why is defun not executed during load-file?

[Emanuel Berg via Users list for the GNU Emacs text editor]

Jean Louis wrote:

>   (eval `(defun ,(intern (symbol-name function-name)) ()

One-liner of the year award :)

-- 
underground experts united
https://dataswamp.org/~incal

Re: Why is defun not executed during load-file?

[Emanuel Berg via Users list for the GNU Emacs text editor]

Stefan Monnier wrote:

> Try to `M-: macroexpand-all '(defun
> hyperscope-generate-add-functions-by-hyperdocument-types
> ...)) to see that your code doesn't do what you think:
> `hyperscope-generate-add-functions-by-hyperdocument-types`
> needs to be a macro.

OK, in this case you mean?

Because in general you don't need a macro to generate
a function?

I wrote this:

(defmacro file-to-variable (file var)
  `(setq ,var ,(file-to-string file)) )

(defmacro file-to-variable-integer (file var)
  `(setq ,var ,(string-to-number (file-to-string file))) )

I wrote that to get a function? Don't remember...

https://dataswamp.org/~incal/emacs-init/file-write-to.el

Also from the unofficial code:

(defun syms-status (sym-list)
  (mapcar (lambda (s) (list s (symbol-value s))) sym-list))

(defmacro vars-status (&rest sym-list)
  `(syms-status ',sym-list))

Also don't remember :)

-- 
underground experts united
https://dataswamp.org/~incal

Re: Why is defun not executed during load-file?

[Stefan Monnier via Users list for the GNU Emacs text editor]

Emanuel Berg via Users list for the GNU Emacs text editor [2021-05-30 21:56:05] wrote:
> Stefan Monnier wrote:
>> Try to `M-: macroexpand-all '(defun
>> hyperscope-generate-add-functions-by-hyperdocument-types
>> ...)) to see that your code doesn't do what you think:
>> `hyperscope-generate-add-functions-by-hyperdocument-types`
>> needs to be a macro.
> OK, in this case you mean?

Everything you quoted from me above was written specifically about your code.
So very much "this case", yes.


        Stefan

Re: Why is defun not executed during load-file?

[Emanuel Berg via Users list for the GNU Emacs text editor]

Stefan Monnier via Users list for the GNU Emacs text editor wrote:

>> OK, in this case you mean?
>
> Everything you quoted from me above was written specifically
> about your code. So very much "this case", yes.

OK...

But actually, how do you put it, it wasn't me :)

-- 
underground experts united
https://dataswamp.org/~incal

Re: Why is defun not executed during load-file?

[Jean Louis]

* Emanuel Berg via Users list for the GNU Emacs text editor <help-gnu-emacs@gnu.org> [2021-05-30 22:47]:
> Jean Louis wrote:
> 
> >   (eval `(defun ,(intern (symbol-name function-name)) ()
> 
> One-liner of the year award :)

I still did not make it. It works when evaluated with hand, but not
with load-file.


-- 
Jean

Take action in Free Software Foundation campaigns:
https://www.fsf.org/campaigns

Sign an open letter in support of Richard M. Stallman
https://stallmansupport.org/

Re: Why is defun not executed during load-file?

[Emanuel Berg via Users list for the GNU Emacs text editor]

Jean Louis wrote:

>>> (eval `(defun ,(intern (symbol-name function-name)) ()
>> 
>> One-liner of the year award :)
>
> I still did not make it. It works when evaluated with hand,
> but not with load-file.

Macros are much more difficult to debug...

-- 
underground experts united
https://dataswamp.org/~incal

Re: Why is defun not executed during load-file?

[Jean Louis]

* Emanuel Berg via Users list for the GNU Emacs text editor <help-gnu-emacs@gnu.org> [2021-05-31 01:11]:
> Jean Louis wrote:
> 
> >>> (eval `(defun ,(intern (symbol-name function-name)) ()
> >> 
> >> One-liner of the year award :)
> >
> > I still did not make it. It works when evaluated with hand,
> > but not with load-file.
> 
> Macros are much more difficult to debug...

I am fine to generate functions without macros, just tell me how. I
need names of functions dynamically generated.


-- 
Jean

Take action in Free Software Foundation campaigns:
https://www.fsf.org/campaigns

Sign an open letter in support of Richard M. Stallman
https://stallmansupport.org/

Re: Why is defun not executed during load-file?

[Drew Adams]

> Macros are much more difficult to debug...

`macroexpand` and `macroexpand-1` are your friends.

Separate that first step, which is just reduction
(aka rewriting / reduction semantics), from the
subsequent step of Lisp-evaluating the sexp that
results from it.

So debug the macro-expansion step, to ensure you
get the sexp you want.  And debug evaluation of
that sexp - separately.

Re: Why is defun not executed during load-file?

[Emanuel Berg via Users list for the GNU Emacs text editor]

Drew Adams wrote:

>> Macros are much more difficult to debug...
>
> `macroexpand` and `macroexpand-1` are your friends.

Also, writing macros compared to writing functions typically
involves more bugs...

-- 
underground experts united
https://dataswamp.org/~incal

Re: Why is defun not executed during load-file?

[Michael Heerdegen]

Jean Louis <bugs@gnu.support> writes:

> I am fine to generate functions without macros, just tell me how. I
> need names of functions dynamically generated.

I think you can use `defalias', a simple function.  That would require
to use lexical binding so that variables like `prompt' can get captured
in a closure.  That would look like this:

#+begin_src emacs-lisp
;;;; Your first file line   -*- lexical-binding: t -*-
;; [...]
(defun hyperscope-generate-the-add-function-by-hyperdocument-type (type-name id function-name)
  (defalias function-name
    (lambda ()
      (interactive)
      (let* ((parent (hyperscope-select-set))
	     (prompt (format "New `%s' hyperdocument name: " type-name))
	     (name (read-from-minibuffer prompt)))
        (hlink-add-generic name "" id parent nil)))
    ;; Docstring gets an argument of `defalias':
    (format "Add new `%s' hyperdocument to Hyperscope." type-name)))
#+end_src


Michael.

Re: Why is defun not executed during load-file?

[Jean Louis]

* Michael Heerdegen <michael_heerdegen@web.de> [2021-05-31 02:27]:
> Jean Louis <bugs@gnu.support> writes:
> 
> > I am fine to generate functions without macros, just tell me how. I
> > need names of functions dynamically generated.
> 
> I think you can use `defalias', a simple function.  That would require
> to use lexical binding so that variables like `prompt' can get captured
> in a closure.  That would look like this:


That I must try. Thanks.


-- 
Jean

Take action in Free Software Foundation campaigns:
https://www.fsf.org/campaigns

Sign an open letter in support of Richard M. Stallman
https://stallmansupport.org/