Feedback from @jaseemabid, Fixes #6

diff --git a/src/Logger.hs b/src/Logger.hs
index 5863cce7e7ee0ddece753d3acc9ee2678d6085ce..c1a58942baa22958f2ab353e9a024e69fd8582bc 100644 (file)
--- a/src/Logger.hs
+++ b/src/Logger.hs
@@ -1,6 +1,5 @@
 module Logger (
-      Logger
-    , initLogger
+      initLogger
     , logMessage
     , logStop
     ) where

diff --git a/src/Main.hs b/src/Main.hs
index 2213ac97a5244f3f9b457b77031dc904c1274a99..b8afb0e830a25853d37632e250633055f814cd53 100644 (file)
--- a/src/Main.hs
+++ b/src/Main.hs
@@ -13,8 +13,8 @@ import Tracker (connect, prepareRequest)
 import Text.ParserCombinators.Parsec (ParseError)
 import Logger
 
-printError :: ParseError -> Logger -> IO ()
-printError e l = logMessage l $ "parse error: \n" ++ show e
+logError :: ParseError -> (String -> IO ()) -> IO ()
+logError e logMsg = logMsg $ "parse error: \n" ++ show e
 
 peerId :: String
 peerId = "-HS0001-*-*-20150215"
@@ -34,22 +34,28 @@ main :: IO ()
 main = do
     args <- getArgs
     logR <- initLogger
-    logMessage logR $ "Starting parsing input file: " ++ (concat args)
+    let logMsg = logMessage logR
+    logMsg $ "Parsing input file: " ++ concat args
     torrentStr <- parse args
     case decode torrentStr of
       Right d ->
           case mkMetaInfo d of
-            Nothing -> logMessage logR "parse error"
+            Nothing -> logMsg "parse error"
             Just m -> do
+              logMsg "Input File OK"
+
               let len = lengthInBytes $ info m
                   (Bdict d') = d
+              
+              logMsg "Trying to fetch peers: "
+
               body <- pack <$> connect (announce m) (prepareRequest d' peerId len)
               
               let peerResponse = show $ getPeers $ getPeerResponse body
-              logMessage logR $ "Peers List : " ++ peerResponse
+              logMsg $ "Peers List : " ++ peerResponse
               
               let hsMsgLen = show $ length $ handShakeMsg d' peerId
-              logMessage logR $ "Hand-shake message length : " ++ hsMsgLen
+              logMsg $ "Hand-shake message length : " ++ hsMsgLen
 
-      Left e -> printError e logR
+      Left e -> logError e logMsg
     logStop logR