Chnage style for NULL check

> if (!g_key_file_has_section (config, key))
> {
> set_error ("No remote '%s' found", remotename);
> goto out;
> }

Sorry, if I misunderstand. But the returned value from g_key_file_get_string is used later. So I think this style is the best?