Compiler warning

Compiler warning

[Andreas Röhler]

Hi,

get a strange

Warning: Unused lexical variable ‘i’

refering to this:

(defun py-forward-block (&optional arg decorator bol)
   "[...]"
   (interactive "p")
   (let (erg)
     (dotimes (i (or arg 1))
       (let ((orig (point)))
	(setq erg (py--end-base 'py-block-re orig decorator bol))))
     (when (and py-verbose-p (called-interactively-p 'any)) (message 
"%s" erg))
     erg))


While code below compiles without warning:

(defun foo (&optional arg)
   (interactive "p")
   (dotimes (i (or arg 1))
     (insert "Me again\n")))

Any idea?

Thanks,
Andreas
GNU Emacs 27.0.50 (build 1, i686-pc-linux-gnu, GTK+ Version 3.14.5) of 
2018-05-29

Re: Compiler warning

[Noam Postavsky]

On 18 July 2018 at 07:38, Andreas Röhler <andreas.roehler@easy-emacs.de> wrote:

> get a strange
>
> Warning: Unused lexical variable ‘i’
>
> refering to this:
>
> (defun py-forward-block (&optional arg decorator bol)
>   "[...]"
>   (interactive "p")
>   (let (erg)
>     (dotimes (i (or arg 1))
>       (let ((orig (point)))
>         (setq erg (py--end-base 'py-block-re orig decorator bol))))

You haven't use `i', so the warning seems fitting. Use `_' instead of
`i' to tell the compiler the variable is ignored on purpose.

> While code below compiles without warning:
>
> (defun foo (&optional arg)
>   (interactive "p")
>   (dotimes (i (or arg 1))
>     (insert "Me again\n")))

Did you compile this one without lexical-binding?

Re: Compiler warning

[Andreas Röhler]

On 18.07.2018 13:48, Noam Postavsky wrote:
> On 18 July 2018 at 07:38, Andreas Röhler <andreas.roehler@easy-emacs.de> wrote:
> 
>> get a strange
>>
>> Warning: Unused lexical variable ‘i’
>>
>> refering to this:
>>
>> (defun py-forward-block (&optional arg decorator bol)
>>    "[...]"
>>    (interactive "p")
>>    (let (erg)
>>      (dotimes (i (or arg 1))
>>        (let ((orig (point)))
>>          (setq erg (py--end-base 'py-block-re orig decorator bol))))
> 
> You haven't use `i', so the warning seems fitting. Use `_' instead of
> `i' to tell the compiler the variable is ignored on purpose.
> 

Thanks.


>> While code below compiles without warning:
>>
>> (defun foo (&optional arg)
>>    (interactive "p")
>>    (dotimes (i (or arg 1))
>>      (insert "Me again\n")))
> 
> Did you compile this one without lexical-binding?
> 


Ehm, yes. Okay, thanks!