However, in this case, the point of the code was probably to show/teach somebody how to solve a problem. When teaching, you want to make the point as clear as possible, and I think John is trying to point out, in this instance, the extra code to remove the reflection warnings detracts from that goal.
I do not disagree with the idea of removing reflection warnings as a rule and not an exception, especially in production software. I should probably not fan this fire, but I did anyways... :) -- Dennis On Fri, May 28, 2010 at 2:45 PM, Laurent PETIT <[email protected]> wrote: > > > 2010/5/28 Michael Gardner <[email protected]> >> >> On May 28, 2010, at 12:42 PM, Laurent PETIT wrote: >> >> > The rule should really always be: no warning at all (with >> > *warn-on-reflection* set to true, of course). >> >> I strongly disagree. Why should you care about those sorts of warnings >> unless you've already identified a bottleneck that needs elimination? > > Said differently than my previous answer : consider removing warnings as the > act of keeping your code in a good state/shape. I tend to not get rid of > warnings enough in my own java code, but for clojure production code, I > would take warnings wayy more seriously than e.g. java warnings. > > My 0,02€, > > -- > Laurent > > -- > You received this message because you are subscribed to the Google > Groups "Clojure" group. > To post to this group, send email to [email protected] > Note that posts from new members are moderated - please be patient with your > first post. > To unsubscribe from this group, send email to > [email protected] > For more options, visit this group at > http://groups.google.com/group/clojure?hl=en -- You received this message because you are subscribed to the Google Groups "Clojure" group. To post to this group, send email to [email protected] Note that posts from new members are moderated - please be patient with your first post. To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/clojure?hl=en
