It looks right to me; except that you may want to move the xfree(label)
back down where it was before (with a NULL check on label), as otherwise
it'll leak memory if some silly user specifies both -S _and_ -L.
-mjc
Nicholas Marriott wrote:
> How about this instead? I think it should have the same effect.
>
> Index: tmux.c
> ===================================================================
> RCS file: /cvs/src/usr.bin/tmux/tmux.c,v
> retrieving revision 1.66
> diff -u -p -r1.66 tmux.c
> --- tmux.c 3 Jan 2010 12:51:05 -0000 1.66
> +++ tmux.c 3 Feb 2010 21:55:27 -0000
> @@ -442,13 +442,29 @@ main(int argc, char **argv)
> }
> }
>
> - if (label == NULL)
> - label = xstrdup("default");
> - if (path == NULL && (path = makesockpath(label)) == NULL) {
> - log_warn("can't create socket");
> - exit(1);
> + /*
> + * Figure out the socket path. If specified on the command-line with
> + * -S or -L, use it, otherwise try $TMUX or assume -L default.
> + */
> + if (path == NULL) {
> + /* No -L. Try $TMUX, or default. */
> + if (label == NULL) {
> + if ((path = getenv("TMUX")) != NULL) {
> + path = xstrdup(path);
> + path[strcspn(path, ",")] = '\0';
> + } else
> + label = xstrdup("default");
> + }
> +
> + /* -L or default set. */
> + if (label != NULL) {
> + if ((path = makesockpath(label)) == NULL) {
> + log_warn("can't create socket");
> + exit(1);
> + }
> + xfree(label);
> + }
> }
> - xfree(label);
>
> if (shellcmd != NULL) {
> msg = MSG_SHELL;
>
>
> On Wed, Feb 03, 2010 at 08:29:00AM -0800, Micah Cowan wrote:
>> Understood. I wasn't sure you would; I just dislike duplicating the same
>>
>> if ((path = makesockpath(...)) == NULL) {
>> give_an_error_msg();
>> exit(1);
>> }
>>
>> twice in a row (once in one branch, and again in a branch of an outer
>> if, IIRC - wherever the xmakesockpath calls are now). And avoiding the
>> double with either an extra flag var (which I know you're not crazy
>> about) or a goto looked even uglier to me. Maybe you'll find a solution
>> you like better.
>>
>> -mjc
>>
>> Nicholas Marriott wrote:
>>> Nice idea, will have a better look tonight but I'm not sure I like
>>> the idea of makesockpath calling exit().
------------------------------------------------------------------------------
The Planet: dedicated and managed hosting, cloud storage, colocation
Stay online with enterprise data centers and the best network in the business
Choose flexible plans and management services without long-term contracts
Personal 24x7 support from experience hosting pros just a phone call away.
http://p.sf.net/sfu/theplanet-com
_______________________________________________
tmux-users mailing list
tmux-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tmux-users
