unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: [Emacs-diffs] /srv/bzr/emacs/trunk r101202: * lisp/emacs-lisp/macroexp.el (macroexpand-all-1): Use pcase.
       [not found] <E1OpQu4-0005hz-SQ@internal.in.savannah.gnu.org>
@ 2010-08-29 23:05 ` Juanma Barranquero
  2010-08-30 15:43   ` Stefan Monnier
  0 siblings, 1 reply; 10+ messages in thread
From: Juanma Barranquero @ 2010-08-29 23:05 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Emacs developers

On Sat, Aug 28, 2010 at 20:52, Stefan Monnier <monnier@iro.umontreal.ca> wrote:

> ------------------------------------------------------------
> revno: 101202
> committer: Stefan Monnier <monnier@iro.umontreal.ca>
> branch nick: trunk
> timestamp: Sat 2010-08-28 20:52:36 +0200
> message:
>  * lisp/emacs-lisp/macroexp.el (macroexpand-all-1): Use pcase.
>  (macroexp-accumulate): Use `declare'.

Likely cause of this?

In toplevel form:
emacs-lisp/macroexp.el:97:27:Error: Lisp nesting exceeds `max-lisp-eval-depth'


    Juanma



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

* Re: [Emacs-diffs] /srv/bzr/emacs/trunk r101202: * lisp/emacs-lisp/macroexp.el (macroexpand-all-1): Use pcase.
  2010-08-29 23:05 ` [Emacs-diffs] /srv/bzr/emacs/trunk r101202: * lisp/emacs-lisp/macroexp.el (macroexpand-all-1): Use pcase Juanma Barranquero
@ 2010-08-30 15:43   ` Stefan Monnier
  2010-08-30 19:38     ` Juanma Barranquero
  0 siblings, 1 reply; 10+ messages in thread
From: Stefan Monnier @ 2010-08-30 15:43 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: Emacs developers

> Likely cause of this?

> In toplevel form:
> emacs-lisp/macroexp.el:97:27:Error: Lisp nesting exceeds `max-lisp-eval-depth'

Yes, although I did bootstrap to check I didn't hit this one (I did hit
it many times while working on this code, so I know it's pushing this
limit).
Do you have more data to provide?


        Stefan



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

* Re: [Emacs-diffs] /srv/bzr/emacs/trunk r101202: * lisp/emacs-lisp/macroexp.el (macroexpand-all-1): Use pcase.
  2010-08-30 15:43   ` Stefan Monnier
@ 2010-08-30 19:38     ` Juanma Barranquero
  2010-09-11 13:02       ` Stefan Monnier
  0 siblings, 1 reply; 10+ messages in thread
From: Juanma Barranquero @ 2010-08-30 19:38 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Emacs developers

On Mon, Aug 30, 2010 at 17:43, Stefan Monnier <monnier@iro.umontreal.ca> wrote:

> Yes, although I did bootstrap to check I didn't hit this one (I did hit
> it many times while working on this code, so I know it's pushing this
> limit).
> Do you have more data to provide?

Not really. It just happens while bootstrapping on Windows:

[...]
Wrote c:/Devel/emacs/repo/trunk/lisp/emacs-lisp/lmenu.elc

In toplevel form:
emacs-lisp/macroexp.el:97:27:Error: Lisp nesting exceeds `max-lisp-eval-depth'
Wrote c:/Devel/emacs/repo/trunk/lisp/emacs-lisp/map-ynp.elc
[...]

    Juanma



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

* Re: [Emacs-diffs] /srv/bzr/emacs/trunk r101202: * lisp/emacs-lisp/macroexp.el (macroexpand-all-1): Use pcase.
  2010-08-30 19:38     ` Juanma Barranquero
@ 2010-09-11 13:02       ` Stefan Monnier
  2010-09-11 13:12         ` Juanma Barranquero
  2010-10-01 10:52         ` Juanma Barranquero
  0 siblings, 2 replies; 10+ messages in thread
From: Stefan Monnier @ 2010-09-11 13:02 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: Emacs developers

>> Yes, although I did bootstrap to check I didn't hit this one (I did hit
>> it many times while working on this code, so I know it's pushing this
>> limit).
>> Do you have more data to provide?

> Not really. It just happens while bootstrapping on Windows:

> [...]
> Wrote c:/Devel/emacs/repo/trunk/lisp/emacs-lisp/lmenu.elc

> In toplevel form:
> emacs-lisp/macroexp.el:97:27:Error: Lisp nesting exceeds `max-lisp-eval-depth'
> Wrote c:/Devel/emacs/repo/trunk/lisp/emacs-lisp/map-ynp.elc
> [...]

BTW, I haven't heard any other complaint about it: are you the only one
to see it?  Do you still see it?


        Stefan



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

* Re: [Emacs-diffs] /srv/bzr/emacs/trunk r101202: * lisp/emacs-lisp/macroexp.el (macroexpand-all-1): Use pcase.
  2010-09-11 13:02       ` Stefan Monnier
@ 2010-09-11 13:12         ` Juanma Barranquero
  2010-10-01 10:52         ` Juanma Barranquero
  1 sibling, 0 replies; 10+ messages in thread
From: Juanma Barranquero @ 2010-09-11 13:12 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Emacs developers

