Re: [libvirt] [PATCH 1/7] datatypes: avoid redundant __FUNCTION__

virLibConnError already includes __FUNCTION__ in its output, so we were redundant.  Furthermore, clang warns that __FUNCTION__ is not a string literal (at least __FUNCTION__ will never contain %, so it was not a security risk).

@@ -344,8 +344,16 @@ virGetDomain(virConnectPtr conn, const char *name, const unsigned char *uuid) {     virDomainPtr ret = NULL;     char uuidstr[VIR_UUID_STRING_BUFLEN]; -    if ((!VIR_IS_CONNECT(conn)) || (name == NULL) || (uuid == NULL)) { -        virLibConnError(VIR_ERR_INVALID_ARG, __FUNCTION__); +    if (!VIR_IS_CONNECT(conn)) { +        virLibConnError(VIR_ERR_INVALID_ARG, _("no connection")); +        return(NULL); +    }