basilisp-lang / basilisp

A Clojure-compatible(-ish) Lisp dialect targeting Python 3.8+
https://basilisp.readthedocs.io
Eclipse Public License 1.0
198 stars 4 forks source link

*warn-on-reflection* support #881

Closed ikappaki closed 4 months ago

ikappaki commented 4 months ago

Hi,

is reflection checking at runtime relevant in Basilisp, and if so, how difficult would it be to implement the *warn-on-reflection dynamic var to warn of such instances at compile time?

I'm only asking because clojure.pprint does sets this flag at the top while there are many typehints set in the code.

Thanks

chrisrink10 commented 4 months ago

*warn-on-reflection* is not needed in Basilisp. Basilisp has other warnings which cover similar cases, but which are generally set per-session. Type hints are not used by the compiler at all in Basilisp.