Is the implementation of jinx--mode-list really correct?

[WeissP]

I suddenly noticed that no misspellings were being detected. After spending some time digging into it and debugging, the if condition in jinx--mode-list doesn't seem quite right to me. It looks like it could just return the first mode that satisfies (and (boundp mode) (symbol-value mode)), even if that isn't actually the major mode or minor mode we want:

if (or (and (boundp mode) (symbol-value mode)) (derived-mode-p mode))

Sorry if I've misunderstood something.

[minad]

Yes, it is correct. Check the entire cl-loop.