Using a more straightforward comparison

Parameter is passed by reference, not by value

Reducing scope of variable

Ensure that there are always more hosts addresses than any other option
- in other works, it is not allowed to have [host] without "name" option
- here is the assertions, but the treatment must be provided in a higher layer

Asserting for the hosts items quantity match

Small code improvements

Adding the options classes to the Makefile

Bring aboard hosts section configuration option classes.

More descriptive variable name, also fixing a bug which cause the program to halt with the wrong arguments

parse method is not const, because it can change the internal state of the objects

Update version number scheme documentation

Bug fix: return the boost::program_options::option_description by value
- If this object is returned by reference or smart pointer, the
  boost::program_options::options_description::add() method will delete it
  causing double free on that object. So we prevent any problem by not exposing
  the underlying object

Grammatical corrections on Readme.code file

Added comments to Configuration class

Bring aboard "hosts-down-limit" configuration option class.

Bring aboard "link-down-interval" configuration option class.

Bring aboard "link-up-interval" configuration option class.

Bring aboard "nameserver" configuration option class.

Bring aboard "ping-fail-limit" configuration option class.

Bring aboard "source-network-interface" configuration option class.

Bring aboard "status-notifier-cmd" configuration option class.

Bring aboard an abstract class for configuration options.
- The subclasses shall reduce the size and complexity from
  ConfigurationOptions class, spliting its functionality among many classes

Changed include guards style to the project's standard

New TODO entries: goals for config branch

Make the ping-protocol option case insensitive

Parsing only if could process the file, now it is simetric with the command line code

Improved Host documentation

Changed ping protocol to a "per host" configuration, instead of one for all hosts

Including configuration classes in makefile

Split configuration responsability between different classes

Bring aboard classes to handle each kind of configuration interface

Created a class to hold configuration options and parsing inteligence
- it does not know about where the configuration came from

Updated Readme file with the TCP ping feature

Fixed postconditions, the equal had wrong pointer interator

Replced old-style C functions by C++ STL versions

Merge branch 'tcpping'

Conflicts:
src/icmp/icmppinger.cpp

Replaced the raw byte array by a MessagePayload object, this class provides methods to handle decoding of network words

Allows to add more data to the MessagePayload through append method.
Also fixed the read and write documentation.

Small code formatting

Pings the host at a specific port (available only to TCP ping)

Reads the host port from configuration file

Changed the type of the header size variables from int to size_t, so it matches the type where it is used

The class variable HeaderSizeInBytes was renamed to Ipv4HeaderSizeInBytes and made global only within file scope.

TcpHeader class uses the MessagePayload class to handle network byte handling

Implemented a method to access the inner array pointer, this is necessary for memory copies.

Fixing some asserts with wrong ranges and reworking some comments

Removed select_source_network_interface() function, replace by simple call to NetworkInterface::bind()

The C's low level socket handling functions were encapsulated in the NetworkInterface class

Bring aboard the Network Interface template class.
- It is a template because it uses the generic socket type, to avoid inheritances.
- This template class hides much of the low level socket handling.

Renamed IcmpChecksumCalculator to IcmpChecksum

Renamed ChecksumCalculation template to Checksum
- The name represents better an object
- Also changed the file extension from .h to .hpp, so keep templates with .hpp extension and classes with .h

Allowing to copy the MessagePayload object, through implementation of Copy Constructor and Assignment Operator

Added the 32 bits version of encode and decode methods

Small code formatting

Asserted the encode16() and decode16() parameters, warning about off-by-one errors

Renamed IcmpMessagePayload to MessagePayload, so it can be used by other messages types

Renamed getters and setters

Moved PingStatus enumeration to its own file, so it can be reused by both Pingers

The Include Guards must match the file's name

Surrounded the receive by a try..catch, so prepare to receive another packet only if the reply isn't a RST

Removing warning about conversion from 'int' to 'uint8_t'

Adding a remark about sequence number from RST

Adding logging to ioctl fails, and replacing the nested if statements by guard clauses (early returns)

Replaced shift operations on IP addresses to an endian-safer approach
- Instead of shifts on sockaddr.sa_data, uses a cast to sockaddr_in.sin_addr.s_addr.
  And as long as the two structures have the same memory layout, the cast is safe.
- Reinterpret four chars (8 bits) as one integer (32 bits), and make it endian safe.

Validates the ifr.ifr_name buffer size before copy (Thomas's advice)

Pinger factory provides only static (class) methods, thus do not require public constructor and destructor.

Set the received status as successful when the RST match what was expected.

Little code format detail

Placed start_receive() method closer to the schedule_timeout_echo_reply(), where it gets called.

Changed the handle_timeout() method to handle_ping_done()
- now it calls the ping call back function
- and do not schedule the next ping anymore, let it to the callback responsability

Renamed create_tcp_header() method to create_ack_request()

Move the ACK segment send functionality to the send_ack_request() method

Split the method, separated some functionality in schedule_timeout_rst_reply()

Renamed timeout variable to the context of TCP instead of ICMP

Sequence number does not identify a particular segment, it is always the same

Missing header files and identation

Retrieve the local IP address from the interface using the low level ioctl, plus some code enhancements

Bring aboard Doxygen documentation support

Added and formatted some error messages

Split methods
- create_tcp_header() to create the TCP header for ACK ping
- print_rst_reply() to print the TCP header for RST reply

Cosmetic changes
- improved variable names
- added "Warning" to some logs

Removed commentted code

Formatting log messages

Working TCP pinger
- Needs to fix the source IP address used in the pseudo IP header

Split the TcpHeader implementation and header

Removed the unused get_identifier() method

Small code formatting

Fixed socket typedef, it is a raw socket, not stream socket

Added the default protocol to use in the config file

Better comment description

Fixed the return statement, the method has the return statement in the correct place

Fully functional pinger factory

Added the remaining TCP pinger arguments
- still not working, but the skeleton is done

Code formatting

Solving some conversion and variable shadow warnings

Adding the ping protocol configuration option

Moving the PingProtocol enumeration from PingerFactory to its own file
- Added a method to convert from string to the enumeration

The "using namespace std" exempt the use of "std::" qualifier

Note to myself: rework the checksum to use it also in the TCP stuff

Fixed include guards name