Warning/error on seq with no body about janet HOT 5 CLOSED

@sarna I think this commit may have a relevant change:

$ janet
Janet 1.35.2-342a29c7 linux/x64/gcc - '(doc)' for help
repl:1:> (seq [i :range [0 5]])
repl:1:1: compile warning (normal): empty loop body
@[nil nil nil nil nil]

from janet.

FWIW, in the above case, the warning (that shows up on stderr) that appears for an empty-body loop broke a "driver" program that was parsing stderr. It had expectations that only certain output would show up on stderr (i.e. that coming from progams being executed).

I think in this case, the warning should actually not happen -- there seem to be legitimate reasons to have empty-bodied loops (in contrast to seq).

from janet.

I guess if one wanted 5 nils, it would be clearer to express it like:

(seq [i :range [0 5]] nil)

from janet.

Although the change introduced makes sense for seq, I'm not sure it does for loop.

I found an example here where an empty-body loop is being used for side-effects:

    (loop [line :iterate (file/read infile :line)
           :until (string/has-prefix? ">THREE " line)])

from janet.

With loop I think no warning is okay. seq is explicitly a list comprehension, so I'd expect the result to be used - not really true with loop.

from janet.