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, 10 Mar 2026 12:19:44 +1000 Message-ID: In-Reply-To: <20260309141723.137364-9-tzimmermann@suse.de> References: <20260309141723.137364-1-tzimmermann@suse.de> <20260309141723.137364-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 function. Clean encapsulation. The return type is `unsigned int` but `FNTSIZE()` dereferences an `int *`. A negative value in the FNTSIZE field would be returned as a large unsigned value. This shouldn't happen in practice, but a `WARN_ON` for negative values might be defensive. --- Generated by Claude Code Patch Reviewer