Re: [patch] bugfix for strip_password



Jindrich Makovicka wrote:
Hi,

In current mc, strip_password misbehaves for URLs like

/#ftp:user:pass hostname/path/path/path

which makes mc display only the string in the form

/#ftp:user hostname

in the title bar, instead of (IMHO correct)

/#ftp:user 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



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]