mirror of
https://github.com/facebook/sapling.git
synced 2024-10-09 08:18:15 +03:00
4d52344fee
Summary: for now this changes: ``` struct FileChange { ...stuff } fn f(x: Option<FileChange>) ``` to ``` struct TrackedFileChange { ...stuff } enum FileChange { TrackedChange(TrackedFileChange), Deleted, } fn f(x: FileChange) ``` This makes it much clearer that `None` actually means the file was deleted. It will also be useful as in the next diff I will add more stuff inside FileChange (for untracked changes), and this refactor will make it easy. (The refactor from using `Option` to putting it all inside the enum isn't really necessary, but IMO it looks much clearer, so I did it.) Reviewed By: StanislavGlebik Differential Revision: D30103454 fbshipit-source-id: afd2f29dc96baf9f3d069ad69bb3555387cff604 |
||
---|---|---|
.. | ||
allocation_tracing | ||
async_limiter | ||
bounded_traversal | ||
copy_utils | ||
dedupmap | ||
futures_watchdog | ||
iterhelpers | ||
reloader | ||
rendezvous | ||
rust | ||
scribe_ext | ||
scuba_ext | ||
sql_construct | ||
timeseries | ||
topo_sort | ||
type_map | ||
uniqueheap |