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: hoist classmap-filter-by-modname up to ddebug_add_module Date: Sat, 14 Mar 2026 07:00:24 +1000 Message-ID: In-Reply-To: <20260313132103.2529746-18-jim.cromie@gmail.com> References: <20260313132103.2529746-1-jim.cromie@gmail.com> <20260313132103.2529746-18-jim.cromie@gmail.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Introduces `dd_mark_vector_subrange` statement-expression macro. **The macro is complex** (nested for-loop with break, local variables, references to caller scope). Hard to audit. Works correctly but readability is a concern. --- Generated by Claude Code Patch Reviewer