--- Begin Message ---
Subject: |
`iter-do' doesn't instrument its body |
Date: |
Sat, 28 Oct 2017 13:07:00 +0200 |
Attached trivial patch simply adds `(declare (debug ...))' to the
macro. I believe it is perfectly safe and trivial, no mention in NEWS
etc. is needed. I would ask to merge it to branch 26 if possible.
Note that it doesn't quite let you debug generators anyway because of
bug #28725, but the change makes sense regardless. In particular, with
`iter2' package you should be able to debug generator functions called
from `iter-do'.
Paul
0001-Add-missing-debug-declaration-to-iter-do.patch
Description: Text Data
--- End Message ---
--- Begin Message ---
Subject: |
Re: bug#29039: `iter-do' doesn't instrument its body |
Date: |
Fri, 03 Nov 2017 15:51:51 +0200 |
> From: Paul Pogonyshev <address@hidden>
> Date: Sat, 28 Oct 2017 14:48:14 +0200
> Cc: address@hidden
>
> Because it allows debugging generator functions with `iter2' package.
> `iter2' replaces only `iter-defun' and `iter-lambda', but uses all the
> rest of infrastructure (`iter-yield', `iter-next', `iter-do' and so
> on).
>
> I'm 100% sure this change cannot break anything in `generator' and is
> logical in itself.
Thanks, pushed.
--- End Message ---