[SLOF] [PATCH] Add a global "dir" method

Segher Boessenkool segher at kernel.crashing.org
Fri Jun 10 06:02:01 AEST 2016


On Thu, Jun 09, 2016 at 10:13:41PM +1000, Benjamin Herrenschmidt wrote:
> On Thu, 2016-06-09 at 03:27 -0500, Segher Boessenkool wrote:
> > 
> > Yes, ten gazillion inputs and outputs now ;-)
> > 
> > You already store that state slightly later, why not do it here already
> > and have simpler code, much easier to read?
> 
> I tried, I actually had it that way... not that much easier and I'm
> paranoid. If I want to come back in 2 years and use find-path for
> something else, and forget it clobbers the global is-dir ... I really
> really hate side effects :)

But *most* things do change state, already.  What is on the *stack* is
state.  Putting things in a locally-used variable is not so bad.

Anyway, your choice :-)


Segher


More information about the SLOF mailing list