forked from lionkov/ninep
-
Notifications
You must be signed in to change notification settings - Fork 19
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
filesystem/debug: improved NewServer #37
Open
jcrussell
wants to merge
9
commits into
Harvey-OS:master
Choose a base branch
from
jcrussell:filesystem
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Commits on Jun 13, 2018
-
filesystem: new API to create FileServer
NewServer with callbacks based on functional options for friendly APIs. Replaces NewUFS. Add .Debug() to return a Debug-enabled version of the FileServer. Remove -root and -debug flags, use functional options instead. Most likely will conflict with Harvey-OS#32.
Configuration menu - View commit details
-
Copy full SHA for 8807902 - Browse repository at this point
Copy the full SHA 8807902View commit details -
Configuration menu - View commit details
-
Copy full SHA for cab800e - Browse repository at this point
Copy the full SHA cab800eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 54ad6d4 - Browse repository at this point
Copy the full SHA 54ad6d4View commit details -
Add debug.Server which wraps a NineServer and logs before and after calling functions for the underlying server. Add Trace option to protocol.Server. Makes Harvey-OS#34 obsolete
Configuration menu - View commit details
-
Copy full SHA for 0a70593 - Browse repository at this point
Copy the full SHA 0a70593View commit details -
filesystem: fix compilation errors
Sloppy merge, should have tested build afterwards.
Configuration menu - View commit details
-
Copy full SHA for e8dc951 - Browse repository at this point
Copy the full SHA e8dc951View commit details -
Configuration menu - View commit details
-
Copy full SHA for 37584d1 - Browse repository at this point
Copy the full SHA 37584d1View commit details -
Set trace for the various servers to no-op log by default. Users can then change the trace function using the Trace option.
Configuration menu - View commit details
-
Copy full SHA for 214246c - Browse repository at this point
Copy the full SHA 214246cView commit details
Commits on Jun 14, 2018
-
ufs: fix bug with default tracer
Should not be nil otherwise the *.Trace will return an error.
Configuration menu - View commit details
-
Copy full SHA for 1afc65e - Browse repository at this point
Copy the full SHA 1afc65eView commit details
Commits on Jun 19, 2018
-
Configuration menu - View commit details
-
Copy full SHA for 35ad287 - Browse repository at this point
Copy the full SHA 35ad287View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.