From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: lib/fonts: Read font size with font_data_size() Date: Tue, 03 Mar 2026 13:19:15 +1000 Message-ID: In-Reply-To: <20260302141255.518657-9-tzimmermann@suse.de> References: <20260302141255.518657-1-tzimmermann@suse.de> <20260302141255.518657-9-tzimmermann@suse.de> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Wraps `FNTSIZE()` in a proper function. Straightforward. The function signature takes `font_data_t *fd` (which is `const unsigned char *`) but the macro `FNTSIZE` casts away const via `((int *)(fd))[-2]`. This is fine for reading, but it's worth noting the const-stripping is hidden inside the macro. No functional issues. --- Generated by Claude Code Patch Reviewer