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: split multi-query strings with % Date: Sat, 14 Mar 2026 07:00:26 +1000 Message-ID: In-Reply-To: <20260313132103.2529746-29-jim.cromie@gmail.com> References: <20260313132103.2529746-1-jim.cromie@gmail.com> <20260313132103.2529746-29-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 **Design concern:** Using `%` as a multi-query separator breaks format stri= ng matching. The commit message acknowledges this but downplays it =E2=80= =94 patterns like `format "error: %s"` are legitimate real-world queries. A= n alternative separator that doesn't conflict with printf format specifiers= should be considered. --- Generated by Claude Code Patch Reviewer