[ros-dev] Re: [ros-svn]
[gdalsnes]18113:-reorderInsertXscendingOrder macro argument
order andupdate uses
ch at csh-consult.dk
Wed Sep 28 17:57:16 CEST 2005
Oh please don't go there. Do you honestly think you (in general) are smarter at
generating microprocessor instructions than a multi-million hour compiler software?
If the execution speed is acceptable, I will always prefer readability of the source code
over execution speed. I don't really care if some function can be executed 1ns faster if
several functions were collapsed into one.
If I could trade a 1 second added boot up time (~8 second total then?) for readable
ReactOS source code, I would do it immediately ;-)
From: ros-dev-bounces at reactos.com [mailto:ros-dev-bounces at reactos.com] On Behalf Of crashofurit
Sent: 28. september 2005 17:22
To: ReactOS Development List
Subject: Re: [ros-dev] Re: [ros-svn] [gdalsnes]18113:-reorderInsertXscendingOrder macro argument order andupdate uses
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.
On 9/28/05, Casper Hornstrup <ch at csh-consult.dk> wrote:
> The alternative is: do the cleanup at every return, use goto or use
> 1)Cleanup at every return is madness. Most functions in ros do a large
> amount of cleanup at each return and I sometimes spot mistakes where
> one/more return misses some cleanup. Those errors are _hard_ to find.
The functions are too large then. Use more smaller functions instead.
Ros-dev mailing list
Ros-dev at reactos.com
<P>My DeviantArt.com page: <A href="http://crashfourit.deviantart.com/" >http://crashfourit.deviantart.com/</A><BR>My FanFiction.net
bio page: <A href="http://www.fanfiction.net/u/726606/" >http://www.fanfiction.net/u/726606/</A><BR>My Blog: <A
href="http://crashfourit.blogspot.com" >http://crashfourit.blogspot.com</A><BR>America's Debate: <A
href="http://www.americasdebate.com/" >http://www.americasdebate.com/</A> </P>
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Ros-dev