[gnuastro-commits] master d91172d: MakeCatalog: properly initializing the '--minvy' column

[Mohammad Akhlaghi]

branch: master
commit d91172de55c14b3e900f99d3301d000e987fc739
Author: Vladimir Markelov <vmatroskin@gmail.com>
Commit: Mohammad Akhlaghi <mohammad@akhlaghi.org>

    MakeCatalog: properly initializing the '--minvy' column
    
    Until now, the '--minvy' column (that gives the Y-axis position of the
    minimum value) was not being initialized when parsing clumps. This was due
    to a typo (mistakenly repeating the relevant '--minvx' column macro two
    times!).
    
    With this commit, the typo has been fixed.
    
    This fixes bug #61293.
---
 NEWS                  | 4 +++-
 bin/mkcatalog/parse.c | 2 +-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/NEWS b/NEWS
index d62ca1b..0d717f2 100644
--- a/NEWS
+++ b/NEWS
@@ -164,8 +164,10 @@ See the end of the file for license conditions.
               with blank values, found and reported by Giulia Golini and
               Raúl Infante-Sainz.
   bug #61287: Two "switch" blocks miss "break" in MakeCatalog when setting
-              output column information: found and fixed by Vladimir
+              output column information, found and fixed by Vladimir
               Markelov.
+  bug #61293: MakeCatalog not properly initializing the '--minvy' column
+              when parsing clumps, found and fixed by Vladimir Markelov.
 
 
 
diff --git a/bin/mkcatalog/parse.c b/bin/mkcatalog/parse.c
index d4997ad..577d75e 100644
--- a/bin/mkcatalog/parse.c
+++ b/bin/mkcatalog/parse.c
@@ -894,7 +894,7 @@ parse_clumps(struct mkcatalog_passparams *pp)
 
   /* For the extrema columns. */
   if( cif[    CCOL_MINVNUM ] || cif[ CCOL_MINVX ]
-      || cif[ CCOL_MINVX   ] || cif[ CCOL_MINVZ ] )
+      || cif[ CCOL_MINVY   ] || cif[ CCOL_MINVZ ] )
     minima_v=parse_init_extrema(cif, GAL_TYPE_FLOAT64, pp->clumpsinobj, 0);
   if( cif[    CCOL_MAXVNUM ] || cif[ CCOL_MAXVX ]
       || cif[ CCOL_MAXVY   ] || cif[ CCOL_MAXVZ ] )