[Top][All Lists]

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

Re: [PATCH] fix flickering timeout message for slow terminals (gfxterm)

From: Robert Millan
Subject: Re: [PATCH] fix flickering timeout message for slow terminals (gfxterm)
Date: Tue, 15 Jan 2008 13:41:15 +0100
User-agent: Mutt/1.5.13 (2006-08-11)

On Tue, Jan 15, 2008 at 01:25:25PM +0100, Marco Gerards wrote:
> Robert Millan <address@hidden> writes:
> > On Mon, Jan 14, 2008 at 09:32:39PM +0100, Yoshinori K. Okuji wrote:
> >> On Monday 14 January 2008 15:10, Robert Millan wrote:
> >> > As subject says.  Based on suggestions from Vesa.
> >> 
> >> Why do you want to use a nested function?
> >
> > No special reason;  just to restrict the namespace use to the scope where
> > it'll be needed, and avoid passing of parameters (offset, timeout).
> >
> > I'm fine with moving it out if that's preferred.
> If it doesn't uglify your code, that's usually a better thing to do.

Ok, here's a new patch.

Robert Millan

<GPLv2> I know my rights; I want my phone call!
<DRM> What use is a phone call, if you are unable to speak?
(as seen on /.)

Attachment: flickering_timeout.diff
Description: Text Data

reply via email to

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