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