View Single Post
Old 10-30-2018, 01:08 PM   #106
NiLuJe
BLAM!
NiLuJe ought to be getting tired of karma fortunes by now.NiLuJe ought to be getting tired of karma fortunes by now.NiLuJe ought to be getting tired of karma fortunes by now.NiLuJe ought to be getting tired of karma fortunes by now.NiLuJe ought to be getting tired of karma fortunes by now.NiLuJe ought to be getting tired of karma fortunes by now.NiLuJe ought to be getting tired of karma fortunes by now.NiLuJe ought to be getting tired of karma fortunes by now.NiLuJe ought to be getting tired of karma fortunes by now.NiLuJe ought to be getting tired of karma fortunes by now.NiLuJe ought to be getting tired of karma fortunes by now.
 
NiLuJe's Avatar
 
Posts: 13,506
Karma: 26047202
Join Date: Jun 2010
Location: Paris, France
Device: Kindle 2i, 3g, 4, 5w, PW, PW2, PW5; Kobo H2O, Forma, Elipsa, Sage, C2E
@jackie_w:

I may have skimmed the code a wee bit too fast...

* Yeah, it only writes the state to the config if retainStateOverRestart is enabled in the [state] section... :/.

* I may have totally missed how the parser handled ini sections... .

i.e., the test should probably be grep -q "invertActive = yes", since state is the section, so, a [state] header, and not a state: prefix like it's handled in the code.

----

Random comment: since here, both -C & -B are passed, one could also simply invert the arguments instead of using -h . (Not that there's anything wrong with -h: I checked ).

Last edited by NiLuJe; 10-30-2018 at 01:13 PM.
NiLuJe is offline   Reply With Quote