Cast some ignored return codes to assist static checkers

Coverity scanning rightly warns that a usually-checked return
status was being ignored in two locations.  These two are in
cases where it's safe to ignore, so I've adjusted the code to
maintain coverage even though they are false positives here.

Author: natoscott

pcp/Instance.c

@@ -81,7 +81,8 @@ static const char* Instance_externalName(Row* super) {
    Instance* this = (Instance*) super;
 
    if (!this->name)
-      pmNameInDom(InDom_getId(this), Instance_getId(this), &this->name);
+      /* ignore any failure here - its safe and we try again next time */
+      (void)pmNameInDom(InDom_getId(this), Instance_getId(this), &this->name);
    return this->name;
 }
 

pcp/Metric.c

@@ -182,7 +182,8 @@ bool Metric_fetch(struct timeval* timestamp) {
 
 void Metric_externalName(Metric metric, int inst, char** externalName) {
    const pmDesc* desc = &pcp->descs[metric];
-   pmNameInDom(desc->indom, inst, externalName);
+   /* ignore a failure here - its safe to do so */
+   (void)pmNameInDom(desc->indom, inst, externalName);
 }
 
 int Metric_lookupText(const char* metric, char** desc) {