Claude review: lib/fonts: Compare font data for equality with font_data_is_equal()

[Claude Code Review Bot <claude-review@example.com>]

Patch Review

The comment has a typo: "is constain" should be "it contains" and "also use" should be "also uses".

The function body:
```c
+	if (font_data_is_internal(lhs) != font_data_is_internal(rhs))
+		return false;
```
Good — prevents matching internal fonts to user-space copies, which would break refcounting.

However, removing the width check from fbcon:
```c
-		    tmp->vc_font.width == w &&
```
The commit message says "The test only compares data buffers. Their interpretation is up each console." But two fonts with different widths could have identical byte sequences yet represent completely different glyphs (different pitch/scanline layout). The `font_data_is_equal` check relies on same-size data implying same interpretation, but `size = height * pitch * charcount` — two fonts could have `(height=16, width=8, pitch=1, charcount=256)` vs `(height=8, width=16, pitch=2, charcount=128)`, both giving `size=4096` with potentially identical bytes but completely different visual results. This seems like a correctness concern.

---
Generated by Claude Code Patch Reviewer