Half-way through.

Author: matejak

bin/argbash

@@ -495,7 +495,7 @@ then
 	# match against suspicious, then inverse match against correct stuff:
 	# #<optional whitespace>\(allowed\|another allowed\|...\)<optional whitespace><opening bracket <or> end of line>
 	# Then, extract all matches (assumed to be alnum chars + '_') from grep and put them in the error msg.
-	grep_output="$(printf "%s" "$output" | grep '^#\s*\(ARG_\|ARGBASH\)' | grep -v '^#\s*\(ARGBASH_SET_INDENT\|ARG_OPTIONAL_SINGLE\|ARG_VERSION\|ARG_VERSION_AUTO\|ARG_HELP\|ARG_OPTIONAL_INCREMENTAL\|ARG_OPTIONAL_REPEATED\|ARG_VERBOSE\|ARG_OPTIONAL_BOOLEAN\|ARG_OPTIONAL_ACTION\|ARG_POSITIONAL_SINGLE\|ARG_POSITIONAL_INF\|ARG_POSITIONAL_MULTI\|ARG_POSITIONAL_DOUBLEDASH\|ARG_OPTION_STACKING\|ARG_RESTRICT_VALUES\|ARG_DEFAULTS_POS\|ARG_LEFTOVERS\|ARGBASH_WRAP\|INCLUDE_PARSING_CODE\|DEFINE_SCRIPT_DIR\|DEFINE_SCRIPT_DIR_GNU\|ARGBASH_SET_DELIM\|ARGBASH_GO\|ARGBASH_PREPARE\|ARG_TYPE_GROUP\|ARG_TYPE_GROUP_SET\|ARG_USE_ENV\|ARG_USE_PROG\)\s*\((\|$\)' | sed -e 's/#\s*\([[:alnum:]_]*\).*/\1 /' | tr -d '\n\r')"
+	grep_output="$(printf "%s" "$output" | grep '^#\s*\(ARG_\|ARGBASH\)' | grep -v '^#\s*\(ARGBASH_SET_INDENT\|ARG_OPTIONAL_SINGLE\|ARG_VERSION\|ARG_VERSION_AUTO\|ARG_HELP\|ARG_OPTIONAL_INCREMENTAL\|ARG_OPTIONAL_REPEATED\|ARG_VERBOSE\|ARG_OPTIONAL_BOOLEAN\|ARG_OPTIONAL_SWITCH_ON\|ARG_OPTIONAL_SWITCH_OFF\|ARG_OPTIONAL_ACTION\|ARG_POSITIONAL_SINGLE\|ARG_POSITIONAL_INF\|ARG_POSITIONAL_MULTI\|ARG_POSITIONAL_DOUBLEDASH\|ARG_OPTION_STACKING\|ARG_RESTRICT_VALUES\|ARG_DEFAULTS_POS\|ARG_LEFTOVERS\|ARGBASH_WRAP\|INCLUDE_PARSING_CODE\|DEFINE_SCRIPT_DIR\|DEFINE_SCRIPT_DIR_GNU\|ARGBASH_SET_DELIM\|ARGBASH_GO\|ARGBASH_PREPARE\|ARG_TYPE_GROUP\|ARG_TYPE_GROUP_SET\|ARG_USE_ENV\|ARG_USE_PROG\)\s*\((\|$\)' | sed -e 's/#\s*\([[:alnum:]_]*\).*/\1 /' | tr -d '\n\r')"
 	test -n "$grep_output" && die "Your script contains possible misspelled Argbash macros: $grep_output" 1
 fi
 if test "$outfname" != '-'

src/collectors.m4

