Fixed the type punning from boost::net::dns::message class

Removed one level of indirection provided by PingScheduler::handle_next_ping

Reading the nameserver form configuration file

Non-virtual destructors for classes that are unlikely to be base classes

Using the boost-net-dns library to resolve DNS host names

Commented the -O2 optmization level because it activates the -Wstrict-aliasing warning, which is issued by the boost-net-dns libraries.

Replaced post-fixed by pre-fixed increment

Increased the warning level
- included -pedantic, -Wshadow, -Wcast-qual and -Wconversion warnings
- fixed some warnings regarding lost of precision from conversions
- asserted that intermediate products from arithmetic operations fall within certain range

Using the std namespace within the souce file

Only the "using namespace std" is allowed, classes from other namespaces are included via "using"
- avoid conflicts with the boost DNS library classes

Renamed packages to more appropriate names
- ping to host, because it contains classes to deal with host
- notify to link, contains classes that deal with many hosts, on link basis

Encapsulated the encode, decode, read and write methods within the IcmpMessagePayload class
- simplify the message classes

Fixed more problems issued by PC-lint
- alternative initialization to avoid calling virtual methods from constructor

Fixed more problems issued by PC-lint (3 times now, bad programmer!)

Increment the replies count when receiving a destination unreachable, avoiding the request timeout message

Separate method to print the timeout of a echo request

Found an issue with sequence number

Pinger now handles ICMP Destination Unreachable messages
- handle_receive_echo_reply rename to handle_receive_icmp_packet, once it handles not just echo replies
- timer member variable has a more meaningful name

Refactoring the ICMP message parsing
- IcmpType enum separated from the IcmpHeader class, thus it has its own header without requiring to add the IcmpHeader's header file
- the IcmpHeader class delegates the parsing polymorphically to specialized IcmpMessage classes  (i.e. allowing the parsing of Destination unreachable messages)
- IcmpHeader class does not requires the Payload member variable anymore, once it delegates the parsing to IcmpMessage classes

More meaningful name for timer objects.

Given more meaningful names to the variables

More wish list items

Implemented the requirement "Support selection of the source network interface, so we can change the source ip address when ping"
- unfortunately, I couldn't find a way to bind to an interface through Bosst Asio. So it was required to rely on native Unix Socket interface.

Reading the network interface from the configuration file.

Fixed more problems issued by PC-lint.

Message aesthetical improvement

Removed the address resolution from the pinger.
- There is a special  class to translate DNS to IP, thus makes no sense to keep this feature here, which simplify this class.

Fixed problems issued by PC-lint.

Included the unit in the variable names
- states clear that we expect seconds, minutes or whatever

Implemented the requirement "If we notified an external system that a link is down, also notify it if the link is back up (=stable for a configurable period of time)"
- the status of the link is reported only once, when it changes from down to up, or vice versa
- allows to configure the minimum amount of time the link must stays stable in order to report its status

Found another BUG (its capitalization gives a hint about the severity of this one)

Standardizing the enumerations

Renamed stable-link-up-interval to link-up-interval

Fixed return value, should be string, not boolean

Included boost system to the required libraries

Renamed StatusNotifier to LinkStatusAnalyzer
- the main purpose of this class is to decide whether the link is down or up, by keeping track the amount of hosts down.
- renamed the variable Analyzer to HostAnalyzer to avoid confusion with the LinkAnalyzer

Renamed PingAnalyzer to HostStatusAnalyzer
- this describes better what the class is suppode to do, analyze the status of the host, using the pings.

Add TODO entry

Read the configured interval that a link must be up to notify

Placed limit qualifier in the end of variables names (i.e. xxx_limit instead of limit_xxx)
- this keeps a pleasing symmetry with other qualifiers, for example xxx_count, xxx_limit, xxx_index

Renamed PingAnalyzer's update_ping_failed_statistics() to analyze_ping_failed_count()

Improved asserts

Implemented the requirement "Intelligently retry if a host doesn't reply: Try it more often"
- PingAnalyzer::exceeded_ping_failed_count_limit() made public, thus the Analyzer can check when is required to speed up the ping (or back to the original interval)
- Included a PingInterval template class to hide the computation logic of interval speed up and slow down

Attributes names according to the company's coding style.

Renamed PingStatusNotifier to StatusNotifier
- the class scope is not restricted to ping status notification, it is
  responsible to notify the system when certain amount of hosts are down,
  independently of the number of pings given to that host.

Included script to notify when system is down or up

Added the requirement "If XX servers don't reply, it has to notify another system (e.g. execute a script)"
- moved the class PingStatusNotifier from ping to notify directory.
- included a new class that encapsulate the process of system notification

Linking against libi2ncommon

Renamed PingStatusNotifier::alert_host_up/down() methods to notify_host_up/down()

Added intelligence to know how many hosts are down

Const-ification

