From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: dyndbg-API: replace DECLARE_DYNDBG_CLASSMAP Date: Sat, 14 Mar 2026 07:00:25 +1000 Message-ID: In-Reply-To: <20260313132103.2529746-23-jim.cromie@gmail.com> References: <20260313132103.2529746-1-jim.cromie@gmail.com> <20260313132103.2529746-23-jim.cromie@gmail.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review The architectural heart of the series =E2=80=94 introduces `DYNAMIC_DEBUG_C= LASSMAP_DEFINE` and `DYNAMIC_DEBUG_CLASSMAP_USE`. **Bug:** `test_dynamic_de= bug.c` has duplicate `MODULE_DESCRIPTION()` lines, which will generate a bu= ild warning. Dead `return 0;` after `default: return -1;` in `param_get_dyn= dbg_classes` =E2=80=94 harmless but indicates confusion. --- Generated by Claude Code Patch Reviewer