if-let macro behavior changed?

[5thWall]

if-let macro behavior changed?

[bakpakin]

So I have bisected this change to Janet version 1.27.0, but I think the new behavior is the correct behavior. The bindings inside if-let are short-circuiting, meaning that in the false case, it is possible that some of the bindings do not have values at all. While we could initialize things to nil, that doesn't sit right with me.

The above code makes sense since new-elapsed will always be truthy, but really should be written:

(let [new-elapsed (+ elapsed 1)]
  (if (> new-elapsed duration)
    (remove-entity wld tween-ent)
    (put tween-data :elapsed-time new-elapsed)))

[sogaiu]

Possibly related #1189

Also, may be you meant 1.32.1 and not 3.32.1?

[AlecTroemel]

Ya letting the false branch of the if-let use variables defined could definitely lead to some confusing code 😆, since you don't necessarily know which definition in the let was false.

Hopefully I did depend on the old behavior anywhere else, otherwise I'll be kicking myself