On Sun, May 30, 2010 at 12:24:34PM +0200, Uli Schlachter wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA256
> 
> Am 29.05.2010 20:01, Perry Hargrave wrote:
> > +--- Find a suitable fallback tag.
> > +-- @param screen The screen number to look for a tag on. [mouse.screen]
> > +-- @param target A table of tags we consider unacceptable. [{selected()}]
> > +function find_fallback(screen, invalids)
> > +    local scr = screen or capi.mouse.screen
> > +    local t = invalids or {selected()}
> 
> I'd propose:
> 
>   local t = invalids or selectedlist(scr)
> 
what worried me about this was if all the tags on a screen were
selected we might be blocking a tag delete when its
unnecessary.

(because find_fallback would return nil, indicating that there would be
untagged clients)

I'm really not sure which would be the more common use-case...

-- 
perry



-- 
To unsubscribe, send mail to [email protected].

Reply via email to