Skip to content

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

Description

@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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions