Filtered view (M!) works again
Andrew V. Samoilov
kai at cmail.ru
Thu Nov 15 10:01:23 UTC 2001
Hi, Pavel!
On Wed, Nov 14, 2001 at 03:33:38PM -0500, Pavel Roskin wrote:
> Hi, Andrew!
>
> On Thu, 8 Nov 2001, Andrew V. Samoilov wrote:
>
> > Hi, Pavel!
> >
> > This patch makes "Filtered view" (M-!) feature work again
> > and fixes memory leak. It was broken recently with file
> > type checks.
>
> Thank you and sorry that I broke it.
>
> I'm sorry for the late reply. I didn't realize that you haven't applied
> your patch yet. I'm applying it now with a small warning fix (gcc wrongly
> thinks that fd may be uninitialized).
Well, it was a reason I did not commit this patch. Unfortunately
it really can be used uninitialized. There is at least one place
(in layout.c, quick view) where view_init called
with command => NULL and file => "". Another unpleasant discovery was:
mc_open ("", ...) and mc_stat ("", ...) operate with current directory
and return usually someting but not -1.
> > P.S. Some days ago I mailed --enable-largefile vs smbfs
> > is fixed. It is not so.
> > sizeof (off_t) == 8 in MC but 4 in samba.
>
> Please feel free to fix samba support.
It seems it is more easy to rewrite it from scratch with
libsmbclient URL syntax:
smb://[[[domain;]user[:password@]]server[/share[/path[/file]]]]
and other good stuffs.
--
Regards,
Andrew.
More information about the mc-devel
mailing list