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, 10 Mar 2026 12:19:45 +1000 Message-ID: In-Reply-To: <20260309141723.137364-14-tzimmermann@suse.de> References: <20260309141723.137364-1-tzimmermann@suse.de> <20260309141723.137364-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 Clean encapsulation. Creates `lib/fonts/font.h` as an internal header for font data structures and macros, and sorts the built-in font declarations alphabetically in the public header. The REFCOUNT/FNTSIZE/FNTSUM macros move to `fonts.c` where they're only used. `struct font_data` and the index defines move to the internal `lib/fonts/font.h`. All font source files switch from `` to `"font.h"`. Good final cleanup patch. No issues. --- Generated by Claude Code Patch Reviewer