rust-lang/rust-clippy

transmute_ptr_to_ref suggestion doesn't with some outer expression types

Open

#1,754 创建于 2017年5月12日

在 GitHub 查看
 (1 评论) (0 反应) (1 负责人)Rust (1,391 fork)batch import
C-bugT-middlegood first issue

仓库指标

Star
 (10,406 star)
PR 合并指标
 (平均合并 19天 22小时) (30 天内合并 113 个 PR)

描述

clippy suggests the following replacement

-                mem::transmute::<*mut c_void, &mut Box<FnMut(&Button)>>(data)(&button)
+                &mut *(data as *mut Box<FnMut(&Button)>)(&button)

but we should suggest parens around the &mut *...

bonus points if the parens only appear if necessary

贡献者指南

研究方向
该问题描述了一个 clippy lint `transmute ptr to ref`，它错误地建议将 `mem::transmute::<*mut c void, &mut Box<FnMut(&Button)>>(data)(&button)` 替换为 `&mut *(data as *mut Box<FnMut(&Button)>)(&button)` 而不加括号。修复涉及修改 clippy 仓库中该 lint 的建议代码（可能在 `clippy lints/src/transmute.rs` 中），当 `&mut *...` 出现在函数调用等上下文中时，将其用括号包裹。观察发现，不加括号的建议在语法上是模糊的。需要找到 `transmute ptr to ref` lint 的实现，检查建议是如何生成的（可能使用 `Applicability` 和 `span lint and sugg`），并添加逻辑来检测表达式是否被用作被调用者（例如，在函数调用内部），然后为建议的代码添加括号。使用提供的示例和类似案例测试更改。
技术栈
rust
领域
developer experiencetooling
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
基本清晰
前置要求
Rustclippy internal architecture
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
75