From b353308712a8288d4288c1666995236acdf68f44 Mon Sep 17 00:00:00 2001 From: xdBronch <51252236+xdBronch@users.noreply.github.com> Date: Sat, 12 Oct 2024 17:12:03 -0400 Subject: [PATCH 1/6] use @returnAddress in panic handler the current code causes the call to panic itself to show up in the stack trace which is noisy --- src/renderer/vaxis/renderer.zig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/renderer/vaxis/renderer.zig b/src/renderer/vaxis/renderer.zig index 01d43e6..f761fc0 100644 --- a/src/renderer/vaxis/renderer.zig +++ b/src/renderer/vaxis/renderer.zig @@ -86,7 +86,7 @@ pub fn panic(msg: []const u8, error_return_trace: ?*std.builtin.StackTrace, ret_ self.vx.deinit(self.allocator, self.tty.anyWriter()); self.tty.deinit(); } - return std.builtin.default_panic(msg, error_return_trace, ret_addr); + return std.builtin.default_panic(msg, error_return_trace, ret_addr orelse @returnAddress()); } pub fn run(self: *Self) !void { From 53e724f26587352f1f4d7c115734788b9d7cb65d Mon Sep 17 00:00:00 2001 From: CJ van den Berg Date: Tue, 11 Feb 2025 18:37:19 +0100 Subject: [PATCH 2/6] fix: spawn at least 4 worker threads This prevents thespian.pid.call() from blocking flow entirely on single CPU systems. --- build.zig.zon | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/build.zig.zon b/build.zig.zon index fd5e775..a414bd6 100644 --- a/build.zig.zon +++ b/build.zig.zon @@ -18,8 +18,8 @@ .hash = "1220220dbc7fe91c1c54438193ca765cebbcb7d58f35cdcaee404a9d2245a42a4362", }, .thespian = .{ - .url = "https://github.com/neurocyte/thespian/archive/db3ad5f45e707a04eaa51aa657995abe43ce967a.tar.gz", - .hash = "1220bbfd147f41fa49d2e5406096f3529c62e9335f4d2a89ae381e679a76ce398f1f", + .url = "https://github.com/neurocyte/thespian/archive/9ca04ddfc715e0f7d29d3f6b39269ad9bf174230.tar.gz", + .hash = "1220b05b5949454bf155a802d5034c060431b8bf59f9d4d2d5241397e9fd201d78d9", }, .themes = .{ .url = "https://github.com/neurocyte/flow-themes/releases/download/master-618a7801d3383049adfe18cc09f5f5086c66995f/flow-themes.tar.gz", From 70a33f1ebfd9205b4ed70961a568aecd7aac25ca Mon Sep 17 00:00:00 2001 From: CJ van den Berg Date: Wed, 12 Feb 2025 10:48:02 +0100 Subject: [PATCH 3/6] feat: add astro file type support closes #175 --- src/syntax/build.zig | 2 ++ src/syntax/build.zig.zon | 4 ++-- src/syntax/src/file_types.zig | 8 ++++++++ 3 files changed, 12 insertions(+), 2 deletions(-) diff --git a/src/syntax/build.zig b/src/syntax/build.zig index c24840e..c74f15f 100644 --- a/src/syntax/build.zig +++ b/src/syntax/build.zig @@ -19,6 +19,7 @@ pub fn build(b: *std.Build) void { .{ .name = "treez", .module = tree_sitter_dep.module("treez") }, ts_queryfile(b, tree_sitter_dep, "queries/cmake/highlights.scm"), ts_queryfile(b, tree_sitter_dep, "tree-sitter-agda/queries/highlights.scm"), + ts_queryfile(b, tree_sitter_dep, "tree-sitter-astro/queries/highlights.scm"), ts_queryfile(b, tree_sitter_dep, "tree-sitter-bash/queries/highlights.scm"), ts_queryfile(b, tree_sitter_dep, "tree-sitter-c-sharp/queries/highlights.scm"), ts_queryfile(b, tree_sitter_dep, "tree-sitter-c/queries/highlights.scm"), @@ -80,6 +81,7 @@ pub fn build(b: *std.Build) void { ts_queryfile(b, tree_sitter_dep, "nvim-treesitter/queries/verilog/highlights.scm"), ts_queryfile(b, tree_sitter_dep, "queries/cmake/injections.scm"), + ts_queryfile(b, tree_sitter_dep, "tree-sitter-astro/queries/injections.scm"), ts_queryfile(b, tree_sitter_dep, "tree-sitter-cpp/queries/injections.scm"), ts_queryfile(b, tree_sitter_dep, "tree-sitter-elixir/queries/injections.scm"), ts_queryfile(b, tree_sitter_dep, "tree-sitter-gitcommit/queries/injections.scm"), diff --git a/src/syntax/build.zig.zon b/src/syntax/build.zig.zon index 46e29c4..0f161c8 100644 --- a/src/syntax/build.zig.zon +++ b/src/syntax/build.zig.zon @@ -4,8 +4,8 @@ .dependencies = .{ .@"tree-sitter" = .{ - .url = "https://github.com/neurocyte/tree-sitter/releases/download/master-69775ce3ba8a5e331bba9feb760d1ba31394eea7/source.tar.gz", - .hash = "1220f9702ca6257f5464b31e576b1e92b0f441bf0e61733c4a2fbf95b7c0c55a3e22", + .url = "https://github.com/neurocyte/tree-sitter/releases/download/master-86dd4d2536f2748c5b4ea0e1e70678039a569aac/source.tar.gz", + .hash = "1220e9fba96c468283129e977767472dee00b16f356e5912431cec8f1a009b6691a2", }, }, .paths = .{ diff --git a/src/syntax/src/file_types.zig b/src/syntax/src/file_types.zig index a634930..f5ad703 100644 --- a/src/syntax/src/file_types.zig +++ b/src/syntax/src/file_types.zig @@ -4,6 +4,14 @@ pub const agda = .{ .comment = "--", }; +pub const astro = .{ + .description = "Astro", + .icon = "", + .extensions = .{"astro"}, + .comment = "//", + .language_server = .{ "astro-ls", "--stdio" }, +}; + pub const bash = .{ .description = "Bash", .color = 0x3e474a, From 9f2e3bf4b48767a7c42bc77e609845a7b285b23c Mon Sep 17 00:00:00 2001 From: CJ van den Berg Date: Wed, 12 Feb 2025 18:09:14 +0100 Subject: [PATCH 4/6] fix: load case data early on startup instead of on demand This is to avoid an issue with the decompressor causing heap corruption on macos. closes #169 --- build.zig | 4 +++- src/buffer/unicode.zig | 10 ++++++++++ src/main.zig | 2 ++ src/tui/editor.zig | 18 ++++-------------- 4 files changed, 19 insertions(+), 15 deletions(-) diff --git a/build.zig b/build.zig index e6af465..99e798e 100644 --- a/build.zig +++ b/build.zig @@ -293,6 +293,7 @@ pub fn build_exe( .imports = &.{ .{ .name = "cbor", .module = cbor_mod }, .{ .name = "thespian", .module = thespian_mod }, + .{ .name = "CaseData", .module = zg_dep.module("CaseData") }, }, }); @@ -476,7 +477,6 @@ pub fn build_exe( .{ .name = "color", .module = color_mod }, .{ .name = "diff", .module = diff_mod }, .{ .name = "help.md", .module = help_mod }, - .{ .name = "CaseData", .module = zg_dep.module("CaseData") }, .{ .name = "fuzzig", .module = fuzzig_dep.module("fuzzig") }, .{ .name = "zeit", .module = zeit_mod }, }, @@ -502,6 +502,7 @@ pub fn build_exe( exe.root_module.addImport("flags", flags_dep.module("flags")); exe.root_module.addImport("cbor", cbor_mod); exe.root_module.addImport("config", config_mod); + exe.root_module.addImport("Buffer", Buffer_mod); exe.root_module.addImport("tui", tui_mod); exe.root_module.addImport("thespian", thespian_mod); exe.root_module.addImport("log", log_mod); @@ -543,6 +544,7 @@ pub fn build_exe( check_exe.root_module.addImport("flags", flags_dep.module("flags")); check_exe.root_module.addImport("cbor", cbor_mod); check_exe.root_module.addImport("config", config_mod); + check_exe.root_module.addImport("Buffer", Buffer_mod); check_exe.root_module.addImport("tui", tui_mod); check_exe.root_module.addImport("thespian", thespian_mod); check_exe.root_module.addImport("log", log_mod); diff --git a/src/buffer/unicode.zig b/src/buffer/unicode.zig index 8d0bb63..60bf389 100644 --- a/src/buffer/unicode.zig +++ b/src/buffer/unicode.zig @@ -59,3 +59,13 @@ pub fn utf8_sanitize(allocator: std.mem.Allocator, input: []const u8) error{ for (input) |byte| try writer.writeAll(try raw_byte_to_utf8(byte, &buf)); return output.toOwnedSlice(allocator); } + +pub const CaseData = @import("CaseData"); +var case_data: ?CaseData = null; +var case_data_arena = std.heap.ArenaAllocator.init(std.heap.page_allocator); + +pub fn get_case_data() *CaseData { + if (case_data) |*cd| return cd; + case_data = CaseData.init(case_data_arena.allocator()) catch @panic("CaseData.init"); + return &case_data.?; +} diff --git a/src/main.zig b/src/main.zig index acf34d4..e7adc38 100644 --- a/src/main.zig +++ b/src/main.zig @@ -41,6 +41,8 @@ pub fn main() anyerror!void { } const a = std.heap.c_allocator; + const case_data = @import("Buffer").unicode.get_case_data(); + _ = case_data; // no need to free case_data as it is globally static const Flags = struct { pub const description = diff --git a/src/tui/editor.zig b/src/tui/editor.zig index 3d4780e..59fa97c 100644 --- a/src/tui/editor.zig +++ b/src/tui/editor.zig @@ -9,7 +9,6 @@ const tracy = @import("tracy"); const text_manip = @import("text_manip"); const syntax = @import("syntax"); const project_manager = @import("project_manager"); -const CaseData = @import("CaseData"); const root_mod = @import("root"); const Plane = @import("renderer").Plane; @@ -350,8 +349,6 @@ pub const Editor = struct { } = null, } = null, - case_data: ?CaseData = null, - const WhitespaceMode = enum { indent, leading, eol, tabs, visible, full, none }; const StyleCache = std.AutoHashMap(u32, ?Widget.Theme.Token); @@ -468,7 +465,6 @@ pub const Editor = struct { self.handlers.deinit(); self.logger.deinit(); if (self.buffer) |p| self.buffer_manager.retire(p, meta.items); - if (self.case_data) |cd| cd.deinit(); } fn from_whitespace_mode(whitespace_mode: []const u8) WhitespaceMode { @@ -492,12 +488,6 @@ pub const Editor = struct { Widget.need_render(); } - fn get_case_data(self: *Self) *CaseData { - if (self.case_data) |*cd| return cd; - self.case_data = CaseData.init(self.allocator) catch @panic("CaseData.init"); - return &self.case_data.?; - } - fn buf_for_update(self: *Self) !*const Buffer { if (!self.pause_undo) { self.cursels_saved.clearAndFree(); @@ -4966,7 +4956,7 @@ pub const Editor = struct { var sfa = std.heap.stackFallback(4096, self.allocator); const cut_text = copy_selection(root, sel.*, sfa.get(), self.metrics) catch return error.Stop; defer allocator.free(cut_text); - const ucased = self.get_case_data().toUpperStr(allocator, cut_text) catch return error.Stop; + const ucased = Buffer.unicode.get_case_data().toUpperStr(allocator, cut_text) catch return error.Stop; defer allocator.free(ucased); root = try self.delete_selection(root, cursel, allocator); root = self.insert(root, cursel, ucased, allocator) catch return error.Stop; @@ -4994,7 +4984,7 @@ pub const Editor = struct { var sfa = std.heap.stackFallback(4096, self.allocator); const cut_text = copy_selection(root, sel.*, sfa.get(), self.metrics) catch return error.Stop; defer allocator.free(cut_text); - const ucased = self.get_case_data().toLowerStr(allocator, cut_text) catch return error.Stop; + const ucased = Buffer.unicode.get_case_data().toLowerStr(allocator, cut_text) catch return error.Stop; defer allocator.free(ucased); root = try self.delete_selection(root, cursel, allocator); root = self.insert(root, cursel, ucased, allocator) catch return error.Stop; @@ -5025,9 +5015,9 @@ pub const Editor = struct { self_: *Self, result: *std.ArrayList(u8), - const Error = @typeInfo(@typeInfo(@TypeOf(CaseData.toUpperStr)).Fn.return_type.?).ErrorUnion.error_set; + const Error = @typeInfo(@typeInfo(@TypeOf(Buffer.unicode.CaseData.toUpperStr)).Fn.return_type.?).ErrorUnion.error_set; pub fn write(writer: *@This(), bytes: []const u8) Error!void { - const cd = writer.self_.get_case_data(); + const cd = Buffer.unicode.get_case_data(); const flipped = if (cd.isLowerStr(bytes)) try cd.toUpperStr(writer.self_.allocator, bytes) else From fb5cd46d0b1fd277d6de3ded0a9d1d99bd73d643 Mon Sep 17 00:00:00 2001 From: CJ van den Berg Date: Wed, 12 Feb 2025 18:17:43 +0100 Subject: [PATCH 5/6] fix: check target missing color module --- build.zig | 1 + 1 file changed, 1 insertion(+) diff --git a/build.zig b/build.zig index 99e798e..b135612 100644 --- a/build.zig +++ b/build.zig @@ -552,6 +552,7 @@ pub fn build_exe( check_exe.root_module.addImport("renderer", renderer_mod); check_exe.root_module.addImport("input", input_mod); check_exe.root_module.addImport("syntax", syntax_mod); + check_exe.root_module.addImport("color", color_mod); check_exe.root_module.addImport("version_info", b.createModule(.{ .root_source_file = version_info_file })); check_step.dependOn(&check_exe.step); From b009d1a23a97a821cca2778db137332b6cd8ba16 Mon Sep 17 00:00:00 2001 From: triangularLuna Date: Wed, 12 Feb 2025 18:13:07 +0100 Subject: [PATCH 6/6] fix: remove 'jk' keybinding in insert mode --- src/keybind/builtin/vim.json | 1 - 1 file changed, 1 deletion(-) diff --git a/src/keybind/builtin/vim.json b/src/keybind/builtin/vim.json index 3291471..26b1a7a 100644 --- a/src/keybind/builtin/vim.json +++ b/src/keybind/builtin/vim.json @@ -102,7 +102,6 @@ "line_numbers": "absolute", "cursor": "beam", "press": [ - ["jk", "enter_mode", "normal"], ["", "enter_mode", "normal"], ["", "delete_forward"], ["", "delete_backward"],