mirrors/ghostty
1
0
Fork 0
mirror of https://github.com/ghostty-org/ghostty.git synced 2026-08-25 00:21:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

surface: keep clipboard content list on stack

Browse Source 
Use a two element stack-based buffer for the one or two ClipboardContent
entries rather than the (arena-based) heap allocator.
This commit is contained in:
Jon Parise
2026-08-19 17:02:45 -04:00
parent 3e7230bf5d
commit 5a8921ecb5
1 changed files with 7 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/Surface.zig
View File

@@ -2257,13 +2257,14 @@ fn copySelectionToClipboards(
const ScreenFormatter = terminal.formatter.ScreenFormatter;
var aw: std.Io.Writer.Allocating = .init(alloc);
var contents: std.ArrayList(apprt.ClipboardContent) = .empty;
var contents_buf: [2]apprt.ClipboardContent = undefined;
var contents: std.ArrayList(apprt.ClipboardContent) = .initBuffer(&contents_buf);
switch (format) {
.plain => {
var formatter: ScreenFormatter = .init(self.io.terminal.screens.active, opts);
formatter.content = .{ .selection = sel };
try formatter.format(&aw.writer);
try contents.append(alloc, .{
contents.appendAssumeCapacity(.{
.mime = "text/plain",
.data = try aw.toOwnedSliceSentinel(0),
});
@@ -2280,7 +2281,7 @@ fn copySelectionToClipboards(
// Note: We don't apply codepoint mappings to VT format since it contains
// escape sequences that should be preserved as-is
try contents.append(alloc, .{
contents.appendAssumeCapacity(.{
.mime = "text/plain",
.data = try aw.toOwnedSliceSentinel(0),
});
@@ -2297,7 +2298,7 @@ fn copySelectionToClipboards(
// Note: We don't apply codepoint mappings to HTML format since HTML
// has its own character encoding and entity system
try contents.append(alloc, .{
contents.appendAssumeCapacity(.{
.mime = "text/html",
.data = try aw.toOwnedSliceSentinel(0),
});
@@ -2308,7 +2309,7 @@ fn copySelectionToClipboards(
var formatter: ScreenFormatter = .init(self.io.terminal.screens.active, opts);
formatter.content = .{ .selection = sel };
try formatter.format(&aw.writer);
try contents.append(alloc, .{
contents.appendAssumeCapacity(.{
.mime = "text/plain",
.data = try aw.toOwnedSliceSentinel(0),
});
@@ -2331,7 +2332,7 @@ fn copySelectionToClipboards(
try formatter.format(&aw.writer);
// Note: We don't apply codepoint mappings to HTML format
try contents.append(alloc, .{
contents.appendAssumeCapacity(.{
.mime = "text/html",
.data = try aw.toOwnedSliceSentinel(0),
});