[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#52383: 29.0.50; [suggestion] Add seq-set-subset-p to seq.el
From: |
Marco Wahl |
Subject: |
bug#52383: 29.0.50; [suggestion] Add seq-set-subset-p to seq.el |
Date: |
Thu, 09 Dec 2021 21:21:14 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
Juri Linkov <juri@linkov.net> writes:
>> BTW function `seq-set-equal-p' could be rewritten using
>> `seq-set-subset-p'.
>
> Isn't the word "set" redundant in the name?
> In `seq-set-equal-p' the word "set" is needed
> to say "is set equal". But in `seq-set-subset-p'
> the word "set" doesn't add much meaning, so better
> would be to name your new function just `seq-subset-p'.
Agreed.
The motivation for the suggested name `seq-set-subset-p' was to stay
close to the name `seq-set-equal-p'. A user could interpret the prefix
"seq-set" as hint that the function implements a set operation.
Thanks!