Hello list,
this patch makes tag.viewidx take a screen index rather than a screen
object, complying with its doc string. This seems to be more consistent
with other parts of the API to me, and also fixes #963. Config breakage
should be minimal, since this function is usually called without the
screen parameter.
Felix
From 4c017e0a8e50cfc6bee823bdc97c2e5533a8d9ab Mon Sep 17 00:00:00 2001
From: Felix Bier <[email protected]>
Date: Sun, 1 Jul 2012 14:57:25 +0200
Subject: [PATCH 1/2] tag.viewidx should take a screen index

This makes tag.viewidx take a screen index rather than a screen object,
complying with its doc string. This fixes #963.

Signed-off-by: Felix Bier <[email protected]>
---
 lib/awful/tag.lua.in | 2 +-
 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-)

diff --git a/lib/awful/tag.lua.in b/lib/awful/tag.lua.in
index 56fb8d6..0d75366 100644
--- a/lib/awful/tag.lua.in
+++ b/lib/awful/tag.lua.in
@@ -345,7 +345,7 @@ end
 -- @param i The relative index to see.
 -- @param screen Optional screen number.
 function tag.viewidx(i, screen)
-    local screen = screen and screen.index or capi.mouse.screen
+    local screen = screen or capi.mouse.screen
     local tags = capi.screen[screen]:tags()
     local showntags = {}
     for k, t in ipairs(tags) do
-- 
1.7.11.1

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to