[patch] bugfix for strip_password

Roland Illig roland.illig at gmx.de
Sun Sep 19 12:36:16 UTC 2004


Jindrich Makovicka wrote:
> Hi,
> 
> In current mc, strip_password misbehaves for URLs like
> 
> /#ftp:user:pass at hostname/path/path/path
> 
> which makes mc display only the string in the form
> 
> /#ftp:user at hostname
> 
> in the title bar, instead of (IMHO correct)
> 
> /#ftp:user at hostname/path/path/path
> 
> The attached patch attempts to re-implement the function with the 
> correct behavior.

Hello Jindrich,

I wrote a unit test for strip_password, and both versions (the old one 
and yours) translate the string as they should. Therefore I don't see a 
need to apply your patch.

Roland



More information about the mc-devel mailing list