Reorganization of cmake files
- included package build
- layout change (don't know why package system was complaining about white spaces in the commands)
- redistribuition of the commands across the files

Included spec file to build the RPM package

Removed UML file, because it is out-dated and is not longer required.

Removed Pinger abstract class, thus avoiding the "design ahead" issue
- BoostPinger do not inherit form Pinger anymore
- reference from Code Complete 2nd edition, page 146, item "Be suspicious of base classes of which there is only one derived class"

Changed PingStatusNotifier interface, it is now an object shared among N hosts
- this shared property allows to know how many hosts are down, once we can keep track of all hosts
- only one PingStatusNotifier instance for all PingAnalyzers objects, they pass the host to identify themselves within the notifier object

Added status-notifier-cmd options to indicate which command to call when the system is down/up.

The limit-ping-fail option is available (and read) from the configuration file
- this option sets the threshold of pings to a host that must fail in order to mark the host as down.

Renamed the limit-to-notify configuration option to limit-hosts-down, because it is a better description of the intention.

Replaced uint by int.
- documented that a integer is non-negative using assertions, not unsigned int (uint).

Handle the error when a host cannot be resolved
- reorganization of PingScheduler to do not ping a host that was not resolved
- from now, the DnsResolver::resolve() method must be explicitly called by the client of the class

Errors flushed through cerr not cout.

Included an invalid host and a host that does not reply.

More bugs discovered and annotated

Uses the ping analyzer to evaluate the ping statistics after each ping.

Returns the amount of IPs that were resolved for the DNSResolver host.

Returns a boolean indicating if the ping was successful or not.

Bring aboard the ping analyze and notification classes.
- PingAnalyzer implements the requirement to analyze if there are multiple dns entries for a host, only call this host "failed" if can't reach any of these multiple entries.
- PingStatusNotifier implements the requirement to notify an external system that a link is down, also notify it if the link is back up.But needs improvements.

Replaced the times-to-ping parameter to simplify the code, now it pings just one time and uses a status on/off approach to check if the ping was performed.
- also BoostPinger::create_echo_request() method is const and self sufficient, use parameter instead of changing the SequenceNumber member variable

Re-organizing the headers files in order to...
- avoid hidden dependencies, by including the class header file first.
- keep a inclusion order of C headers, C++ headers, Boost headers and Project headers.
- be relative to the source directory, this keeps clear which package the source file belongs.

Explicit constructor to avoid implicit constructor-conversions-calls

PingScheduler now pings hosts IPs cyclically
- included a DNS resolver member variable to keep track of resolved IPs
- PingScheduler's constructor takes specific arguments instead of a generic Host object, this states clear which data the PingScheduler needs

Bring aboard the first scratch of a DNS resolver
- given a host DNS, this class resolves and keeps a list of IPs for that host

Aesthetic change to header files according to the project standard

Code improvements on BoostPinger
- included an assert to make sure a non-empty address is sent to the socket
- better names for the handlers methods

Moved Host class from ping directory to config directory, because it is a configuration item

Added TODO file, including known bugs report

More descriptions of the configuration in the Readme file

The echo reply timeout can be configured through parameter to BoostPinger

Renamed PingCheck class to PingScheduler.
- this name describes better the main purpose behind this class, which is to call the pinger in configured intervals

The application now pings many hosts in parallel
- uses an asynchronous approach, based on boost::asio::io_service::run() loop and asynchronous timers

Attribute renamed according to the code conventions

Included a section explaining the basics of the code

Code improvements
- stratified some functionalities into other methods;
- const-correctness
- added variables to clarify code

Removed unused class: PingManager

Removed unused header files

Fixed assertion range on times_to_ping

Changed integer types from std::size_t to uint, where it is required ordinary numbers

More information in the Readme file

The app reads a list of hosts instead of just one
Caveat: it pings the host sequentially, this is not the desirable behavior

Removed all namespace's "using" from header files (check C++ Coding Standards #59)
- writing headers with full namespace qualifiers to avoid type collision
- writing "using" only in implementation files to save namespace qualifier writing

Renamed config directory to conf, in order to avoid confusion with the directory name inside src

Moved ICMP abstraction classes to the icmp directory, leaving the ping directory only for ping classes

Moved PingCheck class to its own file, replaced the old one with a new semantic.

The program now pings in configured intervals
- the code is functional, but requires improvements!

Organizing the member functions according to the IPv4 header description

Avoiding unamed variables (they kill readability)

Pinger interface now allows to specify the number of pings to perform

Defined a rule for variable and constant numeric types
- using std::size_t for ordinary integers, those that do not impose size constraints
- using uint32_t, uint16_t or uint_8_t for integers which require specific sizes

Created an ICMP packet class abstraction and modularizing the BoostPinger
- Hide the ICMP packet management from BoostPinger class
- Split functionality in other methods (e.g send_echo_request() and print_echo_reply()