Re: [PATCH] Display URI's sanely in 'Connect to Server'

Martin Wehner wrote:
> I've only looked superficially at the patch so far, but here's what I
> noticed:
> * It doesn't apply cleanly anymore.
> * It violates the styleguide by omitting the brackets for if statements
> with one-line blocks.
> * Shouldn't it present the Public FTP type UI when the current URI is an
> anonymous one? Currently it just pre-fills the username entry with
> "anonymous".

Thanks. Made the above changes:


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