[Lldb-commits] [lldb] Skip tests if socket name is longer than 107 bytes (PR #137405)

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Mon Apr 28 00:01:28 PDT 2025


https://github.com/labath commented:

Abstract sockets aren't tied to the filesystem, so you don't need to make them "be in" any particular directory, or even look like a path. The only reason I suggested to use `createUniquePath` is because it's a convenient way to create a random (path-like) string, but it looks like you're not making use of that (the first argument to this function has a slightly different meaning than it does for createUniqueDirectory).

The way I envisioned that is to do a
```
createUniquePath("AbstractSocketFromBoundNativeSocket-%%%%%%%%", name, /*MakeAbsolute=*/false);
```

.. though now that I think about it, it may be even better to detach this from the file system completely and use something like `UUID::Generate().GetAsString()` to generate a random name.

https://github.com/llvm/llvm-project/pull/137405


More information about the lldb-commits mailing list