-
Notifications
You must be signed in to change notification settings - Fork 3
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
Fix minor typos #5
Conversation
src/System/SetEnv.hs
Outdated
@@ -57,14 +57,14 @@ eRROR_ENVVAR_NOT_FOUND = 203 | |||
-- package instead. | |||
-- | |||
-- Throws `Control.Exception.IOException` if @name@ is the empty string or | |||
-- contains an equals sign. | |||
-- contains an equal sign. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think "equals sign" is perfectly correct and fine.
setEnv :: String -> String -> IO () | ||
setEnv key value_ | ||
| null value = unsetEnv key | ||
| otherwise = setEnv_ key value | ||
where | ||
-- NOTE: Anything that follows NUL is ignored on both POSIX and Windows. | ||
-- We still strip it manually so that the null check above succeds if a | ||
-- We still strip it manually so that the null check above succeeds if a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch.
I only want the second change, see my comments. If you can adapt this, it's good for merge. |
Made the change. Thanks for writing out the lib. |
Thanks a lot! |
No description provided.