diasurgical/DevilutionX

Replace "int x, int y" arguments with "Point point"

开放

#1,684 创建于 2021年4月25日

在 GitHub 查看贡献者指南
 (2 条评论) (1 个反应) (0 位负责人)C++ (743 个派生)batch import
good first issuehelp wanted

仓库指标

星标
 (7,255 个星标)
PR 合并指标
 (平均合并 2天 12小时) (30 天内合并 15 个 PR)

描述

A lot of functions take two ints as the coordinates for their operations, this is both verbose and error-prone. Instead we should use the Point structure, see an example of this type of cleanup done for the GetDirection() https://github.com/diasurgical/devilutionX/commit/f77c52941afd73d4e734c0696d082218cd04c3a7

Try to split the changes up in one function per commit to making it easy to review and less prone to typos.

贡献者指南

研究方向
研究示例提交（f77c529）以理解将 `int x, int y` 替换为 `Point` 的模式。在代码库中搜索具有两个 int 坐标参数的函数，例如几何或渲染模块中的函数。将每个参数对替换为 `Point` 引用，更新所有调用点，并确保与现有 `Point` 用法保持一致。按照要求为每个函数提交更改，以便于审查。
技术栈
cpp
领域
backend
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-2 天
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
GitC++
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
60