Re: Emacs 30.0 warning from `cl-pushnew' and `memql'

[Emanuel Berg]

Michael Heerdegen wrote:

>> After the last powerup I see this, it refers to a use of
>> `cl-pushnew'.
>>
>>   Warning: `memql' called with literal string that may never match
>>   (arg 1)
>
> Yes, these new warnings hint at problematic calls. In case of
> cl-pushnew you want to specify a :test function to avoid comparing
> strings with an inappropriate test function like the
> default memql.

This does not render the warning

  (cl-pushnew "/usr/share/emacs/site-lisp/elpa-src/slime-2.26.1"
              load-path
              :test #'string=)

but this is not a model I can endorse, this is why you have that
level of abstraction to begin with, i.e., not to be bothered all the
time with implementation details.

Might as well to provide :loop #'cl-dolist how to iterate the
list as well ...

And after that ... is there anything else to specify before we can
get rid of `cl-pushnew' altogether since it amounts to doing
everything manually anyway?

-- 
underground experts united
https://dataswamp.org/~incal