From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: vt: Calculate font-buffer size with vc_font_size() Date: Tue, 10 Mar 2026 12:19:43 +1000 Message-ID: In-Reply-To: <20260309141723.137364-5-tzimmermann@suse.de> References: <20260309141723.137364-1-tzimmermann@suse.de> <20260309141723.137364-5-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 Introduces `vc_font_pitch()` and `vc_font_size()` inline helpers. Clean. The change from `int pitch` with a `pitch <= 0` check to `unsigned int size` with a `!size` check is semantically equivalent since `DIV_ROUND_UP` on unsigned values can never produce zero for non-zero width, and the original pitch check was really guarding against width==0. Fine. --- Generated by Claude Code Patch Reviewer