On 03/15/2016 12:38 PM, Zac Medico wrote: > On 03/15/2016 12:04 PM, Brian Dolbec wrote: >> On Mon, 14 Mar 2016 18:04:56 -0700 >> Zac Medico <[email protected]> wrote: >> >> >>>> The only consumer for that allvalid variable is the metadata >>>> UnusedCheck class. >>>> >>>> So the allvalid variable is True until found False >>>> by whichever checks along the way find it to be False. Like a fuse, >>>> it's good until it's blown, then it can never be good again. I >>>> don't think this particular variable justifies a special class that >>>> more fully mimics a fuse. Impossible to reset it like a breaker. >>> >>> Yeah, let's do it. It's a great opportunity to add clarity to the >>> code, and prevent future goofs. >>> >> >> Done, it is now dynamic_data['validity_fuse'] which is a Fuse instance. > > Nice, thank you! > > We can also use Fuse for the 'can_force' boolean, right? >
For 'changed' as well. -- Thanks, Zac
