> - if (sel.type == SEL_RECTANGULAR)
> - return;
...
> + if(sel.type == SEL_REGULAR) {
> + int len = tlinelen(sel.nb.y);This patch inverts the logic. In the current code the function returns when sel.type == SEL_RECTANGULAR, but in your code it test len only when sel.type == SEL_RECTANGULAR. And I also prefer the return version, because it requires less indentation.
