476b0f83d44b70e9bffc40e9dfde58b9f182a575
max
  Mon May 6 16:13:57 2013 -0700
fixed one-off bug in disulf bind field
diff --git src/hg/hgc/hgc.c src/hg/hgc/hgc.c
index 50b198a..4a83c41 100644
--- src/hg/hgc/hgc.c
+++ src/hg/hgc/hgc.c
@@ -1459,31 +1459,31 @@
 for (kv = keyVals; kv != NULL; kv = kv->next)
     hashAdd(nameToVal, kv->name, kv->val);
 return nameToVal;
 }
 
 void printIdOrLinks(struct asColumn *col, struct hash *fieldToUrl, struct trackDb *tdb, char *idList)
 /* if trackDb does not contain a "urls" entry for current column name, just print idList as it is.
  * Otherwise treat idList as a comma-sep list of IDs and print one row per id, with a link to url,
  * ($$ in url is OK, wildcards like $P, $p, are also OK)
  * */
 {
 // try to find a fieldName=url setting in the "urls" tdb statement, print id if not found
 char *url = NULL;
 if (fieldToUrl!=NULL)
     url = (char*)hashFindVal(fieldToUrl, col->name);
-if ((url==NULL)
+if (url==NULL)
     {
     printf("<td>%s</td></tr>\n", idList);
     return;
     }
 
 // split the id into parts and print each part as a link
 struct slName *slIds = slNameListFromComma(idList);
 struct slName *itemId = NULL;
 printf("<td>");
 for (itemId = slIds; itemId!=NULL; itemId = itemId->next) 
     {
     if (itemId!=slIds)
         printf(", ");
     char *idUrl = replaceInUrl(tdb, url, itemId->name, TRUE);
     printf("<a href=\"%s\" target=\"_blank\">%s</a>", idUrl, itemId->name);