<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Would it be worth using a function rather than a hash, so that more functionality can be added later if necessary?<div><br class="webkit-block-placeholder"></div><div>John</div><div><br></div><div><div><div>On 23 Jan 2008, at 17:20, Xin Zheng wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Tom,<div><br class="webkit-block-placeholder"></div><div>My aim is to clean up the code so I can easily and safely add new functionality. Extracting the output string like this takes a step in that direction.</div><div><br class="webkit-block-placeholder"> </div><div>From the lessons I've learnt while developing software, I would highly recommend this change. <span class="Apple-style-span" style="font-family: Helvetica; font-size: 12px; "></span></div><div><font class="Apple-style-span" face="Helvetica" size="3"> <span class="Apple-style-span" style="font-size: 12px;"><br class="webkit-block-placeholder"></span></font></div><div><span class="Apple-style-span" style="font-family: Helvetica; font-size: 12px; ">This change would make the application more cohesive. <a href="http://en.wikipedia.org/wiki/Cohesion_%28computer_science%29">http://en.wikipedia.org/wiki/Cohesion_%28computer_science%29</a></span></div><div><font class="Apple-style-span" face="Helvetica" size="3"><span class="Apple-style-span" style="font-size: 12px;"> <br class="webkit-block-placeholder"></span></font></div><div><font class="Apple-style-span" face="Helvetica" size="3"><span class="Apple-style-span" style="font-size: 12px;">Xin</span></font></div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">_______________________________________________</div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; ">dev mailing list</div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><a href="mailto:dev@openstreetmap.org">dev@openstreetmap.org</a></div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><a href="http://lists.openstreetmap.org/cgi-bin/mailman/listinfo/dev">http://lists.openstreetmap.org/cgi-bin/mailman/listinfo/dev</a></div> </blockquote></div><br></div></body></html>