On Fri Sep 23 12:11:08 2022 +0000, Santino Mazza wrote:
mmm not sure, I think I'll have to make a bigger refactor to actually get the code cleaner by returning the error. For example when I use `if (get_error()) delete_key(key, 0)`, I will have to repeat code for the load_app_registry case or modify load_registry and the functions it calls to also return the error. I think another option could be to use `set_error` and also return the error.
I've been reading other requests implementations, and I think it will be better to pass load_app_registry a pointer to `reply->hkey` and do all the stuff in there, so we don't have to return an error and also don't have to make use of get_error.