Fix #8558

joa-quim · joa-quim · commit 9369240246c9 · 2025-01-06T23:03:04.000Z
Issue was that with binary input the number of requested columns did not take -Cn into account.
diff --git a/src/gmtbinstats.c b/src/gmtbinstats.c
@@ -295,7 +295,7 @@ static int parse (struct GMT_CTRL *GMT, struct GMTBINSTATS_CTRL *Ctrl, struct GM
 		n_errors += gmt_M_check_condition (GMT, Ctrl->S.mode == -2, "Option -S: Unable to decode radius\n");
 		n_errors += gmt_M_check_condition (GMT, Ctrl->S.mode == -3, "Option -S: Radius is negative\n");
 	}
-	n_errors += gmt_check_binary_io (GMT, (Ctrl->W.active) ? 4 : 3);
+	n_errors += gmt_check_binary_io(GMT, (Ctrl->C.mode == GMTBINSTATS_COUNT) ? 2 : (Ctrl->W.active) ? 4 : 3);
 	if (Ctrl->W.active) {	/* Update the mode if median or mode */
 		if (Ctrl->C.mode == GMTBINSTATS_MEDIAN) Ctrl->C.mode = GMTBINSTATS_MEDIANW;
 		if (Ctrl->C.mode == GMTBINSTATS_MODE) Ctrl->C.mode = GMTBINSTATS_MODEW;

Original file line number	Diff line number	Diff line change
`@@ -295,7 +295,7 @@ static int parse (struct GMT_CTRL GMT, struct GMTBINSTATS_CTRL Ctrl, struct GM`
`295`	`295`	`n_errors += gmt_M_check_condition (GMT, Ctrl->S.mode == -2, "Option -S: Unable to decode radius\n");`
`296`	`296`	`n_errors += gmt_M_check_condition (GMT, Ctrl->S.mode == -3, "Option -S: Radius is negative\n");`
`297`	`297`	`}`
`298`		`- n_errors += gmt_check_binary_io (GMT, (Ctrl->W.active) ? 4 : 3);`
	`298`	`+ n_errors += gmt_check_binary_io(GMT, (Ctrl->C.mode == GMTBINSTATS_COUNT) ? 2 : (Ctrl->W.active) ? 4 : 3);`
`299`	`299`	`if (Ctrl->W.active) { /* Update the mode if median or mode */`
`300`	`300`	`if (Ctrl->C.mode == GMTBINSTATS_MEDIAN) Ctrl->C.mode = GMTBINSTATS_MEDIANW;`
`301`	`301`	`if (Ctrl->C.mode == GMTBINSTATS_MODE) Ctrl->C.mode = GMTBINSTATS_MODEW;`