envoyproxy/envoy

Clean up PANIC macros

Open

#19,229 创建于 2021年12月9日

在 GitHub 查看
 (12 评论) (0 反应) (0 负责人)C++ (5,373 fork)batch import
beginnerenhancementhelp wantedtech debt

仓库指标

Star
 (27,997 star)
PR 合并指标
 (平均合并 8天) (30 天内合并 378 个 PR)

描述

https://github.com/envoyproxy/envoy/pull/19204 replaced NOT_REACHED_GCOVR_EXCL_LINE macros with more clear PANIC_DUE_TO_CORRUPT_ENUM macros.

While more clear, the panics are still suboptimal becasue doing break; instead of return; results in production crashes. These should be cleaned up and replaced with sensible defaults on a case by case basis.

贡献者指南

研究方向
将每个 PANIC DUE TO CORRUPT ENUM 宏替换为合理的默认返回值，而不是 break，以确保生产稳定性。
技术栈
cpp
领域
backend
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
3-5 天
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
C++Git
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
60