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

Merge pull request #2843 from ghostty-org/push-lqkuvlyttoul

Browse Source 
config: need to dupe filepath for diagnostics
This commit is contained in:
Mitchell Hashimoto
2024-11-27 08:51:51 -08:00
committed by GitHub
parent f55b70912d 5b01cb353d
commit 423f58c24c
3 changed files with 11 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/cli/args.zig
View File

@ -104,7 +104,7 @@ pub fn parse(
try dst._diagnostics.append(arena_alloc, .{
.key = try arena_alloc.dupeZ(u8, arg),
.message = "invalid field",
.location = diags.Location.fromIter(iter),
.location = try diags.Location.fromIter(iter, arena_alloc),
});
continue;
@ -145,7 +145,7 @@ pub fn parse(
try dst._diagnostics.append(arena_alloc, .{
.key = try arena_alloc.dupeZ(u8, key),
.message = message,
.location = diags.Location.fromIter(iter),
.location = try diags.Location.fromIter(iter, arena_alloc),
});
};
}
@ -1140,7 +1140,7 @@ pub fn ArgsIterator(comptime Iterator: type) type {
}
/// Returns a location for a diagnostic message.
pub fn location(self: *const Self) ?diags.Location {
pub fn location(self: *const Self, _: Allocator) error{}!?diags.Location {
return .{ .cli = self.index };
}
};
@ -1262,12 +1262,15 @@ pub fn LineIterator(comptime ReaderType: type) type {
}
/// Returns a location for a diagnostic message.
pub fn location(self: *const Self) ?diags.Location {
pub fn location(
self: *const Self,
alloc: Allocator,
) Allocator.Error!?diags.Location {
// If we have no filepath then we have no location.
if (self.filepath.len == 0) return null;
return .{ .file = .{
.path = self.filepath,
.path = try alloc.dupe(u8, self.filepath),
.line = self.line,
} };
}

4
src/cli/diagnostics.zig
View File

@ -56,7 +56,7 @@ pub const Location = union(enum) {
pub const Key = @typeInfo(Location).Union.tag_type.?;
pub fn fromIter(iter: anytype) Location {
pub fn fromIter(iter: anytype, alloc: Allocator) Allocator.Error!Location {
const Iter = t: {
const T = @TypeOf(iter);
break :t switch (@typeInfo(T)) {
@ -67,7 +67,7 @@ pub const Location = union(enum) {
};
if (!@hasDecl(Iter, "location")) return .none;
return iter.location() orelse .none;
return (try iter.location(alloc)) orelse .none;
}
pub fn clone(self: *const Location, alloc: Allocator) Allocator.Error!Location {

2
src/config/Config.zig
View File

@ -2992,7 +2992,7 @@ pub fn parseManuallyHook(
if (command.items.len == 0) {
try self._diagnostics.append(alloc, .{
.location = cli.Location.fromIter(iter),
.location = try cli.Location.fromIter(iter, alloc),
.message = try std.fmt.allocPrintZ(
alloc,
"missing command after {s}",