[email protected] wrote: > /* Special FTPOption that maps NLST directly to LIST */ > - if (!is_list && (fsc->options & FTP_OPT_NLSTISLIST) || dashl) { > + if ((!is_list && (fsc->options & FTP_OPT_NLSTISLIST)) || dashl) { > is_list = 1; > }
Wouldn't this be
if (!is_list && ((fsc->options & FTP_OPT_NLSTISLIST) || dashl)) {
as there is no point otherwise? Actually we could drop the !is_list test,
considering that forced-override isn't harmful.
