[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#70063: [PATCH] fix warning-suppress for list type "warning type"
From: |
Xuan Wang |
Subject: |
bug#70063: [PATCH] fix warning-suppress for list type "warning type" |
Date: |
Thu, 28 Mar 2024 21:07:34 -0400 |
Tags: patch
Bug report:
In warnings.el, the function warning-suppress is supposed to save the
user's choice into variable warning-suppress-types or
warning-suppress-log-types.
The type of a warning can be a simple symbol, or a list of symbols in
the format of '(warning-type subtype subsubtype).
The change introduced by commit d5ee49c25c8f59ab17c40eebdf38a769c2f5588b
fixes the problem in situations where the warning type is a simple
symbol, but this commit's code saves wrong data if the warning type is a
list.
old behavior:
type warning-suppress-types
pkg -> '((pkg)) Correct
(pkg subtype) -> '(((pkg subtype))) Incorrect
Refer to the implementation of warning-suppress-p, an element in
warning-suppress-types must be a list of symbols, must not be a simple
symbol, and must not be a list of lists (if this is the case, nothing is
going to be matched).
How to reproduce the bug:
Use any script that can emit a warning type that is a list. Press the
button on the warning buffer to suppress the warning. Run the script and
emit the warning again; you will see the warning buffer pop up, so the
warning is not suppressed.
How this patch fix the bug:
Check if the "warning type" is a list using "consp" first. If the
warning type is a list, add it directly to warning-suppress-types.
Otherwise, use the old behavior, wrap it in a list, then add it to
warning-suppress-types.
new behavior:
type warning-suppress-types
pkg -> '((pkg)) Correct
(pkg subtype) -> '((pkg subtype)) Correct
0001-fix-warning-suppress-for-list-type-warning-type.patch
Description: Text Data
- bug#70063: [PATCH] fix warning-suppress for list type "warning type",
Xuan Wang <=