diff options
author | Florian Klink <flokli@flokli.de> | 2024-04-15T12·14+0300 |
---|---|---|
committer | clbot <clbot@tvl.fyi> | 2024-04-15T14·49+0000 |
commit | b025a30d271458386769e8721231e1e219481f57 (patch) | |
tree | 527e18db09312e6779d6c8bf4828f42a08ec0be1 /tools/cheddar | |
parent | fb852b0245aab7637ffca2b7583be2e282cfe063 (diff) |
refactor(tvix/castore/fs): remove From<Node> for InodeData r/7931
These were copying unnecessarily. Instead, have a InodeData::from_node(), which *consumes* the Node entirely, returns `InodeData` and the split-off name (which is not part of InodeData). Callers can then use the result in various helper functions, like: - InodeData::as_fuse_type - InodeData::as_fuse_file_attr - InodeData::as_fuse_entry … to prepare their replies to the kernel. This removes not only a bunch of clones, but also a lot of copy-pasted code. Change-Id: Idbca5f25cc29e96c1f4c614b33dff2becb0a8738 Reviewed-on: https://cl.tvl.fyi/c/depot/+/11435 Tested-by: BuildkiteCI Autosubmit: flokli <flokli@flokli.de> Reviewed-by: Connor Brewster <cbrewster@hey.com>
Diffstat (limited to 'tools/cheddar')
0 files changed, 0 insertions, 0 deletions