My memory was bad. There's no rampant bug in the code if one does not place type hints. My bad.
2010/5/29 Dennis <[email protected]> > 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]<clojure%[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]<clojure%[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
