Commit c84e086a2f3d7f45eade35de1bfb32c34c1d8ecf | zig

author	MrDmitry <aidenhaledev@gmail.com> 2024-01-26 10:22:08-05:00
committer	MrDmitry <aidenhaledev@gmail.com> 2024-01-26 15:59:05-05:00
log	c84e086a2f3d7f45eade35de1bfb32c34c1d8ecf
tree	4badd25b7ae36c24949e9768847b58ebd98ff09f
parent	9ecbf533889d50d5a8eb1a36891a4a9b91a6a1f9

Get rid of direct dependency on std.fmt

Replace unnecessary panic with step error

1 files changed, 45 insertions(+), 45 deletions(-)

lib
- std
  - Build
    Step
    ConfigHeader.zig

lib/std/Build/Step/ConfigHeader.zig+45-45

 @@ -7,7 +7,7 @@ pub const Style = union(enum) {
     /// The configure format supported by autotools. It uses `#undef foo` to
     /// mark lines that can be substituted with different values.
     autoconf: std.Build.LazyPath,
-    /// The configure format supported by CMake. It uses `@@FOO@@` and
+    /// The configure format supported by CMake. It uses `@FOO@`, `${}` and
     /// `#cmakedefine` for template substitution.
     cmake: std.Build.LazyPath,
     /// Instead of starting with an input file, start with nothing.
 @@ -32,28 +32,6 @@ pub const Value = union(enum) {
     string: []const u8,
 };
-fn formatValueCMake(data: Value, comptime fmt: []const u8, options: std.fmt.FormatOptions, writer: anytype) !void {
-    _ = fmt;
-    _ = options;
-    switch (data) {
-        .undef, .defined => {},
-        .boolean => |b| {
-            try writer.print("{d}", .{@intFromBool(b)});
-        },
-        .int => |i| {
-            try writer.print("{d}", .{i});
-        },
-        .ident, .string => |s| {
-            try writer.writeAll(s);
-        },
-    }
-}
-fn fmtValueCMake(value: Value) std.fmt.Formatter(formatValueCMake) {
-    return .{ .data = value };
-}
 step: Step,
 values: std.StringArrayHashMap(Value),
 output_file: std.Build.GeneratedFile,
 @@ -344,7 +322,11 @@ fn render_cmake(
                 continue;
             },
             else => {
-                @panic("Failed to substitute");
+                try step.addError("{s}:{d}: unable to substitute variable: error: {s}", .{
+                    src_path, line_index + 1, @errorName(err),
+                });
+                any_errors = true;
+                continue;
             },
         };
         defer allocator.free(line);
 @@ -549,8 +531,8 @@ fn expand_variables_cmake(
     contents: []const u8,
     values: std.StringArrayHashMap(Value),
 ) ![]const u8 {
-    var result = allocator.alloc(u8, 0) catch @panic("OOM");
+    var result = std.ArrayList(u8).init(allocator);
-    errdefer allocator.free(result);
+    errdefer result.deinit();
     const valid_varname_chars = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789/_.+-";
     const open_var = "${";
 @@ -571,8 +553,8 @@ fn expand_variables_cmake(
                         // closed immediately, preserve as a literal
                         break :blk;
                     }
-                    const valid_varname_end = std.mem.indexOfNonePos(u8, contents, curr + 1, valid_varname_chars);
+                    const valid_varname_end = std.mem.indexOfNonePos(u8, contents, curr + 1, valid_varname_chars) orelse 0;
-                    if (valid_varname_end == null or valid_varname_end != close_pos) {
+                    if (valid_varname_end != close_pos) {
                         // contains invalid characters, preserve as a literal
                         break :blk;
                     }
 @@ -580,9 +562,19 @@ fn expand_variables_cmake(
                     const key = contents[curr + 1 .. close_pos];
                     const value = values.get(key) orelse .undef;
                     const missing = contents[source_offset..curr];
-                    const buf = try std.fmt.allocPrint(allocator, "{s}{s}{}", .{ result, missing, fmtValueCMake(value) });
+                    try result.appendSlice(missing);
-                    allocator.free(result);
+                    switch (value) {
-                    result = buf;
+                        .undef, .defined => {},
+                        .boolean => |b| {
+                            try result.append(if (b) '1' else '0');
+                        },
+                        .int => |i| {
+                            try result.writer().print("{d}", .{i});
+                        },
+                        .ident, .string => |s| {
+                            try result.appendSlice(s);
+                        },
+                    }
                     curr = close_pos;
                     source_offset = close_pos + 1;
 @@ -597,15 +589,14 @@ fn expand_variables_cmake(
                     break :blk;
                 }
                 const missing = contents[source_offset..curr];
-                const buf = try std.fmt.allocPrint(allocator, "{s}{s}{s}", .{ result, missing, open_var });
+                try result.appendSlice(missing);
-                allocator.free(result);
+                try result.appendSlice(open_var);
-                result = buf;
                 source_offset = curr + open_var.len;
                 curr = next;
                 try var_stack.append(Position{
                     .source = curr,
-                    .target = result.len - open_var.len,
+                    .target = result.items.len - open_var.len,
                 });
                 continue :loop;
 @@ -620,14 +611,24 @@ fn expand_variables_cmake(
                     source_offset += open_var.len;
                 }
                 const missing = contents[source_offset..curr];
-                const key_start = open_pos.target + open_var.len;
+                try result.appendSlice(missing);
-                const key = try std.fmt.allocPrint(allocator, "{s}{s}", .{ result[key_start..], missing });
-                defer allocator.free(key);
+                const key_start = open_pos.target + open_var.len;
+                const key = result.items[key_start..];
                 const value = values.get(key) orelse .undef;
-                const buf = try std.fmt.allocPrint(allocator, "{s}{}", .{ result[0..open_pos.target], fmtValueCMake(value) });
+                result.shrinkRetainingCapacity(result.items.len - key.len - open_var.len);
-                allocator.free(result);
+                switch (value) {
-                result = buf;
+                    .undef, .defined => {},
+                    .boolean => |b| {
+                        try result.append(if (b) '1' else '0');
+                    },
+                    .int => |i| {
+                        try result.writer().print("{d}", .{i});
+                    },
+                    .ident, .string => |s| {
+                        try result.appendSlice(s);
+                    },
+                }
                 source_offset = curr + 1;
 @@ -646,12 +647,11 @@ fn expand_variables_cmake(
     }
     if (source_offset != contents.len) {
-        const buf = try std.fmt.allocPrint(allocator, "{s}{s}", .{ result, contents[source_offset..] });
+        const missing = contents[source_offset..];
-        allocator.free(result);
+        try result.appendSlice(missing);
-        result = buf;
     }
-    return result;
+    return result.toOwnedSlice();
 }
 fn testReplaceVariables(