[Top][All Lists]

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

Re: [Nano-devel] [PATCH] browser: merging opendir() in to one

From: Rishabh Dave
Subject: Re: [Nano-devel] [PATCH] browser: merging opendir() in to one
Date: Sun, 22 May 2016 19:35:54 +0530

On Sun, May 22, 2016 at 1:08 AM, Benno Schulenberg
<address@hidden> wrote:
> On Sat, May 21, 2016, at 13:33, Rishabh Dave wrote:
>> Did just the first step: shifting, both, opendir() from
>> do_browser_from() and closedir() from browser_init() to do_browser().
> Again, you changed the behavior.  But to be fair: just in theory,
> because it is not visible.  But it will be visible if after the
> beep() you would add statusbar("woosh"); napms(200); -- then run:
> 'mkdir woosh && cd woosh && ../src/nano' and in another terminal:
> 'rmdir woosh'.  Then type: ^R ^T.  Woosh.  It wouldn't have shown
> the browser menu before your patch.

I don't understand "It wouldn't have shown the browser menu before
your patch"-part. With my patch and your modifications it doesn't show
browser menu. It just shows message "woosh". Nano 2.5.3 and nano with
my patch both only beep there.

> Anyway, I find it uninformative that ^T only beeps when the working
> directory has disappeared, so I am going to add an error message
> for that.  Then the path==NULL case will disappear.
> Bleh...  We have a memory leak there, in do_browse_from().  And in
> get_full_path() too.  Both when getcwd() returns NULL...
> Okay, those are fixed now.
> Try again.  Make your patch again, with a signed-off line, and
> a good commit message.

Umm... Do you mean that I should repeat exactly what I have done in
last patch? No second step (merging rest opendir()s), no handling

reply via email to

[Prev in Thread] Current Thread [Next in Thread]