Applied-Software/ghostty
5
0
Fork 0
mirror of https://github.com/ghostty-org/ghostty.git synced 2025-07-14 15:56:13 +03:00
Code Issues Packages Projects Releases Wiki Activity

address many fontmem todos

Browse Source
This commit is contained in:
Mitchell Hashimoto
2024-04-06 20:00:22 -07:00
parent 3b0b9c2cfd
commit e3402cef4d
4 changed files with 4 additions and 36 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/font/SharedGrid.zig
View File

@ -18,9 +18,6 @@
//! reinitialized and all surfaces should switch over to using that one.
const SharedGrid = @This();
// TODO(fontmem):
// - consider config changes and how they affect the shared grid.
const std = @import("std");
const assert = std.debug.assert;
const Allocator = std.mem.Allocator;

10
src/font/face/Metrics.zig
View File

@ -174,12 +174,10 @@ pub const Modifier = union(enum) {
const autoHash = std.hash.autoHash;
autoHash(hasher, std.meta.activeTag(self));
switch (self) {
// floats can't be hashed directly so we round it to the
// nearest int and then hash that. This is not perfect but
// hash collisions due to the modifier being wrong are really
// rare so we should fix this up later.
// TODO(fontmem): make better
.percent => |v| autoHash(hasher, @as(i64, @intFromFloat(v))),
// floats can't be hashed directly so we bitcast to i64.
// for the purpose of what we're trying to do this seems
// good enough but I would prefer value hashing.
.percent => |v| autoHash(hasher, @as(i64, @bitCast(v))),
.absolute => |v| autoHash(hasher, v),
}
}

15
src/renderer/Metal.zig
View File

@ -1363,21 +1363,6 @@ fn prepKittyGraphics(
/// Update the configuration.
pub fn changeConfig(self: *Metal, config: *DerivedConfig) !void {
// On configuration change we always reset our font group. There
// are a variety of configurations that can change font settings
// so to be safe we just always reset it. This has a performance hit
// when its not necessary but config reloading shouldn't be so
// common to cause a problem.
//
// TODO(fontmem): we no longer do this. the surface should handle
// font changes, create a new grid for us, and send it via message
// passing or something.
//
// self.font_group.reset();
// self.font_group.group.styles = config.font_styles;
// self.font_group.atlas_greyscale.clear();
// self.font_group.atlas_color.clear();
// We always redo the font shaper in case font features changed. We
// could check to see if there was an actual config change but this is
// easier and rare enough to not cause performance issues.

12
src/renderer/OpenGL.zig
View File

@ -1530,18 +1530,6 @@ fn gridSize(self: *const OpenGL, screen_size: renderer.ScreenSize) renderer.Grid
/// Update the configuration.
pub fn changeConfig(self: *OpenGL, config: *DerivedConfig) !void {
// On configuration change we always reset our font group. There
// are a variety of configurations that can change font settings
// so to be safe we just always reset it. This has a performance hit
// when its not necessary but config reloading shouldn't be so
// common to cause a problem.
//
// TODO(fontmem): see Metal
// self.font_group.reset();
// self.font_group.group.styles = config.font_styles;
// self.font_group.atlas_greyscale.clear();
// self.font_group.atlas_color.clear();
// We always redo the font shaper in case font features changed. We
// could check to see if there was an actual config change but this is
// easier and rare enough to not cause performance issues.