search

MilesCranmer / DispatchDoctor.jl

The dispatch doctor prescribes type stability
Apache License 2.0
128 stars 6 forks source link

Don't gensym `return_type` #21

Closed danielwe closed 1 month ago

danielwe commented 1 month ago

Just a suggestion, and a rather inconsequential one at that---but there doesn't appear to be any need for a gensymmed name for the return type variable. Using a plain symbol here slightly improves the readability of the outputs of @macroexpand, @code_warntype, et cetera.

danielwe commented 1 month ago

Oh I see why it's needed now, to avoid conflicts with the actual function body. A bit too quick on the trigger there, sorry.