mirror of
https://github.com/astral-sh/ruff.git
synced 2025-10-03 23:25:14 +00:00
Avoid removing parentheses on ctypes.WinError (#9027)
Re-resolves https://github.com/astral-sh/ruff/issues/6730.
This commit is contained in:
parent
ef7778d794
commit
b918647927
3 changed files with 17 additions and 3 deletions
|
@ -82,3 +82,14 @@ raise IndexError();
|
||||||
|
|
||||||
# RSE102
|
# RSE102
|
||||||
raise Foo()
|
raise Foo()
|
||||||
|
|
||||||
|
# OK
|
||||||
|
raise ctypes.WinError()
|
||||||
|
|
||||||
|
|
||||||
|
def func():
|
||||||
|
pass
|
||||||
|
|
||||||
|
|
||||||
|
# OK
|
||||||
|
raise func()
|
||||||
|
|
|
@ -78,9 +78,7 @@ pub(crate) fn unnecessary_paren_on_raise_exception(checker: &mut Checker, expr:
|
||||||
|
|
||||||
// `ctypes.WinError()` is a function, not a class. It's part of the standard library, so
|
// `ctypes.WinError()` is a function, not a class. It's part of the standard library, so
|
||||||
// we might as well get it right.
|
// we might as well get it right.
|
||||||
if exception_type
|
if exception_type.is_none()
|
||||||
.as_ref()
|
|
||||||
.is_some_and(ExceptionType::is_builtin)
|
|
||||||
&& checker
|
&& checker
|
||||||
.semantic()
|
.semantic()
|
||||||
.resolve_call_path(func)
|
.resolve_call_path(func)
|
||||||
|
|
|
@ -266,6 +266,8 @@ RSE102.py:84:10: RSE102 [*] Unnecessary parentheses on raised exception
|
||||||
83 | # RSE102
|
83 | # RSE102
|
||||||
84 | raise Foo()
|
84 | raise Foo()
|
||||||
| ^^ RSE102
|
| ^^ RSE102
|
||||||
|
85 |
|
||||||
|
86 | # OK
|
||||||
|
|
|
|
||||||
= help: Remove unnecessary parentheses
|
= help: Remove unnecessary parentheses
|
||||||
|
|
||||||
|
@ -275,5 +277,8 @@ RSE102.py:84:10: RSE102 [*] Unnecessary parentheses on raised exception
|
||||||
83 83 | # RSE102
|
83 83 | # RSE102
|
||||||
84 |-raise Foo()
|
84 |-raise Foo()
|
||||||
84 |+raise Foo
|
84 |+raise Foo
|
||||||
|
85 85 |
|
||||||
|
86 86 | # OK
|
||||||
|
87 87 | raise ctypes.WinError()
|
||||||
|
|
||||||
|
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue