On Thu, Nov 08, 2012 at 04:33:37AM +0000, Ben Boeckel wrote:
> On Wed, Nov 07, 2012 at 15:13:21 GMT, Thomas Adam wrote:
> >> $optflag_a && nwopts="-a -t $winid $nwopts"
> >
> > It is perhaps preferrable to not treat optflag_a as some literal boolean,
> > but rather use something like:
> >
> > [ -n "$optflag_a" ] && nwopts="-a -t $winid $nwopts"
> 
> I tend to use `true` and `false` for booleans. It's an exec either way.

[ is typically a shell builtin.

> It also allows for some option to determing the `-a` argument in some
> more complex way.
> 
> >> if target=$(echo $pmtable | tr : "\n" | \
> >>         grep -E "(=|^)$paneid(=|$)" | head -n 1 | grep .) ; then
> >
> >
> > Ugh.  :)  What is this trying to do?  (I know what it's doing, it's a
> > rhetorical question).  At this point, the "if" condition is wholly
> > incorrect.  I suggest this:
> 
> It seems to be trying to count matches. The `-q` and `-c` flags should
> be used instead.
> 
> -- Ben
> 
> 
> ------------------------------------------------------------------------------
> Everyone hates slow websites. So do we.
> Make your web apps faster with AppDynamics
> Download AppDynamics Lite for free today:
> http://p.sf.net/sfu/appdyn_d2d_nov
> _______________________________________________
> tmux-users mailing list
> tmux-users@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/tmux-users

------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_nov
_______________________________________________
tmux-users mailing list
tmux-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tmux-users

Reply via email to