@@ -348,12 +348,14 @@ dnl $3: help
 dnl $4: default (=off)
 argbash_api([ARG_OPTIONAL_BOOLEAN], _CHECK_PASSED_ARGS_COUNT(1, 4)[m4_do(
 	[[$0($@)]],
-	[m4_ifnblank([$4], [m4_case([$4],
-		[on], [_ARG_OPTIONAL_SWITCH_ON([$1], [$2], [$3])],
-		[off], [_ARG_OPTIONAL_SWITCH_OFF([$1], [$2], [$3])],
-		[_COLLECTOR_FEEDBACK([Problem with argument '$1': Only 'on' or 'off' are allowed as boolean defaults, you have specified '$4'.])])])],
-	[_ADD_OPTIONAL_ARGUMENT_IF_POSSIBLE([$1], [$2], [$3],
-		m4_default_quoted([$4], [off]), [bool])],
+	[m4_ifnblank([$4],
+		[m4_case([$4],
+			[off], [_ARG_OPTIONAL_SWITCH_ON([$1], [$2], [$3])],
+			[on], [_ARG_OPTIONAL_SWITCH_OFF([$1], [$2], [$3])],
+			[_COLLECTOR_FEEDBACK([Problem with argument '$1': Only 'on' or 'off' are allowed as boolean defaults, you have specified '$4'.])])],
+		[_ADD_OPTIONAL_ARGUMENT_IF_POSSIBLE([$1], [$2], [$3],
+			m4_default_quoted([$4], [off]), [bool])],
+	)],
 )])
 
 

src/function_generators.m4

@@ -168,7 +168,7 @@ m4_define([_MAKE_ARGV_PARSING_FUNCTION], [MAKE_FUNCTION(
 	[parse_commandline], [m4_do(
 		[_JOIN_INDENTED(1,
 			_IF_HAVE_POSITIONAL_ARGS([[_positionals_count=0],]),
-			[while test $[]# -gt 0],
+			[[while test $][# -gt 0]],
 			[do],
 		)],
 		[_IF_HAVE_OPTIONAL_ARGS(
@@ -193,8 +193,8 @@ m4_define([_MAKE_ARGV_PARSING_FUNCTION_POSIX], [MAKE_FUNCTION(
 	[[The parsing of the command-line]],
 	[parse_commandline], [m4_do(
 		[_JOIN_INDENTED(1,
-			[while getopts '_GET_GETOPTS_STRING()' _key],
-			[do],
+			[[while getopts ']_GET_GETOPTS_STRING()[' _key]],
+			[[do]],
 		)],
 		[_EVAL_OPTIONALS_GETOPTS],
 		[_INDENT_()done[]_ENDL_()],

src/stuff.m4

@@ -422,7 +422,7 @@ dnl $3: Name of the value-to-variable macro
 dnl $4: The name of the argument-holding variable
 dnl $5: Where to get the last value (optional)
 m4_define([_VAL_OPT_ADD_SPACE_WITHOUT_GETOPT_OR_SHORT_OPT], [_JOIN_INDENTED(_INDENT_LEVEL_IN_ARGV_CASE_BODY,
-	[test $[]# -lt 2 && die "Missing value for the optional argument '$_key'." 1],
+	[[test $][# -lt 2 && die "Missing value for the optional argument '$_key'." 1]],
 	[$3([$1], ["@S|@2"], [$4])],
 	[_APPEND_WRAPPED_ARGUMENT_TO_ARRAY_SPACE([$4], [m4_default_quoted([$5], [@S|@2])])],
 	[shift],

tests/regressiontests/test-onlyopt.m4

@@ -2,7 +2,7 @@
 
 m4_define([if_not_posix], [m4_if(m4_quote(_OUTPUT_TYPE), [posix-script], [$2], [$1])])
 
-dnl m4_define([test], [m4_fatal([BOOM!!!])])
+dnl m4_define([test], [huuuuuuuuuu])
 m4_define([incrx], [m4_fatal([BOOM!!!])])
 # ARG_OPTIONAL_SINGLE([opt-arg], o, [@opt-arg@], x)
 # ARG_VERSION([echo "$0 FOO"])