]> gitweb @ CieloNegro.org - Lucu.git/commitdiff
Cosmetic changes suggested by hlint.
authorPHO <pho@cielonegro.org>
Thu, 13 Oct 2011 13:14:17 +0000 (22:14 +0900)
committerPHO <pho@cielonegro.org>
Thu, 13 Oct 2011 13:14:17 +0000 (22:14 +0900)
Ditz-issue: 8959dadc07db1bd363283dee401073f6e48dc7fa

14 files changed:
ImplantFile.hs
Network/HTTP/Lucu/Authorization.hs
Network/HTTP/Lucu/ContentCoding.hs
Network/HTTP/Lucu/DefaultPage.hs
Network/HTTP/Lucu/Headers.hs
Network/HTTP/Lucu/HttpVersion.hs
Network/HTTP/Lucu/Httpd.hs
Network/HTTP/Lucu/MIMEType/Guess.hs
Network/HTTP/Lucu/MultipartForm.hs
Network/HTTP/Lucu/Parser/Http.hs
Network/HTTP/Lucu/Resource.hs
Network/HTTP/Lucu/Utils.hs
cabal-package.mk
data/Makefile

index 0e91f1c6bdbb7bd2b5667d68790325ad5ad88a95..3b80e60844d3835cda549d64b5bd99325d7b282f 100644 (file)
@@ -159,12 +159,10 @@ mkImports useGZip
                    False False Nothing Nothing Nothing
       ]
       ⧺
-      if useGZip then
-          [ ImportDecl (⊥) (ModuleName "Codec.Compression.GZip")
-                       False False Nothing Nothing Nothing
-          ]
-      else
-          []
+      [ ImportDecl (⊥) (ModuleName "Codec.Compression.GZip")
+                   False False Nothing Nothing Nothing
+        | useGZip
+      ]
 
 resourceDecl ∷ Name → Bool → [Decl]
 resourceDecl symName useGZip
@@ -234,22 +232,22 @@ resGetRaw
 setContentEncodingGZipStmt ∷ Stmt
 setContentEncodingGZipStmt
     = qualStmt $
-      metaFunction "setContentEncoding" $
-      [ listE [ strE "gzip" ] ]
+      metaFunction "setContentEncoding"
+                       [ listE [ strE "gzip" ] ]
 
 foundEntityStmt ∷ Stmt
 foundEntityStmt
     = qualStmt $
-      metaFunction "foundEntity" $
-      [ var $ name "entityTag"
-      , var $ name "lastModified"
-      ]
+      metaFunction "foundEntity"
+                       [ var $ name "entityTag"
+                       , var $ name "lastModified"
+                       ]
 
 setContentTypeStmt ∷ Stmt
 setContentTypeStmt
     = qualStmt $
-      metaFunction "setContentType" $
-      [var $ name "contentType"]
+      metaFunction "setContentType"
+                       [var $ name "contentType"]
 
 outputStmt ∷ Exp → Stmt
 outputStmt e
@@ -342,9 +340,9 @@ eTagToString = A.toString ∘ A.fromAsciiBuilder ∘ printETag
 getModuleName ∷ [CmdOpt] → IO ModuleName
 getModuleName opts
     = case modNameOpts of
