Description: ------------ Hello again ! $>pear ... upgrade-all ok: Console_Getargs 1.2.0 I have downloaded example.php from CVS (1.7) ---------> If I run: [...]$ php -q example.php para ... Parameters: para ---------> Everything OK ---------> But if I run: [...]$ php -q example.php p-ra ... ERROR! ... Unknown argument ra ---------> Test on both '--' not only the 2nd one ! Regards. bertrand. Reproduce code: --------------- // diff Getargs.php Getargs-minus.php 634c634 < if (strlen($arg) > 1 && $arg{1} == '-') { --- > if (preg_match('/--.+/', $arg)) { 893c893 < if (strlen($arg) > 1 && $arg{1} == '-' || --- > if (preg_match('/--.+/', $arg) ||

I missed summary again ??? sorry, bertrand Gugger

if (strlen($arg) > 1 && $arg{0} == '-' && $arg{1} == '-') { is probably faster than a regex and does not require PCRE.

>>if (strlen($arg) > 1 && $arg{0} == '-' && $arg{1} == '-') { >>is probably faster than a regex and does not require PCRE. You're right, but quickness is not in question ! (or we need to talk from other stuff in Getargs)

>>is probably faster than a regex and does not require PCRE. >You're right, but quickness is not in question ! yes, but not adding a new dependency to a stable package is the way to go :-)

>>>is probably faster than a regex and does not require PCRE. >>You're right, but quickness is not in question ! >yes, but not adding a new dependency to a stable package is the way to go :-) Does it? Do you mean: If the php script never ask some regexp stuff, then this PHP extension is not loaded ? I'm not understanding the sense of not doing regexp Sure you mean it's better to glue PHP's kernel, should be best, I agree. regexp not in the kernel ? OK, here, we don't need it. >>> if (strlen($arg) > 1 && $arg{0} == '-' && $arg{1} == '-') { is absolutly right ! (faster and more understandable)

Fixed. The arg{0} == '-' check was actually needed in two places, where you guys noted and in the isValue() method. The fix will be added to the next release (1.2.1)