<div>
<br>
</div>
<div></div>
<p style="color: #A0A0A8;">Le mardi 11 décembre 2012 à 20:56, David Golden a écrit :</p>
<blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px;">
<span><div><div><div>On Tue, Dec 11, 2012 at 9:10 AM, damien krotkine <<a href="mailto:dkrotkine@gmail.com">dkrotkine@gmail.com</a>> wrote:</div><blockquote type="cite"><div><div>The only drawback is performance : there is an additional indirection, and</div><div>code dereferencing might be costly. The optimization of the route caching</div><div>stops wher your route code starts, so it won't be optimized. You don't get</div><div>performance hits ( or less, at least I think) with attributes.</div></div></blockquote><div><br></div><div>I'm not sure it's clear cut. Right now, there is a before hook that</div><div>runs for *every* route that has to do this:</div><div><br></div><div>* method call to get the coderef for the route</div><div>* function call to retrieve attributes for coderef</div><div>* logic on attributes returned</div><div><br></div></div></div></span></blockquote><div><br></div><div>Hm you're right, I had not realized it'd be for *every* route. Indeed we need more benchmark here :) But I like the attribute style. If it can help in decision making, catalyst makes use of them as well, and it's rather elegant.</div><div> </div><blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px;"><span><div><div><div></div><div>If only a fraction of routes require authentication, then that's a lot</div><div>of extra overhead compared to wrapping only the subs that need it.</div><div>And even if all routes require auth, the indirection is one call</div><div>compared to two. The two calls might be cacheable, though, but even</div><div>still, a single extra sub call only on routes that need it isn't too</div><div>bad.</div><div><br></div><div>David</div><div><br></div><div>-- </div><div>David Golden <<a href="mailto:xdg@xdg.me">xdg@xdg.me</a>></div><div>Take back your inbox! → <a href="http://www.bunchmail.com">http://www.bunchmail.com</a>/</div><div>Twitter/IRC: @xdg</div><div>_______________________________________________</div><div>dancer-users mailing list</div><div><a href="mailto:dancer-users@dancer.pm">dancer-users@dancer.pm</a></div><div><a href="http://lists.preshweb.co.uk/mailman/listinfo/dancer-users">http://lists.preshweb.co.uk/mailman/listinfo/dancer-users</a></div></div></div></span>
</blockquote>
<div>
<br>
</div>