some work on avoiding partial functions
There are still hundreds of places that use partial functions head, tail, init, and last.
This commit is contained in:
parent
95e748cbd4
commit
28699c95a7
4 changed files with 45 additions and 14 deletions
18
Git.hs
18
Git.hs
|
@ -507,11 +507,7 @@ configStore s repo = do
|
|||
configParse :: String -> M.Map String String
|
||||
configParse s = M.fromList $ map pair $ lines s
|
||||
where
|
||||
pair l = (key l, val l)
|
||||
key l = head $ keyval l
|
||||
val l = join sep $ drop 1 $ keyval l
|
||||
keyval l = split sep l :: [String]
|
||||
sep = "="
|
||||
pair = separate (== '=')
|
||||
|
||||
{- Calculates a list of a repo's configured remotes, by parsing its config. -}
|
||||
configRemotes :: Repo -> IO [Repo]
|
||||
|
@ -550,13 +546,11 @@ genRemote s repo = gen $ calcloc s
|
|||
scpstyle v = ":" `isInfixOf` v && not ("//" `isInfixOf` v)
|
||||
scptourl v = "ssh://" ++ host ++ slash dir
|
||||
where
|
||||
bits = split ":" v
|
||||
host = head bits
|
||||
dir = join ":" $ drop 1 bits
|
||||
slash d | d == "" = "/~/" ++ dir
|
||||
| head d == '/' = dir
|
||||
| head d == '~' = '/':dir
|
||||
| otherwise = "/~/" ++ dir
|
||||
(host, dir) = separate (== ':') v
|
||||
slash d | d == "" = "/~/" ++ d
|
||||
| "/" `isPrefixOf` d = d
|
||||
| "~" `isPrefixOf` d = '/':d
|
||||
| otherwise = "/~/" ++ d
|
||||
|
||||
{- Checks if a string from git config is a true value. -}
|
||||
configTrue :: String -> Bool
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue