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: Remove internal symbols and macros from public header file Date: Tue, 03 Mar 2026 13:19:16 +1000 Message-ID: In-Reply-To: <20260302141255.518657-14-tzimmermann@suse.de> References: <20260302141255.518657-1-tzimmermann@suse.de> <20260302141255.518657-14-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 Creates `lib/fonts/font.h` as an internal header and moves `REFCOUNT`/`FNTSIZE`/`FNTSUM` macros, `struct font_data`, `FONT_EXTRA_WORDS`, and font index defines out of the public header. The built-in font declarations are reorganized alphabetically. This is good cleanup. All font source files now include the local `"font.h"` instead of `` directly (the local header includes the public one). No issues. --- Generated by Claude Code Patch Reviewer