Skip to content
Snippets Groups Projects
Commit c820571b authored by Jonathan Lebon's avatar Jonathan Lebon
Browse files

errors: check for an error before prefixing

Minor tweak to the new `GLNX_AUTO_PREFIX_ERROR`. Since the common case
is that there's no errors, let's bring down the same check that
`g_prefix_error` does to avoid a function call most of the time.
parent 7d6a31fb
Branches
Tags
No related merge requests found
......@@ -94,7 +94,7 @@ glnx_prefix_error (GError **error, const char *fmt, ...)
* ```
* gboolean start_http_request (..., GError **error)
* {
* GLNX_AUTO_PREFIX_ERROR("HTTP request", error)
* GLNX_AUTO_PREFIX_ERROR ("HTTP request", error)
*
* if (!libhttp_request_start (..., error))
* return FALSE;
......@@ -109,7 +109,8 @@ typedef struct {
static inline void
glnx_cleanup_auto_prefix_error (GLnxAutoErrorPrefix *prefix)
{
g_prefix_error (prefix->error, "%s: ", prefix->prefix);
if (prefix->error && *(prefix->error))
g_prefix_error (prefix->error, "%s: ", prefix->prefix);
}
G_DEFINE_AUTO_CLEANUP_CLEAR_FUNC(GLnxAutoErrorPrefix, glnx_cleanup_auto_prefix_error)
#define GLNX_AUTO_PREFIX_ERROR(text, error) \
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment