From 066632261492ee7624117ad09269f57526aca4c0 Mon Sep 17 00:00:00 2001
From: Andrew Kelley <andrew@ziglang.org>
Date: Thu, 9 Feb 2023 10:00:25 -0700
Subject: [PATCH] std.Build.Cache: remove debug log statements

Now that this API is used by the build system, these debug logs are
problematic because build scripts run in debug mode, making these logs
noisy output.
---
 lib/std/Build/Cache.zig | 25 +------------------------
 1 file changed, 1 insertion(+), 24 deletions(-)

diff --git a/lib/std/Build/Cache.zig b/lib/std/Build/Cache.zig
index e316322c68f0547b1c0ad4f2a173c3ef0a12a1e3..c459fca6332c7ca04e8e8f5cc50402dce40eb658 100644
--- a/lib/std/Build/Cache.zig
+++ b/lib/std/Build/Cache.zig
@@ -53,7 +53,7 @@ mutex: std.Thread.Mutex = .{},
 /// are replaced with single-character indicators. This is not to save
 /// space but to eliminate absolute file paths. This improves portability
 /// and usefulness of the cache for advanced use cases.
-prefixes_buffer: [3]Directory = undefined,
+prefixes_buffer: [4]Directory = undefined,
 prefixes_len: usize = 0,
 
 pub const DepTokenizer = @import("Cache/DepTokenizer.zig");
@@ -71,9 +71,6 @@ const Allocator = std.mem.Allocator;
 const log = std.log.scoped(.cache);
 
 pub fn addPrefix(cache: *Cache, directory: Directory) void {
-    if (directory.path) |p| {
-        log.debug("Cache.addPrefix {d} {s}", .{ cache.prefixes_len, p });
-    }
     cache.prefixes_buffer[cache.prefixes_len] = directory;
     cache.prefixes_len += 1;
 }
@@ -120,8 +117,6 @@ fn findPrefixResolved(cache: *const Cache, resolved_path: []u8) !PrefixedPath {
                 .prefix = @intCast(u8, i),
                 .sub_path = sub_path,
             };
-        } else {
-            log.debug("'{s}' does not start with '{s}'", .{ resolved_path, p });
         }
     }
 
@@ -319,10 +314,6 @@ pub const Manifest = struct {
         const prefixed_path = try self.cache.findPrefix(file_path);
         errdefer gpa.free(prefixed_path.sub_path);
 
-        log.debug("Manifest.addFile {s} -> {d} {s}", .{
-            file_path, prefixed_path.prefix, prefixed_path.sub_path,
-        });
-
         self.files.addOneAssumeCapacity().* = .{
             .prefixed_path = prefixed_path,
             .contents = null,
@@ -687,10 +678,6 @@ pub const Manifest = struct {
         const prefixed_path = try self.cache.findPrefix(file_path);
         errdefer gpa.free(prefixed_path.sub_path);
 
-        log.debug("Manifest.addFilePostFetch {s} -> {d} {s}", .{
-            file_path, prefixed_path.prefix, prefixed_path.sub_path,
-        });
-
         const new_ch_file = try self.files.addOne(gpa);
         new_ch_file.* = .{
             .prefixed_path = prefixed_path,
@@ -717,10 +704,6 @@ pub const Manifest = struct {
         const prefixed_path = try self.cache.findPrefix(file_path);
         errdefer gpa.free(prefixed_path.sub_path);
 
-        log.debug("Manifest.addFilePost {s} -> {d} {s}", .{
-            file_path, prefixed_path.prefix, prefixed_path.sub_path,
-        });
-
         const new_ch_file = try self.files.addOne(gpa);
         new_ch_file.* = .{
             .prefixed_path = prefixed_path,
@@ -748,15 +731,9 @@ pub const Manifest = struct {
         const ch_file = try self.files.addOne(gpa);
         errdefer self.files.shrinkRetainingCapacity(self.files.items.len - 1);
 
-        log.debug("Manifest.addFilePostContents resolved_path={s}", .{resolved_path});
-
         const prefixed_path = try self.cache.findPrefixResolved(resolved_path);
         errdefer gpa.free(prefixed_path.sub_path);
 
-        log.debug("Manifest.addFilePostContents -> {d} {s}", .{
-            prefixed_path.prefix, prefixed_path.sub_path,
-        });
-
         ch_file.* = .{
             .prefixed_path = prefixed_path,
             .max_file_size = null,
-- 
2.54.0