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 <shr3ks...@gmail.com> > 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 <laurent.pe...@gmail.com> > wrote: > > > > > > 2010/5/28 Michael Gardner <gardne...@gmail.com> > >> > >> 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 clojure@googlegroups.com > > Note that posts from new members are moderated - please be patient with > your > > first post. > > To unsubscribe from this group, send email to > > clojure+unsubscr...@googlegroups.com<clojure%2bunsubscr...@googlegroups.com> > > 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 clojure@googlegroups.com > Note that posts from new members are moderated - please be patient with > your first post. > To unsubscribe from this group, send email to > clojure+unsubscr...@googlegroups.com<clojure%2bunsubscr...@googlegroups.com> > 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 clojure@googlegroups.com Note that posts from new members are moderated - please be patient with your first post. To unsubscribe from this group, send email to clojure+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/clojure?hl=en
