zsh-workers
 help / color / mirror / code / Atom feed
From: Bart Schaefer <schaefer@brasslantern.com>
To: zsh-workers@zsh.org
Subject: Re: min() max() math functions (was: Re: Feature request (@M):# with context matches)
Date: Wed, 10 Feb 2016 09:41:34 -0800	[thread overview]
Message-ID: <160210094134.ZM2246@torch.brasslantern.com> (raw)
In-Reply-To: <20160210092324.GA6339@tarsus.local2>

On Feb 10,  9:23am, Daniel Shahaf wrote:
} Subject: Re: min() max() math functions (was: Re: Feature request (@M):# w
}
} Bart Schaefer wrote on Mon, Feb 08, 2016 at 21:22:22 -0800:
} > 
} >     emulate zsh -c 'autoload -U zmathfunc'
} 
} Shouldn't this form be used more often?  As in, wouldn't using this form
} be correct for the majority of autoloadable functions?

It has other side-effects which are desirable in the zmathfunc case but
may not be in other cases.  (Specifically, the stickyness is transitive
to functions defined when the autoloaded function runs.)

Whether "the majority" of autoloads would benefit, I don't know.  Some
functions perform a task in a very isolated way, others are intended
to preserve the behavior the user's setopts have defined, and it's not
possible to readily determine which have the latter intent.

} > Although it seems an oversight that you can't combine -u and -M for the
} > functions command.
} 
} Would this be easy to add?

I took a quick look and decided it wasn't straightforward enough to mess
with today.  The different interpretation of the argument list for -M
makes it complicated to untangle the current implementation.  And one
can get the same effect by doing 

    autoload mathfunc
    functions -M mf 0 0 mathfunc

in either order (that is, the shell function doesn't have to be defined
to link it to a math context name).

} (Just another flag in the stub function structure, maybe?)

Doesn't even need that.

} > Ehh, I'm not sure it's the job of any module's doc to explain what it
} > does NOT define, much less to cross-reference zshcontrib.
} 
} Why is it a problem for zshmodules to reference zshcontrib?  Should
} zmathfunc's documentation be located elsewhere?

It's not exactly a problem, more like a bad precedent.  E.g. I was careful
to have zshcontrib for the *-word-match functions refer to the vim text
objects, but I would not reference in the other direction.  It should IMO
be possible to throw away the zshcontrib manual and still have a complete
reference to the "bare" shell.

The most obvious counter-argument is that the entire completion system
depends on mutually cooperating options, builtins, and shell functions.
And the complist module already references the colors autoload, so I
should probably just shrug and ignore my sense of propriety.


  reply	other threads:[~2016-02-10 17:41 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CAKc7PVDCOPzbU25+vTAXMQjonS1sznDX_SBq4-8rUZPQewa5Tg@mail.gmail.com>
     [not found] ` <CAKc7PVAe0N2bp2XS5U4mOKnTgxQRsj4UGy7NQpHput9VsbnAJw@mail.gmail.com>
     [not found]   ` <160130085456.ZM9730__49922.0612728552$1454172936$gmane$org@torch.brasslantern.com>
     [not found]     ` <20160207002212.GC24068@tarsus.local2>
     [not found]       ` <160206170040.ZM1927__13399.3204137825$1454806909$gmane$org@torch.brasslantern.com>
     [not found]         ` <20160209032308.GA20947@tarsus.local2>
2016-02-09  5:22           ` Bart Schaefer
2016-02-10  9:23             ` Daniel Shahaf
2016-02-10 17:41               ` Bart Schaefer [this message]
2016-02-14 14:33                 ` Daniel Shahaf

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=160210094134.ZM2246@torch.brasslantern.com \
    --to=schaefer@brasslantern.com \
    --cc=zsh-workers@zsh.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).