Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix incorrect face usage and undeclared list elements type #1872

Merged
merged 2 commits into from
Mar 5, 2025

Conversation

Hi-Angel
Copy link
Contributor

@Hi-Angel Hi-Angel commented Mar 4, 2025

This fixes a few warnings. Couldn't fix more because it complains if-let being an obsolete macro and that if-let* should be preferred, but I'm not sure what's the best course of action here. if-let* appeared since 26.1, so it's either upping the minimal version 26.1 or adding a wrapper for older Emacses. Emacs 26.1 was released in 2018.

Hi-Angel added 2 commits March 5, 2025 01:47
A face is represented by a symbol, however representing a symbol
inside lists requires no quoting. So remove the quote.

Fixes warning:

    haskell-c2hs.el:37:19: Error: in defface for ‘haskell-c2hs-hook-pair-face’: Value for face attribute ‘:inherit’ should not be quoted
Fixes:
    haskell-commands.el:49:10: Error: in defcustom for ‘haskell-mode-stylish-haskell-args’: ‘list’ without arguments
@purcell
Copy link
Member

purcell commented Mar 5, 2025

Yes, if-let has had a complicated history. An option is requiring the compat package, but I'd prefer to avoid pulling in external dependencies for something trivial like this. Having a separate let and if/when is perfectly fine, so rewriting to that might be the best option.

@purcell purcell merged commit 6a533c5 into haskell:master Mar 5, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants