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()

The method BoostPinger::get_identifier() is not static anymore, this allows to have more than one BoostPinger object at the same time

Renamed IcmpBody to IcmpData according to the RFC 792 (Echo or Echo Reply Message section)

Changed the ping() method interface, which requires just a destination string instead of a whole Host object

The BoostPinger now pings N times, instead of infinite pinging
- moved the io_service calls to inside the BoostPinger
- splited the BoostPinger::start_send(), the code responsible to send is inside BoostPinger::send()

Renamed the ipv4_header files to ipv4header, removed the '_' to keep the project standard

Renamed the ipv4_header files to ipv4header, removed the '_' to keep the project standard

Read the interval between each ping to the host

Refactored the checksum calculation and IcmpHeader class
- Included the checksum in the IcmpHeader constructor
- Created a type for ICMP package body, namely Icmpbody
- Transformed the compute_checksum template function into a template class, ChecksumCalculator, and defined an specialization type, namely IcmpChecksumCalculator

Renamed the icmp_header files to icmpheader, removed the '_' to keep the project standard

Added a constructor with arguments and reinforce the "ICMP type" variable's type

The header classes operate on specific data sizes, thus using types with named sizes on it states clear which size of the data is expected

Keep together the getters and setters who operate on the same data, this is the project's standard

Changed getter and setter methods names to keep standardized (and verb+object advice from chapter 7 of code complete)

Reorganization of the ConfigurationReader class interface
- Sticking methods together
- Providing more descriptive name to some methods
- Adding new methods to encapsulate some functionalities

Executable has the same name as the project

Moved boost library search to the main CMakeLists.txt file, so cmake knows right from the beginning which libraries are required

Renamed variables and constants to stick with company's coding style

Reading host name from file. This first attempt pings just one host

Improved Host interface by including const parameters and assertions

Config file clean up (KISS)

Source code ornamentation

Added assertions to document interface assumptions

Improved configuration classes abstraction and source code
- Replaced magic strings by named constants
- Renamed ConfigurationReader::read() to ConfigurationReader::parse() according to the interface provided by other methods
- Added a method to fill the Configuration object
- Sorry for sending too many changes in the same commit =(

Project name #define is sent to source code in compile time

Included the default configuration file