Re: eqin comments differ from implementation

On Thu, Dec 15, 2011 at 10:00:18AM +0100, arno formella wrote:
>   using indent 2.2.10 I observed that the break-function-decl-args-end
> long option was not recognized well.
> 
>   digging into the source I found that the comment of function "eqin"
> says that "true" is returned when the two argument strings are equal,
> however, its implementation returns "true" whenever the first argument
> is a prefix of the second one, which explains the observed behavior.
> 
>   suggestion:  insert  the line
> 
> if(*s2) return false;
> 
This is not proper fix. It breaks parsing other options. Run "make -C
regression" after compilation to see all the failures.

-- Petr

_______________________________________________
bug-indent mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/bug-indent

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.18 (GNU/Linux)

iF4EABEIAAYFAk7rOasACgkQEsnFx2fG+qJeFQD/WcGZ1Go+XjMWOZPh/mIS2vBG
P9ZRG2M4y1knsIBqcmsA/jViRux2dY4cUMqdpzOokVW/My/W3t9Pxi9ANq7g+BJf
=ezoL
-----END PGP SIGNATURE-----