Zsh Mailing List Archive
Messages sorted by: Reverse Date, Date, Thread, Author

Re: [RFC PATCH v3] vcs_info: choose backend by basedir



On Wed, Nov 4, 2020 at 12:38 PM Daniel Shahaf <d.s@xxxxxxxxxxxxxxxxxx> wrote:
> About extensibility, needs for extensibility aren't always foreseen.

I guess they can't be foreseen both ways, i.e. extensibility may be
needed where you didn't expect, but also maybe extensibility won't be
needed where you expected and designed. But creating a "more
extensible" interface takes more time, and this way you usually have
more complex code. So maybe this "premature extensibility" isn't worth
it?

>
> As to their own boolean styles, I'm not sure I agree.  zstyle values are
> arrays, so we could let parameters be provided in additional words, as
> in:
> .
>     zstyle $pattern $style enablement-order foo=bar
>     zstyle $pattern $style closest-.vcs-dir lorem=ipsum
>
> Hmm.  Perhaps it would make more sense to name the style "disk-layout",
> and then the extra parameters could be used to choose whether to select
> the closest or farthest .${vcs} dir.
>
> Or perhaps an enum really is overkill, as you say.

After your reply, for me it now seems to be even more overkill than before




Messages sorted by: Reverse Date, Date, Thread, Author