On Sat, Sep 11, 2010 at 15:02, Stefan Monnier <monnier@iro.umontreal.ca> wrote:

> BTW, I haven't heard any other complaint about it: are you the only one
> to see it?

Hard to know, don't you agree? :-)

> Do you still see it?

Yes.

It only happens on bootstraps, not on normal recompilations.

    Juanma



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

* Re: [Emacs-diffs] /srv/bzr/emacs/trunk r101202: * lisp/emacs-lisp/macroexp.el (macroexpand-all-1): Use pcase.
  2010-09-11 13:02       ` Stefan Monnier
  2010-09-11 13:12         ` Juanma Barranquero
@ 2010-10-01 10:52         ` Juanma Barranquero
  2010-10-19 21:09           ` Stefan Monnier
  1 sibling, 1 reply; 10+ messages in thread
From: Juanma Barranquero @ 2010-10-01 10:52 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Emacs developers

On Sat, Sep 11, 2010 at 15:02, Stefan Monnier <monnier@iro.umontreal.ca> wrote:

> BTW, I haven't heard any other complaint about it: are you the only one
> to see it?

Sean's latest Windows binary
(http://alpha.gnu.org/gnu/emacs/windows/emacs-20100920-bin-i386.zip)
is also lacking lisp/emacs-lisp/macroexp.elc, so it seems like he's
seeing the same problem without realizing it.

    Juanma



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

* Re: [Emacs-diffs] /srv/bzr/emacs/trunk r101202: * lisp/emacs-lisp/macroexp.el (macroexpand-all-1): Use pcase.
  2010-10-01 10:52         ` Juanma Barranquero
@ 2010-10-19 21:09           ` Stefan Monnier
  2010-10-25 23:38             ` Juanma Barranquero
  0 siblings, 1 reply; 10+ messages in thread
From: Stefan Monnier @ 2010-10-19 21:09 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: Emacs developers

>> BTW, I haven't heard any other complaint about it: are you the only one
>> to see it?

> Sean's latest Windows binary
> (http://alpha.gnu.org/gnu/emacs/windows/emacs-20100920-bin-i386.zip)
> is also lacking lisp/emacs-lisp/macroexp.elc, so it seems like he's
> seeing the same problem without realizing it.

Can you try and figure out what value of max-lisp-eval-depth is
necessary to avoid this problem?


        Stefan



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

* Re: [Emacs-diffs] /srv/bzr/emacs/trunk r101202: * lisp/emacs-lisp/macroexp.el (macroexpand-all-1): Use pcase.
  2010-10-19 21:09           ` Stefan Monnier
@ 2010-10-25 23:38             ` Juanma Barranquero
  2010-10-26 15:15               ` Stefan Monnier
  0 siblings, 1 reply; 10+ messages in thread
From: Juanma Barranquero @ 2010-10-25 23:38 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Emacs developers

On Tue, Oct 19, 2010 at 23:09, Stefan Monnier <monnier@iro.umontreal.ca> wrote:

> Can you try and figure out what value of max-lisp-eval-depth is
> necessary to avoid this problem?

512 works, 506 fails, and I'm bored of bisecting...

    Juanma



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

* Re: [Emacs-diffs] /srv/bzr/emacs/trunk r101202: * lisp/emacs-lisp/macroexp.el (macroexpand-all-1): Use pcase.
  2010-10-25 23:38             ` Juanma Barranquero
@ 2010-10-26 15:15               ` Stefan Monnier
  2010-10-26 22:23                 ` Juanma Barranquero
  0 siblings, 1 reply; 10+ messages in thread
From: Stefan Monnier @ 2010-10-26 15:15 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: Emacs developers

>> Can you try and figure out what value of max-lisp-eval-depth is
>> necessary to avoid this problem?
> 512 works, 506 fails, and I'm bored of bisecting...

Oh, that's a lot more precision than I needed.  Can you increase the
default value from 500 to something like 550 or 600?


        Stefan



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

* Re: [Emacs-diffs] /srv/bzr/emacs/trunk r101202: * lisp/emacs-lisp/macroexp.el (macroexpand-all-1): Use pcase.
  2010-10-26 15:15               ` Stefan Monnier
@ 2010-10-26 22:23                 ` Juanma Barranquero
  0 siblings, 0 replies; 10+ messages in thread
From: Juanma Barranquero @ 2010-10-26 22:23 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Emacs developers

> Can you increase the
> default value from 500 to something like 550 or 600?

Set to 600.

    Juanma



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

end of thread, other threads:[~2010-10-26 22:23 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <E1OpQu4-0005hz-SQ@internal.in.savannah.gnu.org>
2010-08-29 23:05 ` [Emacs-diffs] /srv/bzr/emacs/trunk r101202: * lisp/emacs-lisp/macroexp.el (macroexpand-all-1): Use pcase Juanma Barranquero
2010-08-30 15:43   ` Stefan Monnier
2010-08-30 19:38     ` Juanma Barranquero
2010-09-11 13:02       ` Stefan Monnier
2010-09-11 13:12         ` Juanma Barranquero
2010-10-01 10:52         ` Juanma Barranquero
2010-10-19 21:09           ` Stefan Monnier
2010-10-25 23:38             ` Juanma Barranquero
2010-10-26 15:15               ` Stefan Monnier
2010-10-26 22:23                 ` Juanma Barranquero

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).