[PATCH] awful.util.table.concat - name and number of arguments

2009-04-15 Thread koniu
Hi, I'm very happy about this functionality making it into awful.util but I see two issues: 1. Why not allow adding together more than 2 tables? (there's also inconsistency about this in the comments) 2. table.concat is an existing lua function which does something else - takes a table and

Re: [PATCH] awful.util.table.concat - name and number of arguments

2009-04-15 Thread Julien Danjou
At 1239824491 time_t, koniu wrote: Patches for both attached. I agree with you, pushed. Cheers, -- Julien Danjou // ᐰ jul...@danjou.info http://julien.danjou.info // 9A0D 5FD9 EB42 22F6 8974 C95C A462 B51E C2FE E5CD // This is the end of my signature. signature.asc Description: Digital

Re: [PATCH] awful.util.table.concat - name and number of arguments

2009-04-15 Thread koniu
On Wed, Apr 15, 2009 at 21:13, Julien Danjou jul...@danjou.info wrote: At 1239824491 time_t, koniu wrote: Patches for both attached. I agree with you, pushed. Thanks Julien, I'm afraid I didn't do enough testing, turns out to be rtable.insert not table.insert, here's a quickie to amend that.