svn commit: r190098 - in head/sys/sparc64: fhc sparc64

Julian Elischer julian at elischer.org
Sat Mar 28 22:13:19 PDT 2009


Christoph Mallon wrote:
> Julian Elischer schrieb:
>> M. Warner Losh wrote:
>>> In message: <49C5F88C.3070600 at freebsd.org>
>>>             Andriy Gapon <avg at freebsd.org> writes:
>>> : E.g. you can have a simple 3 line block where you need a local 
>>> variable
>>> : but that block is located 50 lines from start of an enclosing 
>>> function.
>>> : Very convenient when you need to quickly glance the variable's type 
>>> (not).
>>>
>>> No you don't.  There's absolutely nothing wrong with putting them at
>>> the top.  In fact, it is simpler, really, than having to go hunting
>>> for dozens of different declarations.  As someone who has spent a lot
>>> of time looking at code, the time wasted looking for these damn-fool
>>> things really adds up.
>>
>> and in a complicated function, if you have them all over the place you 
>> have no idea as to what the potential stack usage of the function is..
>> This matters in the kernel.
> 
> The declared local variables are neither a lower *nor* an upper bound 
> for stack usage.

no but it can give you an idea as to what is going on..
most modern compilers can reuse space for  non overlapping
variable scope but it's still useful.. I was not saying it
needs to be that way, just that there are often arguments for
doing thing sone way or another that can be non-obvious.


More information about the svn-src-head mailing list