61dc7885a5f1001e2bc189501379088f2b53379d
galt
  Mon Feb 23 15:42:36 2026 -0800
fixes things found automated codereview. for bigBedCmdSupport lib moved slAddHead, removed stray debugmsg and fixed comment and added static keyword. fixed bigChainToChain chromFound = FALSE, removed stray test makefile, fixed existing typo "tines", and added a missing space after issing space before optionMultiVal(). refs #37146

diff --git src/hg/utils/bigGenePredToGenePred/bigGenePredToGenePred.c src/hg/utils/bigGenePredToGenePred/bigGenePredToGenePred.c
index ba628e251e7..3677cc9f934 100644
--- src/hg/utils/bigGenePredToGenePred/bigGenePredToGenePred.c
+++ src/hg/utils/bigGenePredToGenePred/bigGenePredToGenePred.c
@@ -1,153 +1,153 @@
 /* bigGenePredToGenePred - convert bigGenePred file to genePred. */
 #include "common.h"
 #include "linefile.h"
 #include "hash.h"
 #include "options.h"
 #include "obscure.h"
 #include "bigGenePred.h"
 #include "basicBed.h"
 #include "bigBedCmdSupport.h"
 
 char *clChrom = NULL;
 int clStart = -1;
 int clEnd = -1;
 char *clBed = NULL;
 char *clPos = NULL;
 struct slName *clRange = NULL;
 struct hash *chromHash = NULL;
 boolean skipChromCheck = FALSE;
 
 struct lm *lm = NULL;
 
 void usage()
 /* Explain usage and exit. */
 {
 errAbort(
   "bigGenePredToGenePred - convert bigGenePred file to genePred.\n"
   "usage:\n"
   "   bigGenePredToGenePred bigGenePred.bb genePred.gp\n"
   "   -chrom=chr1 - if set restrict output to given chromosome\n"
   "   -start=N - if set, restrict output to only that over start\n"
   "   -end=N - if set, restict output to only that under end\n"
   "   -range=\"chrom start end\" - if set, restrict output to only that within range from start to end. \n"
   "          This range start is a half-open 0-based coordinate like used in BED files. \n"
   "   -range=chrom:start-end - if set, restrict output to only that within range from start to end. \n"
   "          This range start is a 1-based start position. \n"
   "    Do not use range with chrom, start, and/or end options. \n"
   "   -range may be specified multiple times for multiple ranges. \n"
   "   -bed=in.bed - restrict output to all regions in a BED file\n"
   "   -positions=in.pos - restrict output to all regions in a position file with 1-based start\n"
   "   -udcDir=/dir/to/cache - place to put cache for remote bigBed/bigWigs\n"
   "   -skipChromCheck - skip checking chrom name.\n"
   "\n"
   );
 }
 
 /* Command line validation table. */
 static struct optionSpec options[] = {
    {"chrom", OPTION_STRING},
    {"start", OPTION_INT},
    {"end", OPTION_INT},
    {"bed", OPTION_STRING},
    {"range", OPTION_STRING|OPTION_MULTI},
    {"positions", OPTION_STRING},
    {"udcDir", OPTION_STRING},
    {"skipChromCheck", OPTION_BOOLEAN},
    {NULL, 0},
 };
 
 static void processChromChunk(struct bbiFile *bbi, char *chrom,
                               int start, int end, char *bedName, FILE *f)
 /* Output one chunk.  Only blocks where start is in the range will be written
- * to avoid outputting a block multiple tines.  */
+ * to avoid outputting a block multiple times.  */
 {
 char *chromName = chrom;
 int itemsLeft = 0;
 struct bigBedInterval *bbList = bigBedIntervalQuery(bbi, chromName, start, end, itemsLeft, lm);
 for(; bbList != NULL; bbList = bbList->next)
     {
     struct genePredExt *gp = genePredFromBigGenePred(chromName, bbList);
     genePredTabOut((struct genePred *)gp, f);
     }
 }
 
 void bigGenePredToGenePred(char *bigGenePredFile, char *genePredFile)
 /* bigGenePredToGenePred - convert bigGenePred file to genePred. */
 {
 struct bbiFile *bbi = bigBedFileOpen(bigGenePredFile);
 FILE *f = mustOpen(genePredFile, "w");
 struct bbiChromInfo *chrom, *chromList = bbiChromList(bbi);
 if (!skipChromCheck)
     chromHash = makeChromHash(chromList);
 if (clBed != NULL)
     {
     genericBigToNonBigFromBed(bbi, chromHash, clBed, f, &processChromChunk);
     }
 else if (clPos != NULL)
     {
     genericBigToNonBigFromPos(bbi, chromHash, clPos, f, &processChromChunk);
     }
 else if (clRange != NULL)
     {
     genericBigToNonBigFromRange(bbi, chromHash, f, clRange, &processChromChunk);
     }
 else
     {
     boolean chromFound = FALSE;
     for (chrom = chromList; chrom != NULL; chrom = chrom->next)
 	{
 
 	if (clChrom != NULL && !sameString(clChrom, chrom->name))
 	    continue;
 
 	chromFound = TRUE;
 	int start = 0, end = chrom->size;
 	if (clStart >= 0)
 	    start = clStart;
 	if (clEnd >= 0)
 	    {
 	    end = clEnd;
 	    if (end > chrom->size)
 		end = chrom->size;
 	    }
 	if (start > end)
 	    errAbort("invalid range, start=%d > end=%d", start, end);
 
 	processChromChunk(bbi, chrom->name, start, end, NULL, f);
 	}
     if (clChrom && !chromFound && !skipChromCheck)
 	errAbort("specified chrom %s not found in bigGenePred", clChrom);
     }
 bbiChromInfoFreeList(&chromList);
 carefulClose(&f);
 bbiFileClose(&bbi);
 }
 
 int main(int argc, char *argv[])
 /* Process command line. */
 {
 lm = lmInit(0);
 optionInit(&argc, argv, options);
 if (argc != 3)
     usage();
 clChrom = optionVal("chrom", clChrom);
 clStart = optionInt("start", clStart);
 clEnd = optionInt("end", clEnd);
 clBed = optionVal("bed", clBed);
 clRange = optionMultiVal("range", clRange);
 clPos = optionVal("positions", clPos);
 skipChromCheck = optionExists("skipChromCheck");
 udcSetDefaultDir(optionVal("udcDir", udcDefaultDir()));
 
 if ((clBed || clPos || clRange) && (clChrom || (clStart >= 0) || (clEnd >= 0)))
     errAbort("-bed or -positions or -range can not be used with -chrom -start or -end options");
 if ((clBed && clPos) || (clBed && clRange) || (clPos && clRange))
     errAbort("-bed, -positions, and -range can not be used together");
 
 bigGenePredToGenePred(argv[1], argv[2]);
 
 lmCleanup(&lm);
 if (verboseLevel() > 1)
     printVmPeak();
 return 0;
 }