strange try+ reflection warnings (0.10.3) #37

Closed
martintrojer opened this Issue Nov 8, 2013 · 1 comment

Comments

Projects
None yet
2 participants
user> *warn-on-reflection*
true
user> (slingshot.slingshot/try+ (inc 1) (catch Exception e (.getMessage e)))
Reflection warning, /tmp/form-init8815318833609231005.clj:1:54 - reference to field getMessage can't be resolved.
2

;; but

user> (try (inc 1) (catch Exception e (.getMessage e)))
2
Contributor

pjstadig commented Mar 9, 2014

I believe this is because slingshot is not adding a type hint for the class selector case. I've tried adding type hints in that case, but for some reason the type hint isn't propagating. https://github.com/pjstadig/slingshot/tree/fix-reflection

@scgilardi scgilardi closed this in #41 Sep 17, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment