5b8c4168d4807c729fc8b1f199d9eb03c9411069
galt
  Thu Mar 3 18:40:54 2016 -0800
Replacing simple literal NOSQLINJ in string with the #define NOSQLINJ. This is slightly better because the compiler can catch a mis-spelling of the NOSQLINJ keyword. This was suggested by Angie.

diff --git src/hg/hgTracks/pubsTracks.c src/hg/hgTracks/pubsTracks.c
index 54a2c75..1981921 100644
--- src/hg/hgTracks/pubsTracks.c
+++ src/hg/hgTracks/pubsTracks.c
@@ -62,31 +62,31 @@
 /* assignment of pubs classes to colors */
 static struct hash* pubsClassColors = NULL;
 
 static void pubsParseClassColors() 
 /* parse class colors from hgFixed.pubsClassColors into the hash pubsClassColors */
 {
 if (pubsClassColors!=NULL)
     return;
 
 pubsClassColors = hashNew(0);
 struct sqlConnection *conn = hAllocConn("hgFixed");
 if (!sqlTableExists(conn, "pubsClassColors")) 
     {
     return;
     }
-char *query = "NOSQLINJ SELECT class, rgbColor FROM pubsClassColors";
+char *query = NOSQLINJ "SELECT class, rgbColor FROM pubsClassColors";
 struct sqlResult *sr = sqlGetResult(conn, query);
 char **row = NULL;
 while ((row = sqlNextRow(sr)) != NULL)
     {
     char *class = row[0];
     char *colStr = row[1];
     // copied from genePredItemClassColor - is there no function for this?
     // convert comma sep rgb string to array
     char *rgbVals[5];
     chopString(colStr, ",", rgbVals, sizeof(rgbVals));
     struct rgbColor *rgb;
     AllocVar(rgb);
     rgb->r = (sqlUnsigned(rgbVals[0]));
     rgb->g = (sqlUnsigned(rgbVals[1]));
     rgb->b = (sqlUnsigned(rgbVals[2]));