Using smaller functions instead when we are only using the code once, may help on readability but can cost speed and space. (Think of what standard code C uses to enter and exit a function.) But then again, you can get around that by using declaring those functions as inline.
<br><br>
<div><span class="gmail_quote">On 9/28/05, <b class="gmail_sendername">Casper Hornstrup</b> <<a href="mailto:ch@csh-consult.dk">ch@csh-consult.dk</a>> wrote:</span>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid"><br>> The alternative is: do the cleanup at every return, use goto or use<br>> try/finally.<br>> 1)Cleanup at every return is madness. Most functions in ros do a large
<br>> amount of cleanup at each return and I sometimes spot mistakes where<br>> one/more return misses some cleanup. Those errors are _hard_ to find.<br><br>The functions are too large then. Use more smaller functions instead.
<br><br>Casper<br><br><br>_______________________________________________<br>Ros-dev mailing list<br><a href="mailto:Ros-dev@reactos.com">Ros-dev@reactos.com</a><br><a href="http://reactos.com:8080/mailman/listinfo/ros-dev">
http://reactos.com:8080/mailman/listinfo/ros-dev</a><br></blockquote></div><br><br clear="all"><br>-- <br><P>My DeviantArt.com page: <A href="<a href="http://crashfourit.deviantart.com/"">http://crashfourit.deviantart.com/"
</a>><a href="http://crashfourit.deviantart.com/">http://crashfourit.deviantart.com/</a></A><BR>My FanFiction.net bio page: <A href="<a href="http://www.fanfiction.net/u/726606/"">http://www.fanfiction.net/u/726606/"
</a>><a href="http://www.fanfiction.net/u/726606/">http://www.fanfiction.net/u/726606/</a></A><BR>My Blog: <A href="<a href="http://crashfourit.blogspot.com"">http://crashfourit.blogspot.com"
</a>><a href="http://crashfourit.blogspot.com">http://crashfourit.blogspot.com</a></A><BR>America's Debate: <A href="<a href="http://www.americasdebate.com/"">http://www.americasdebate.com/"
</a>><a href="http://www.americasdebate.com/">http://www.americasdebate.com/</a></A> </P>