From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: staging: fbtft: replace empty macro args with identity converter Date: Mon, 13 Apr 2026 18:56:20 +1000 Message-ID: In-Reply-To: <20260412172147.2817-2-mzndmzn@gmail.com> References: <20260412172147.2817-2-mzndmzn@gmail.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Overall Series Review Subject: staging: fbtft: replace empty macro args with identity converter Author: Baker Patches: 2 Reviewed: 2026-04-13T18:56:20.957579 --- This mbox contains two standalone `[PATCH]` submissions (not a numbered ser= ies) from the same author, both modifying the same file and the same lines.= The second patch (Message-ID `2817-2`) replies to the first (`2817-1`), su= ggesting it is intended as a revision or improvement. **Both patches are br= oken and will fail to compile**, because they reduce the invocation of a 4-= parameter macro to only 3 arguments without updating the macro definition. = Additionally, the stated premise =E2=80=94 that empty macro arguments are u= ndefined behavior =E2=80=94 is incorrect under C99 and later standards, whi= ch the kernel uses. **Neither patch should be applied as-is.** --- --- Generated by Claude Code Patch Reviewer