Skip to content

Conversation

@matsl
Copy link
Collaborator

@matsl matsl commented Nov 7, 2025

What

Silence two byte compilation warnings.

Why

Byte compile should be as clean as possible.

@matsl matsl requested a review from rswgnu November 7, 2025 18:32
(format "*eww-%s*" (url-host (url-generic-parse-url
(eww--dwim-expand-url url)))))))
(eww-mode)
(defvar url-allow-non-local-files) ; Defined from Emacs 29.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix the Changelog comment that references a different variable name.

Copy link
Collaborator Author

@matsl matsl Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch. It is unclear. The reference is to the function where the defvar is placed. But the text does not mention which variable is defvar'ed. I tried to resolve it by adding the name of the variable while keeping the reference to the function. WDYT, OK?

@matsl matsl force-pushed the silence_a_few_minor_warnings branch from 989bd03 to 2fd052e Compare November 7, 2025 23:02
@matsl matsl requested a review from rswgnu November 7, 2025 23:06
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.

3 participants