[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#31696: 27.0.50; dotimes-with-progress-reporter: Polimorphic 2nd argu
bug#31696: 27.0.50; dotimes-with-progress-reporter: Polimorphic 2nd argument
Sun, 17 Jun 2018 18:36:56 +0900
Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux)
Tino Calancha <address@hidden> writes:
> On Sun, 3 Jun 2018, Noam Postavsky wrote:
>> Tino Calancha <address@hidden> writes:
>>> +(defmacro dotimes-with-progress-reporter (spec reporter-or-message &rest
>>> + (let ((prep (make-symbol "--dotimes-prep--"))
>>> + ((end (make-symbol "--dotimes-end--"))))
>>> + `(let ((,prep ,reporter-or-message)
>>> + (,end ,(cadr spec)))
>>> + (when (stringp ,prep)
>>> + (setq ,prep (make-progress-reporter ,prep 0 ,end)))
>>> + (dotimes ,spec
>> That's still going to evaluate (cadr spec) twice. You need to change
>> the spec passed to `dotimes' so that it also uses `end'.
> I did it in Bug#31697 but forgot here :-|
> Thank you very much for let me know!
Implemented in master branch as commit
'dotimes-with-progress-reporter: Polymorphic 2nd argument'