From 6ae4cfc9e5f6ff6f47845f418db92f3d04615ed0 Mon Sep 17 00:00:00 2001 From: Filip Leonarski Date: Mon, 3 Aug 2026 20:52:41 +0200 Subject: [PATCH] writer: take --verbose without an argument, and say what -d takes --verbose was declared required_argument while -v carries no colon in the getopt string, so the long form consumed whatever followed it. "jfjoch_writer --verbose tcp://host:5400" swallowed the address as the flag's argument and then failed for want of a data source - the short form was fine, which is presumably why it went unnoticed. The usage line for the root directory asked for . It is a path. Co-Authored-By: Claude Opus 5 (1M context) --- writer/jfjoch_writer.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/writer/jfjoch_writer.cpp b/writer/jfjoch_writer.cpp index 0006f83a..9e4166dc 100644 --- a/writer/jfjoch_writer.cpp +++ b/writer/jfjoch_writer.cpp @@ -22,12 +22,13 @@ void print_usage() { logger.Info("Available options:"); logger.Info("-T | --tcp Use raw TCP/IP instead of ZeroMQ"); logger.Info("-j | --nproc= Number of forks (only with -T)"); - logger.Info("-d | --root_dir= Root directory for file writing"); + logger.Info("-d| --root_dir= Root directory for file writing"); logger.Info("-r | --zmq_repub_port= ZeroMQ port for PUSH socket to republish images"); logger.Info("-f | --zmq_file_port= ZeroMQ port for PUB socket to inform about finalized files"); logger.Info("-w | --rcv_watermark= Receiving ZeroMQ socket watermark (default = 100)"); logger.Info("-W | --repub_watermark= Republish ZeroMQ socket watermark (default = 1000)"); logger.Info("-v | --verbose Verbose output"); + logger.Info("-h This message"); logger.Info(""); } @@ -80,7 +81,7 @@ int main(int argc, char **argv) { {"zmq_file_port", required_argument, 0, 'f'}, {"rcv_watermark", required_argument, 0, 'w'}, {"repub_watermark", required_argument, 0, 'W'}, - {"verbose", required_argument, 0, 'v'}, + {"verbose", no_argument, 0, 'v'}, {"tcp", no_argument, 0, 'T'}, {"nproc", required_argument, 0, 'j'}, {0, 0, 0, 0}