63b239a6a9008928a4b7fa9b0eab06afa2c7a2a0
angie
  Wed May 8 07:21:10 2013 -0700
stricter-compiler warning: address of a char[] will always evaluate as true, use a different test
diff --git src/hg/hgc/pubs.c src/hg/hgc/pubs.c
index 6f64a39..96d5c45 100644
--- src/hg/hgc/pubs.c
+++ src/hg/hgc/pubs.c
@@ -653,32 +653,31 @@
 netParseUrl(clickedFileUrl, &npu);
 struct hash *params = NULL;
 struct cgiVar* paramList = NULL;
 char *paramStr = strchr(npu.file, '?');
 cgiParseInput(paramStr, &params, &paramList);
 struct cgiVar *var = hashFindVal(params, "file");
 char *figId = NULL;
 if (var!=NULL && var->val!=NULL)
     figId = var->val;
 
 char yifPageUrl[4096];
 if (figId) 
     {
     safef(yifPageUrl, sizeof(yifPageUrl), "http://krauthammerlab.med.yale.edu/imagefinder/Figure.external?sp=S%s%%2F%s", extId, figId);
     }
-
-if (!yifPageUrl)
+else
     return;
 
 web2StartSection("section", 
     "<A href=\"%s\">Yale Image Finder</a>: figure where sequences were found", 
     yifPageUrl);
 
 web2ImgLink(yifPageUrl, clickedFileUrl, "Image from YIF", 600, 10, 10); 
 
 web2EndSection();
 }
 
 static bool printSeqSection(char *articleId, char *title, bool showDesc, struct sqlConnection *conn, struct hash* clickedSeqs, bool isClickedSection, bool fasta, char *pslTable, char *articleTable)
 /* print a section with a table of sequences, show only sequences with IDs in hash,
  * There are two sections, respective sequences are shown depending on isClickedSection and clickedSeqs 
  *   - seqs that were clicked on (isClickedSection=True) -> show only seqs in clickedSeqs