-        []                      → fail "a module name must be given."
-        (OptModName modName):[] → return $ ModuleName modName
-        _                       → fail "too many --module options."
+        []                    → fail "a module name must be given."
+        OptModName modName:[] → return $ ModuleName modName
+        _                     → fail "too many --module options."
     where
       modNameOpts ∷ [CmdOpt]
       modNameOpts = filter (\ x → case x of
@@ -354,9 +352,9 @@ getModuleName opts
 getSymbolName ∷ [CmdOpt] → ModuleName → IO Name
 getSymbolName opts (ModuleName modName)
     = case symNameOpts of
-        []                      → return defaultSymName
-        (OptSymName symName):[] → return $ name symName
-        _                       → fail "too many --symbol options."
+        []                    → return defaultSymName
+        OptSymName symName:[] → return $ name symName
+        _                     → fail "too many --symbol options."
     where
       symNameOpts ∷ [CmdOpt]
       symNameOpts = filter (\ x → case x of
@@ -378,7 +376,7 @@ getMIMEType ∷ [CmdOpt] → FilePath → IO MIMEType
 getMIMEType opts srcFile
     = case mimeTypeOpts of
         []  → return defaultType
-        (OptMIMEType ty):[]
+        OptMIMEType ty:[]
             → case A.fromChars ty of
                  Just a  → return $ parseMIMEType a
                  Nothing → fail "MIME type must not contain any non-ASCII letters."
@@ -406,9 +404,9 @@ getLastModified fpath = (posixSecondsToUTCTime ∘ fromRational ∘ toRational 
 getETag ∷ [CmdOpt] → Lazy.ByteString → IO ETag
 getETag opts input
     = case eTagOpts of
-        []               → return $ mkETagFromInput
-        (OptETag str):[] → return $ strToETag str
-        _                → fail "too many --etag options."
+        []             → return mkETagFromInput
+        OptETag str:[] → return $ strToETag str
+        _              → fail "too many --etag options."
     where
       eTagOpts ∷ [CmdOpt]
       eTagOpts = filter (\ x → case x of
@@ -432,9 +430,9 @@ openInput fpath = Lazy.readFile fpath
 openOutput ∷ [CmdOpt] → IO Handle
 openOutput opts
     = case outputOpts of
-        []                   → return stdout
-        (OptOutput fpath):[] → openFile fpath WriteMode
-        _                    → fail "two many --output options."
+        []                 → return stdout
+        OptOutput fpath:[] → openFile fpath WriteMode
+        _                  → fail "two many --output options."
     where
       outputOpts ∷ [CmdOpt]
       outputOpts = filter (\ x → case x of
index d91fe29024dc3a8ac6e27bb1e989ffa287044b9b..789b5d1c971dfba93f0612e46d1fe79b86fdc9ad 100644 (file)
@@ -29,7 +29,7 @@ import Prelude.Unicode
 -- \"WWW-Authenticate\" header. See
 -- 'Network.HTTP.Lucu.Resource.setWWWAuthenticate'.
 data AuthChallenge
-    = BasicAuthChallenge Realm
+    = BasicAuthChallenge !Realm
       deriving (Eq)
 
 -- |'Realm' is just a string which must not contain any non-ASCII letters.
@@ -39,7 +39,7 @@ type Realm = Ascii
 -- \"Authorization\" header. See
 -- 'Network.HTTP.Lucu.Resource.getAuthorization'.
 data AuthCredential
-    = BasicAuthCredential UserID Password
+    = BasicAuthCredential !UserID !Password
       deriving (Show, Eq)
 
 -- |'UserID' is just a string which must not contain colon and any
index 315d23793fa477b9ce4166411c3e47bbf8b9fa4e..3ce7806dee86d5271ec50e8e1d4a68d7fd97a1ad 100644 (file)
@@ -47,20 +47,12 @@ accEncP = do coding ← toCIAscii <$> token
 
 normalizeCoding ∷ CIAscii → CIAscii
 normalizeCoding coding
-    = if coding ≡ "x-gzip" then
-          "gzip"
-      else
-          if coding ≡ "x-compress" then
-              "compress"
-          else
-              coding
+    | coding ≡ "x-gzip"     = "gzip"
+    | coding ≡ "x-compress" = "compress"
+    | otherwise             = coding
 
 unnormalizeCoding ∷ CIAscii → CIAscii
 unnormalizeCoding coding
-    = if coding ≡ "gzip" then
-          "x-gzip"
-      else
-          if coding ≡ "compress" then
-              "x-compress"
-          else
-              coding
+    | coding ≡ "gzip"     = "x-gzip"
+    | coding ≡ "compress" = "x-compress"
+    | otherwise           = coding
index e1bdf1ce6e3daeb34af5e7f3b07ccf955364265d..b530455f6ce99d686843f343aa4d4ba7042baab7 100644 (file)
@@ -1,7 +1,6 @@
 {-# LANGUAGE
     OverloadedStrings
   , RecordWildCards
-  , UnboxedTuples
   , UnicodeSyntax
   #-}
 module Network.HTTP.Lucu.DefaultPage
index 3308bbfad7abe200bbc92951c9c8b2618a502839..e72022c72c0d5f11f253504fd5ed34c4288bd1d4 100644 (file)
@@ -127,7 +127,7 @@ headersP = do xs ← P.many header
       {-# INLINE content #-}
       content = A.unsafeFromByteString
                 <$>
-                takeWhile1 (\c → ((¬) (isSPHT c)) ∧ isText c)
+                takeWhile1 (\c → (¬) (isSPHT c) ∧ isText c)
 
       joinValues ∷ [Ascii] → Ascii
       {-# INLINE joinValues #-}
index a5db1e29193a10aaeec1b4c67ca57092f906a477..9ad1c0a07ccd964d9cdcee554ff604a5c558856d 100644 (file)
@@ -1,9 +1,7 @@
 {-# LANGUAGE
-    BangPatterns
-  , OverloadedStrings
+    OverloadedStrings
   , UnicodeSyntax
   #-}
-
 -- |Manipulation of HTTP version string.
 module Network.HTTP.Lucu.HttpVersion
     ( HttpVersion(..)
index 0bb92b1635c74a1498b1ee2936f807d6153d356d..d180202759a903fe8a478eae1ecaf28d5b50c1e1 100644 (file)
@@ -101,7 +101,7 @@ runHttpd cnf tree fbs
                let addr = head addrs
                bracketOnError
                    (socket (addrFamily addr) (addrSocketType addr) (addrProtocol addr))
-                   (sClose)
+                   sClose
                    (\ sock ->
                         do setSocketOption sock ReuseAddr 1
                            bindSocket sock (addrAddress addr)
index 3917cf25dac133c9487454090fbb5ac63b30dba5..2664d79b859e853565090f5581e103e7f9fce17f 100644 (file)
@@ -93,7 +93,7 @@ compile = M.fromList ∘ concat ∘ map tr
 --
 -- * A definition of module named @moduleName@.
 --
--- * @variableName  'ExtMap'@ whose content is a serialization of
+-- * @variableName :: 'ExtMap'@ whose content is a serialization of
 --   @extMap@.
 --
 -- The module "Network.HTTP.Lucu.MIMEType.DefaultExtensionMap" is
index c36b81905cb147ee37eb28c1463bcf8e51069dee..7d0866cd4225fe0ec54f8c77f84a01fc24e5dbe6 100644 (file)
@@ -53,7 +53,7 @@ data ContDispo
 
 printContDispo ∷ ContDispo → Ascii
 printContDispo d
-    = A.fromAsciiBuilder $
+    = A.fromAsciiBuilder
       ( A.toAsciiBuilder (A.fromCIAscii $ dType d)
         ⊕
         printParams (dParams d) )
@@ -122,7 +122,7 @@ getContDispo ∷ Monad m ⇒ Headers → m ContDispo
 getContDispo hdr
     = case getHeader "Content-Disposition" hdr of
         Nothing
-            → fail ("There is a part without Content-Disposition in the multipart/form-data.")
+            → fail "There is a part without Content-Disposition in the multipart/form-data."
         Just str
             → let p  = do d ← contDispoP
                           endOfInput
index 520034247726f3ec6398eb8b69b143eb08456ceb..4153dcb6eaf62f071e5ba7fe3406daa9d3c4c881 100644 (file)
@@ -1,6 +1,5 @@
 {-# LANGUAGE
-    BangPatterns
-  , OverloadedStrings
+    OverloadedStrings
   , ScopedTypeVariables
   , UnicodeSyntax
   #-}
@@ -79,18 +78,16 @@ isChar = (≤ '\x7F')
 -- c)@
 isToken ∷ Char → Bool
 {-# INLINE isToken #-}
-isToken !c
-    = (¬) (isCtl c ∨ isSeparator c)
+isToken c = (¬) (isCtl c ∨ isSeparator c)
 
 -- |@'listOf' p@ is similar to @'sepBy' p ('char' \',\')@ but it
 -- allows any occurrences of 'lws' before and after each tokens.
 listOf ∷ Parser a → Parser [a]
 {-# INLINEABLE listOf #-}
-listOf p
-    = do skipMany lws
-         sepBy p $ do skipMany lws
-                      _ <- char ','
-                      skipMany lws
+listOf p = do skipMany lws
+              sepBy p $ do skipMany lws
+                           _ ← char ','
+                           skipMany lws
 
 -- |'token' is similar to @'takeWhile1' 'isToken'@
 token ∷ Parser Ascii
@@ -100,12 +97,12 @@ token = A.unsafeFromByteString <$> takeWhile1 isToken
 -- |The CRLF: 0x0D 0x0A.
 crlf ∷ Parser ()
 {-# INLINE crlf #-}
-crlf = string "\x0D\x0A"  return ()
+crlf = string "\x0D\x0A" *> return ()
 
 -- |The SP: 0x20.
 sp ∷ Parser ()
 {-# INLINE sp #-}
-sp = char '\x20'  return ()
+sp = char '\x20' *> return ()
 
 -- |HTTP LWS: crlf? (sp | ht)+
 lws ∷ Parser ()
@@ -142,7 +139,7 @@ quotedStr = try $
 
       quotedPair ∷ Parser Char
       {-# INLINE quotedPair #-}
-      quotedPair = char '\\'  satisfy isChar
+      quotedPair = char '\\' *> satisfy isChar
 
 -- |'qvalue' accepts a so-called qvalue.
 qvalue ∷ Parser Double
index 975744c5164f30ee6323fc1b48a222b710fc5da8..87d2a33338d1e0a726c80af4fef3e1735fbc3e23 100644 (file)
@@ -5,7 +5,7 @@
   , RecordWildCards
   , UnicodeSyntax
   #-}
-
+{-# OPTIONS_HADDOCK prune #-}
 -- |This is the Resource Monad; monadic actions to define the behavior
 -- of each resources. The 'Resource' Monad is a kind of 'Prelude.IO'
 -- Monad thus it implements 'Control.Monad.Trans.MonadIO' class. It is
@@ -262,9 +262,9 @@ getRequestVersion = reqVersion <$> getRequest
 -- >
 -- > resFoo = ResourceDef {
 -- >     resIsGreedy = True
--- >   , resGet = Just $ do requestURI    getRequestURI
--- >                        resourcePath  getResourcePath
--- >                        pathInfo      getPathInfo
+-- >   , resGet = Just $ do requestURI   <- getRequestURI
+-- >                        resourcePath <- getResourcePath
+-- >                        pathInfo     <- getPathInfo
 -- >                        -- uriPath requestURI == "/foo/bar/baz"
 -- >                        -- resourcePath       == ["foo"]
 -- >                        -- pathInfo           == ["bar", "baz"]
index a488aafbb64161feee9e67404c05f29a065236ec..51025248c5455f46f31ac938ffc3e376ede1b5d5 100644 (file)
@@ -1,6 +1,5 @@
 {-# LANGUAGE
-    BangPatterns
-  , OverloadedStrings
+    OverloadedStrings
   , UnicodeSyntax
   #-}
 -- |Utility functions used internally in the Lucu httpd. These
@@ -32,6 +31,7 @@ import Prelude.Unicode
 -- |> splitBy (== ':') "ab:c:def"
 --  > ==> ["ab", "c", "def"]
 splitBy ∷ (a → Bool) → [a] → [[a]]
+{-# INLINEABLE splitBy #-}
 splitBy isSep src
     = case break isSep src
       of (last , []       ) → [last]
index e4121394d75de781787a0b046c55cc660a380060..b2bf65597e8426a73d30fbdb637f08f33d54f7dd 100644 (file)
@@ -18,6 +18,7 @@ HPC      ?= hpc
 DITZ     ?= ditz
 
 CONFIGURE_ARGS ?= --disable-optimization
+HLINT_OPTS     ?= --cross --report=dist/report.html
 
 SETUP_FILE := $(wildcard Setup.*hs)
 CABAL_FILE := $(wildcard *.cabal)
@@ -117,7 +118,7 @@ fixme:
                || echo 'No FIXME or THINKME found.'
 
 lint:
-       $(HLINT) . --cross --report
+       $(HLINT) . $(HLINT_OPTS)
 
 push: push-repo push-ditz push-doc
 
index 04bd97f73c0d4365fe443634aa7c486718c108bc..c73c1f3c2e5416eea76156db425503848ad718d8 100644 (file)
@@ -10,7 +10,7 @@ dist/DefaultExtensionMap.hs: mime.types compiler
        fi
 
 compiler:
-       ghc --make CompileMimeTypes -i.. -odir dist -hidir dist
+       ghc -Wall --make CompileMimeTypes -i.. -odir dist -hidir dist
 
 clean:
        rm -rf dist DefaultExtensionMap.hs CompileMimeTypes