From 60aea8c7765bf2382b1d24d039f3951e00434e3c Mon Sep 17 00:00:00 2001 From: Michael Orlitzky Date: Tue, 14 Jan 2014 12:06:14 -0500 Subject: [PATCH] Minor documentation fixes; create all (even internal) docs with `make doc`. --- doc/man1/htsn.1 | 2 +- htsn.cabal | 2 +- makefile | 7 ++--- src/Configuration.hs | 6 ++-- src/ExitCodes.hs | 7 ++++- src/FeedHosts.hs | 12 ++++---- src/Main.hs | 65 ++++++++++++++++++++++++++------------------ src/Xml.hs | 29 +++++++++++++------- 8 files changed, 78 insertions(+), 52 deletions(-) diff --git a/doc/man1/htsn.1 b/doc/man1/htsn.1 index 8076e8d..78bb8d4 100644 --- a/doc/man1/htsn.1 +++ b/doc/man1/htsn.1 @@ -150,7 +150,7 @@ system configuration directory. We then look for a file named \(dq.htsnrc\(dq in the user's home directory. The latter will override the former. .P -The user's home directory is simply $HOME on Unix; on Windows its +The user's home directory is simply $HOME on Unix; on Windows it's wherever %APPDATA% points. The system configuration directory is determined by Cabal; the \(dqsysconfdir\(dq parameter during the \(dqconfigure\(dq step is used. diff --git a/htsn.cabal b/htsn.cabal index fb82e5e..8681ba7 100644 --- a/htsn.cabal +++ b/htsn.cabal @@ -191,7 +191,7 @@ description: \".htsnrc\" in the user's home directory. The latter will override the former. . - The user's home directory is simply $HOME on Unix; on Windows its + The user's home directory is simply $HOME on Unix; on Windows it's wherever %APPDATA% points. The system configuration directory is determined by Cabal; the /sysconfdir/ parameter during the \"configure\" step is used. diff --git a/makefile b/makefile index 7f9a199..7e09787 100644 --- a/makefile +++ b/makefile @@ -20,10 +20,9 @@ profile: $(PN).cabal $(SRCS) doc: runghc Setup.hs configure --user --prefix=/ - runghc Setup.hs hscolour --executables - runghc Setup.hs haddock --internal \ - --executables \ - --hyperlink-source + runghc Setup.hs hscolour --all + runghc Setup.hs haddock --all \ + --haddock-options="--ignore-all-exports" clean: runghc Setup.hs clean diff --git a/src/Configuration.hs b/src/Configuration.hs index 49213c3..042cddb 100644 --- a/src/Configuration.hs +++ b/src/Configuration.hs @@ -49,9 +49,9 @@ instance Default Configuration where username = def } --- | Merge a Configuration with an OptionalConfiguration. This is more --- or less the Monoid instance for OptionalConfiguration, but since --- the two types are different, we have to repeat ourselves. +-- | Merge a 'Configuration' with an 'OptionalConfiguration'. This is +-- more or less the Monoid instance for 'OptionalConfiguration', but +-- since the two types are different, we have to repeat ourselves. merge_optional :: Configuration -> OC.OptionalConfiguration -> Configuration diff --git a/src/ExitCodes.hs b/src/ExitCodes.hs index 4f8cd53..ade20bf 100644 --- a/src/ExitCodes.hs +++ b/src/ExitCodes.hs @@ -1,5 +1,6 @@ -- | All exit codes that the program can return (excepting --- ExitSuccess). +-- 'ExitSuccess'). +-- module ExitCodes ( exit_no_feed_hosts, exit_no_password, @@ -8,19 +9,23 @@ module ExitCodes ( where -- | No feed hosts were given on the command line or in the config file. +-- exit_no_feed_hosts :: Int exit_no_feed_hosts = 1 -- | No password was given on the command line or in the config file. +-- exit_no_password :: Int exit_no_password = 2 -- | No username was given on the command line or in the config file. +-- exit_no_username :: Int exit_no_username = 3 -- | When running as a daemon, the existence of a fixed PID file is -- used to determine whether or not the daemon is already -- running. If the PID file already exists, we shouldn't start. +-- exit_pidfile_exists :: Int exit_pidfile_exists = 4 diff --git a/src/FeedHosts.hs b/src/FeedHosts.hs index b3fcc48..58f82f5 100644 --- a/src/FeedHosts.hs +++ b/src/FeedHosts.hs @@ -2,9 +2,9 @@ -- | A newtype around a list of Strings which represent the feed -- hosts. This is all to avoid an orphan instance of Configured for --- [String] if we had defined one in e.g. OptionalConfiguration. +-- [String] if we had defined one in e.g. 'OptionalConfiguration'. -- --- This was placed under the "TSN" namespace because its Default +-- This was placed under the \"TSN\" namespace because its Default -- instance is specific to TSN, even though otherwise it's just a -- list of strings. -- @@ -29,11 +29,11 @@ newtype FeedHosts = deriving (Data, Show, Typeable) +-- | The default list of feed hosts. These were found by checking +-- PTR records in the neighborhood of the IP address in use. There +-- is a feed4.sportsnetwork.com, but it was not operational when +-- this was written. instance Default FeedHosts where - -- | The default list of feed hosts. These were found by checking - -- PTR records in the neighborhood of the IP address in use. There - -- is a feed4.sportsnetwork.com, but it was not operational when - -- this was written. def = FeedHosts ["feed1.sportsnetwork.com", "feed2.sportsnetwork.com", "feed3.sportsnetwork.com"] diff --git a/src/Main.hs b/src/Main.hs index 38dd2e5..dcb38b2 100644 --- a/src/Main.hs +++ b/src/Main.hs @@ -4,6 +4,7 @@ module Main where +-- System imports. import Control.Concurrent ( threadDelay ) import Control.Exception ( bracket, throw ) import Control.Monad ( when ) @@ -13,6 +14,13 @@ import Data.Monoid ( (<>) ) import Network ( connectTo, PortID (PortNumber) ) +import Network.Services.TSN.Logging ( init_logging ) +import Network.Services.TSN.Report ( + report_debug, + report_info, + report_warning, + report_error ) +import Network.Services.TSN.Terminal ( display_sent ) import System.Console.CmdArgs ( def ) import System.Directory ( doesFileExist ) import System.Exit ( ExitCode(..), exitWith ) @@ -30,6 +38,7 @@ import System.IO ( import System.IO.Error ( catchIOError ) import System.Timeout ( timeout ) +-- Local imports. import CommandLine ( get_args ) import Configuration ( Configuration(..), merge_optional ) import ExitCodes ( @@ -38,22 +47,15 @@ import ExitCodes ( exit_no_username, exit_pidfile_exists ) import FeedHosts ( FeedHosts(..) ) -import Network.Services.TSN.Logging ( init_logging ) import qualified OptionalConfiguration as OC ( OptionalConfiguration(..), from_rc ) -import Network.Services.TSN.Report ( - report_debug, - report_info, - report_warning, - report_error ) -import Network.Services.TSN.Terminal ( display_sent ) import Xml ( parse_xmlfid ) import Unix ( full_daemonize ) --- | Receive a single line of text from a Handle, and send it to the --- debug log. +-- | Receive a single line of text from a 'Handle', and record it for +-- debugging purposes. -- recv_line :: Handle -> IO String recv_line h = do @@ -62,14 +64,16 @@ recv_line h = do return line --- | Takes a Configuration, and an XML document (as a String). The XML --- document is written to the output directory, as specified by the --- Configuration. +-- | Takes a 'Configuration', and an XML document (as a 'String'). The +-- XML document is written to the output directory, as specified by +-- the 'Configuration'. -- -- This can fail, but we don't purposefully throw any exceptions. If -- something goes wrong, we would rather log it and keep going. -- -save_document :: Configuration -> String -> IO () +save_document :: Configuration + -> String -- ^ String representation of an XML document + -> IO () save_document cfg doc = case either_path of Left err -> report_error err @@ -87,9 +91,9 @@ save_document cfg doc = either_path = fmap ((output_directory cfg) ) filename --- | Loop forever, writing the buffer to file whenever a --- tag is seen. This is the low-level "loop forever" function that --- we stay in as long as we are connected to one feed. +-- | Loop forever, writing the @buffer@ to file whenever a +-- \ tag is seen. This is the low-level \"loop forever\" +-- function that we stay in as long as we are connected to one feed. -- -- The documentation at -- states @@ -98,7 +102,13 @@ save_document cfg doc = -- for a given document. We therefore rely on this to simplify -- processing. -- -loop :: Configuration -> Handle -> [String] -> IO () +-- The bang pattern at least on @buffer@ is necessary for +-- performance reasons. +-- +loop :: Configuration + -> Handle -- ^ Handle to the feed (network connection) + -> [String] -- ^ Current XML document buffer, line-by-line, in reverse + -> IO () loop !cfg !h !buffer = do line <- recv_line h let new_buffer = line : buffer @@ -124,9 +134,9 @@ loop !cfg !h !buffer = do -- (apparently) successfully guessed it. -- -- The first thing TSN sends once we've connected is the string --- "Username: ", containing 10 ASCII characters. We then send a +-- \"Username: \", containing 10 ASCII characters. We then send a -- username, followed by a newline. If TSN likes the username, the --- second they'll send is the string "Password: ", also containing +-- second they'll send is the string \"Password: \", also containing -- 10 ASCII characters, to which we reply in kind. -- -- Assuming the above will always hold, it is implemented as follows: @@ -139,8 +149,8 @@ loop !cfg !h !buffer = do -- -- 4. Send password if we got the password prompt -- --- If TSN likes the password as well, they send the string "The --- Sports Network" before finally beginning to stream the feed. +-- If TSN likes the password as well, they send the string \"The +-- Sports Network\" before finally beginning to stream the feed. -- log_in :: Configuration -> Handle -> IO () log_in cfg h = do @@ -188,13 +198,15 @@ log_in cfg h = do -- -- Steps: -- --- 1. Connect to the host on the XML port +-- 1. Connect to @host@ on the XML feed port. -- --- 2. Log in +-- 2. Log in. -- -- 3. Go into the eternal read/save loop. -- -connect_and_parse :: Configuration -> String -> IO () +connect_and_parse :: Configuration + -> String -- ^ Hostname to connect to + -> IO () connect_and_parse cfg host = do report_info $ "Connecting to " ++ host ++ "." bracket acquire_handle release_handle action @@ -234,7 +246,8 @@ connect_and_parse cfg host = do -- | A wrapper around threadDelay which takes seconds instead of -- microseconds as its argument. -- -thread_sleep :: Int -> IO () +thread_sleep :: Int -- ^ Number of seconds for which to sleep. + -> IO () thread_sleep seconds = do let microseconds = seconds * (10 ^ (6 :: Int)) threadDelay microseconds @@ -300,7 +313,7 @@ main = do else run_program where - -- | This is the top-level "loop forever" function. If an + -- | This is the top-level \"loop forever\" function. If an -- exception is thrown, it will propagate up to this point, where -- it will be logged and ignored in style. -- diff --git a/src/Xml.hs b/src/Xml.hs index 7f82a7d..1502d1c 100644 --- a/src/Xml.hs +++ b/src/Xml.hs @@ -3,6 +3,7 @@ -- module Xml ( parse_xmlfid, + -- * Tests xml_tests ) where @@ -20,10 +21,10 @@ import Text.XML.HXT.Core ( xreadDoc ) --- | A tiny parser written in HXT to extract the "XML_File_ID" element --- from a document. If we fail to parse an XML_File_ID, we return --- the reason wrapped in a 'Left' constructor. The reason should be --- one of two things: +-- | A tiny parser written in HXT to extract the \"XML_File_ID\" +-- element from a document. If we fail to parse an XML_File_ID, we +-- return the reason wrapped in a 'Left' constructor. The reason +-- should be one of two things: -- -- 1. No XML_File_ID elements were found. -- @@ -32,12 +33,12 @@ import Text.XML.HXT.Core ( -- -- We use an Either rather than a Maybe because we do expect some -- non-integer XML_File_IDs. In the examples, you will see --- NHL_DepthChart_XML.XML with an XML_File_ID of "49618.61" and --- CFL_Boxscore_XML1.xml with an XML_File_ID of "R28916". According --- to Brijesh Patel of TSN, these are special category files and not --- part of the usual feed. +-- NHL_DepthChart_XML.XML with an XML_File_ID of \"49618.61\" and +-- CFL_Boxscore_XML1.xml with an XML_File_ID of +-- \"R28916\". According to Brijesh Patel of TSN, these are special +-- category files and not part of the usual feed. -- --- We want to report them differently, "just in case." +-- We want to report them differently, \"just in case.\" -- parse_xmlfid :: String -- ^ The XML Document -> Either String Integer @@ -64,7 +65,12 @@ parse_xmlfid doc = parse_results = map read_either_integer elements --- * Tasty Tests +-- +-- Tasty Tests +-- + +-- | A list of all tests for this module. +-- xml_tests :: TestTree xml_tests = testGroup @@ -72,6 +78,9 @@ xml_tests = [ xml_file_id_tests ] +-- | Ensure that we parse the correct XML_File_ID out of some known +-- examples. +-- xml_file_id_tests :: TestTree xml_file_id_tests = testCase "XML_File_ID is parsed correctly" $ do -- 2.43.2