Skip to content

Commit e2dad91

Browse files
muirdmmeta-codesync[bot]
authored andcommitted
glob: add shared result path type
Summary: Add `GlobPath`, a compact path representation that shares each directory prefix and stores only the basename per result. Comparisons and Thrift serialization operate directly on the pieces without materializing the full path. Paths converted before shared traversal state is available retain one owned full-path buffer, avoiding separate prefix and basename allocations. `toIOBuf()` returns its head wrapper by value to avoid one temporary allocation per serialization call. This also adds `PathComponent::intoStorage()` so traversal can move `PathComponent` storage into result objects without another basename allocation. This commit introduces and tests the representation; the following commit wires it into glob traversal and Thrift responses. Reviewed By: genevievehelsel Differential Revision: D113152048 fbshipit-source-id: a761dc3c447150f9800c1f39b667717794a3ff53
1 parent be720a6 commit e2dad91

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

build/fbcode_builder/manifests/watchman

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ fbcode/watchman/oss = .
3030
fbcode/eden/fs = eden/fs
3131

3232
[shipit.strip]
33-
^fbcode/eden/fs/(?!.*\.thrift|service/shipit_test_file\.txt)
33+
^fbcode/eden/fs/(?!.*\.thrift|service/shipit_test_file\.txt|utils/GlobPath\.h$)
3434

3535
[cmake.defines.fb=on]
3636
ENABLE_EDEN_SUPPORT=ON

0 commit comments

Comments
 (0)