dcb42ca54277d403ea36613201459567ddee8d88
jcasper
  Wed Mar 8 19:24:51 2023 -0800
Fixing a couple of overzealous interpretations of bedParseRgb failing, refs #30569

diff --git src/hg/lib/bedDetail.c src/hg/lib/bedDetail.c
index 67ad816..85f1616 100644
--- src/hg/lib/bedDetail.c
+++ src/hg/lib/bedDetail.c
@@ -361,31 +361,31 @@
          lineFileAbort(lf,
              "thickEnd out of range for %s:%d-%d, thick:%d-%d (chromStart to chromEnd, or 0 if no CDS)",
                        item->name, item->chromStart, item->chromEnd,
                        item->thickStart, item->thickEnd);
      }
 else
      item->thickEnd = item->chromEnd;
 if (wordCount > 8)
     {
     char *comma;
     /*  Allow comma separated list of rgb values here   */
     comma = strchr(row[8], ',');
     if (comma)
         {
         int rgb = bedParseRgb(row[8]);
-        if (rgb < 0)
+        if (rgb == -1)
             lineFileAbort(lf,
                 "Expecting 3 comma separated numbers for r,g,b bed item color.");
         else
             item->reserved = rgb;
         }
     else
         item->reserved = lineFileNeedNum(lf, row, 8);
     }
 if (wordCount > 9)
     item->blockCount = lineFileNeedNum(lf, row, 9);
 if (wordCount > 10)
     {
     sqlSignedDynamicArray(row[10], &item->blockSizes, &count);
     if (count != item->blockCount)
         lineFileAbort(lf,  "expecting %d elements in array", item->blockCount);