a/ziglings
1
0
Fork 0
mirror of https://codeberg.org/andyscott/ziglings.git synced 2024-12-22 14:03:10 -05:00
Code Issues Projects Releases Packages Wiki Activity

build: make PrintStep thread safe

Browse source 
Update PrintStep to always printing to stderr, using std.debug.print, so
that the message is written atomically.

Note that currently it is not an issue, since PrintStep prints the
message before everything else.
This commit is contained in:
Manlio Perillo 2023-04-10 21:24:49 +02:00
parent 1ee8aabcf3
commit 47876e6371
1 changed files with 4 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
build.zig
View file

@ -557,7 +557,7 @@ pub fn build(b: *Build) !void {
const use_healed = b.option(bool, "healed", "Run exercises from patches/healed") orelse false; const use_healed = b.option(bool, "healed", "Run exercises from patches/healed") orelse false;
const exno: ?usize = b.option(usize, "n", "Select exercise"); const exno: ?usize = b.option(usize, "n", "Select exercise");
const header_step = PrintStep.create(b, logo, std.io.getStdErr()); const header_step = PrintStep.create(b, logo);
if (exno) |n| { if (exno) |n| {
if (n == 0 or n > exercises.len - 1) { if (n == 0 or n > exercises.len - 1) {
@ -861,13 +861,12 @@ const ZiglingStep = struct {
} }
}; };
// Print a message to a file. // Print a message to stderr.
const PrintStep = struct { const PrintStep = struct {
step: Step, step: Step,
message: []const u8, message: []const u8,
file: std.fs.File,
pub fn create(owner: *Build, message: []const u8, file: std.fs.File) *PrintStep { pub fn create(owner: *Build, message: []const u8) *PrintStep {
const self = owner.allocator.create(PrintStep) catch @panic("OOM"); const self = owner.allocator.create(PrintStep) catch @panic("OOM");
self.* = .{ self.* = .{
.step = Step.init(.{ .step = Step.init(.{
@ -877,7 +876,6 @@ const PrintStep = struct {
.makeFn = make, .makeFn = make,
}), }),
.message = message, .message = message,
.file = file,
}; };
return self; return self;
@ -887,7 +885,7 @@ const PrintStep = struct {
_ = prog_node; _ = prog_node;
const p = @fieldParentPtr(PrintStep, "step", step); const p = @fieldParentPtr(PrintStep, "step", step);
try p.file.writeAll(p.message); print("{s}", .{p.message});
} }
}; };