Thanks for this. For strtonum - I think it does no harm to let people do -p0 or -p99999999 to mean as small or as big as possible.
With split-window, -p is a % of the available space, but you have made it a % of the existing size? Is this more useful? You should be able to get the available space by looking at the size of the parent cell (wp->layout_cell->parent->sx and sy). On Sun, Oct 06, 2019 at 03:34:33PM -0700, Anindya Mukherjee wrote: > Thanks Thomas for the tips! I have adjusted the code accordingly. > Regarding the args_strtonum call: I want to disallow percent <=0 and > > 100. I thought adding 1, 100 is a convenient way of doing that, and I get > a "too small/large" error when invalid values are supplied.A Is there any > issue with doing it this way? > Best, > Anindya > On Sun, Oct 6, 2019 at 3:03 PM Thomas Adam <[email protected]> wrote: > > Hi, > > Thanks.A Comments below: > > 1.A You shouldn't need to use 0U for integer definitions. > 2.A In terms of calling args_strtonum() for percentage:A 0, INT_MAX > should be OK, rather than 1, 100, no? > 3.A When you're resizing in either L/R or U/D, the layout cells are > either LEFTRIGHT or TOPBOTTOM.A Rather than embed your percentage > check in all of the if statements that check for 'L', 'R', etc., you > can pull this check out to the top, something like: > > if (layout == LAYOUT_TOPBOTTOM) > A A adjust = (wp->sy * percentage) / 100 > else > A A adjust = (wp->sx * percentage) / 100 > > I'm sure you get the idea. > > Kindly, > Thomas > > On Sun, 6 Oct 2019 at 20:56, Anindya Mukherjee <[email protected]> > wrote: > > > > Hi, I have been playing with adding a percent option to resize-pane. > It's on the todo list (small things) for tmux, and I find it very > useful. Right now it's working for me but being new to tmux code I would > love to have some eyes on it. I have attached my changes. > > > > If this is worthwhile/useful then I can do it properly via github. In > that case any advice on the proper procedure is welcome! > > > > Anindya > > > > -- > > You received this message because you are subscribed to the Google > Groups "tmux-users" group. > > To unsubscribe from this group and stop receiving emails from it, send > an email to [email protected]. > > To view this discussion on the web, visit > > https://groups.google.com/d/msgid/tmux-users/CAN%2Bi5iO9yh4TSNJ_r7397%2BCwW55%2BwHqDOs20CBYpFmoXeZfHTA%40mail.gmail.com. -- You received this message because you are subscribed to the Google Groups "tmux-users" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web, visit https://groups.google.com/d/msgid/tmux-users/20191007115719.f452hhjauva57jsi%40yelena.
