jerluc / wmii

Automatically exported from code.google.com/p/wmii
MIT License
0 stars 0 forks source link

wmii dies when a hanging '|' is used in /tagrules regex #152

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
$ echo '/vlc|/ -> ~'|wmiir write /tagrules

Result:
wmii dies, printing the following error:
regerror: missing operand for |

What version of the product are you using (wmii -v)? On what operating
system?
wmii-hg2582, libixp-hg116, Arch Linux x86_64, Linux 2.6.30.9

Please provide any additional information below.
I was going to test whether '|' should be escaped, but I thought I would see 
what happens with an empty 
alternative. And it crashed, taking down my session (I guess this is the exact 
reason why I should put wmii 
under a loop in .xinitrc). 

Original issue reported on code.google.com by rogu...@gmail.com on 3 Nov 2009 at 10:52

GoogleCodeExporter commented 9 years ago
This issue was closed by revision a6a9a00266.

Original comment by maglion...@gmail.com on 4 Nov 2009 at 1:51

GoogleCodeExporter commented 9 years ago
Thanks. I'm actually surprised that that regexp doesn't work. It actually 
aborted,
rather than crashed, which I suppose is bad design on the part of the regexp 
library.
And, as for the loop, I'm just paranoid. wmii actually crashes less than most 
other
window managers, but I expect a lot of users to be using the development tip, 
which
can be often unstable, so I recommend the loop.

Original comment by maglion...@gmail.com on 4 Nov 2009 at 1:54