396071902267c1654f7eafaf60962aad935a9558
galt
  Thu Sep 15 17:08:36 2016 -0700
Changing htmlSafef to use %s|none| instead of %-s because %-s has some legitimate uses in html output which means left-justifying the text. Jim uses it in hgBlat. This will make it more general, although it is longer to read and write. For sqlSafef, there was no legitimate expected need to have left-justified output in the production of a SQL statement.

diff --git src/hg/hgTables/identifiers.c src/hg/hgTables/identifiers.c
index 8b83535..272350f 100644
--- src/hg/hgTables/identifiers.c
+++ src/hg/hgTables/identifiers.c
@@ -492,31 +492,31 @@
 	    if (exampleCount < 10)
 		{
 		++exampleCount;
 		dyStringPrintf(exampleMissingIds, "%s\n", term->name);
 		}
 	    fprintf(f, "%s\n", term->name);
 	    }
 	carefulClose(&f);
 
 	warn("Note: %d of the %d given identifiers have no match in "
 	     "table %s, field %s%s%s%s%s.  "
 	     "Try the \"describe table schema\" button for more "
 	     "information about the table and field.\n"
 	     "%d %smissing identifier(s):\n"
 	     "%s\n"
-	     "<a href='%-s'>Complete list of missing identifiers<a>\n", 
+	     "<a href='%s|none|'>Complete list of missing identifiers<a>\n",
 	     (totalTerms - foundTerms), totalTerms,
 	     curTable, idField,
 	     (xrefTable ? (xrefIsSame ? "" : " or in alias table ") : ""),
 	     (xrefTable ? (xrefIsSame ? "" : xrefTable) : ""),
 	     (xrefTable ? (xrefIsSame ? " or in field " : ", field ") : ""),
 	     (xrefTable ? aliasField : ""),
 	     exampleCount,
 	     exampleCount < missingCount ? "example " : "",
 	     exampleMissingIds->string,
 	     tn.forHtml
 	    );
 	webNewSection("Table Browser");
 	}
     lmCleanup(&lm);
     hashFree(&matchHash);