LibGfx: Use size_t instead of int for glyph count

The count is always non-negative
abyesilyurt · Apr 18, 2021 · 3c894d1 · 3c894d1
1 parent c1971df
commit 3c894d1
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/Userland/Libraries/LibGfx/BitmapFont.h b/Userland/Libraries/LibGfx/BitmapFont.h
@@ -107,7 +107,7 @@ class BitmapFont : public Font {
  m_glyph_widths[ch] = width;
  }
 
- int glyph_count() const { return m_glyph_count; }
+ size_t glyph_count() const { return m_glyph_count; }
 
  FontTypes type() { return m_type; }
  void set_type(FontTypes type);

diff --git a/Userland/Libraries/LibGfx/Font.h b/Userland/Libraries/LibGfx/Font.h
@@ -134,7 +134,7 @@ class Font : public RefCounted<Font> {
 
  virtual u8 glyph_spacing() const = 0;
 
- virtual int glyph_count() const = 0;
+ virtual size_t glyph_count() const = 0;
 
  virtual String family() const = 0;
  virtual String variant() const = 0;

diff --git a/Userland/Libraries/LibTTF/Font.h b/Userland/Libraries/LibTTF/Font.h
@@ -156,7 +156,7 @@ class ScaledFont : public Gfx::Font {
  virtual String name() const override { return String::formatted("{} {}", family(), variant()); }
  virtual bool is_fixed_width() const override { return m_font->is_fixed_width(); }
  virtual u8 glyph_spacing() const override { return m_x_scale; } // FIXME: Read from font
- virtual int glyph_count() const override { return m_font->glyph_count(); }
+ virtual size_t glyph_count() const override { return m_font->glyph_count(); }
  virtual String family() const override { return m_font->family(); }
  virtual String variant() const override { return m_font->variant(); }
  virtual String qualified_name() const override { return String::formatted("{} {} {}", family(), presentation_size(), weight()); }