Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add important documentation on the async nature of Vfs::startImpl() #12078

Merged
merged 1 commit into from
Feb 24, 2025

Conversation

erikjv
Copy link
Collaborator

@erikjv erikjv commented Feb 21, 2025

This is to inform/warn VFS implementors of the fact that starting the VFS implementation is assumed to possibly start in the background. But more importantly: the client will not "wait" for one VFS instance to be started before calling start on a second instance. Meaning: synchronization (when needed) has to be done by the VFS impementation itself.

This is to inform/warn VFS implementors of the fact that starting the
VFS implementation is assumed to possibly start in the background. But
more importantly: the client will not "wait" for one VFS instance to be
started before calling `start` on a second instance. Meaning:
synchronization (when needed) has to be done by the VFS impementation
itself.
@erikjv erikjv added Windows feature:vfs native virtual files and placeholder implementation labels Feb 21, 2025
@erikjv erikjv self-assigned this Feb 21, 2025
Copy link

update-docs bot commented Feb 21, 2025

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@DeepDiver1975 DeepDiver1975 merged commit 58d21cb into master Feb 24, 2025
2 of 4 checks passed
@DeepDiver1975 DeepDiver1975 deleted the work/add-vfs/doc branch February 24, 2025 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature:vfs native virtual files and placeholder implementation Windows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants