Hello Valters, > On 17 Jul 2023, at 20:15, Valters Jansons wrote: > > On Mon, Jul 17, 2023 at 10:09 PM Michael Tremer > wrote: >> Thanks for reviewing my changes. This was indeed a thing that I did observe as well and fixed it right here: >> >> https://git.ipfire.org/?p=location/libloc.git;a=commitdiff;h=0e894966750e89d1b83c6ad90f37945d4d349118 >> >> The root node got deleted and lots of the code just relied on it being there. Especially when the tree was empty. > > I suspected you would already be aware of the expectations that were > being broken, so wanted to highlight it. Shame on me for not > double-checking whether you had already resolved it before sending the > message! No, I like it when code is double-checked. I did notice myself but after I pushed my first set of changes. I knew there was some work left to do, but deleting the root node was something I thought I had covered when I did not. So, please keep the feedback coming. It will help us all to have a better version of libloc. Best, -Michael > Once again: thank you for the changes! > > -Valters