PC-Lint warnings fixed:
- Info 713: Loss of precision (initialization) (unsigned int to int)

PC-Lint warnings fixed:
- Warning 568: non-negative quantity is never less than zero.

PC-Lint warnings fixed:
- Info 731: Boolean argument to equal/not equal;
- Info 866: Unusual use of '?' in argument to sizeof;
- Info 751: local typedef 'boost_static_assert_typedef_x' (line x) not referenced.

PC-Lint warnings fixed:
- Info 713: Loss of precision (arg. no. 1) (unsigned int to int);
- Info 701: Shift left of signed quantity (int).

PC-Lint warnings fixed:
- Warning 1551: Function may throw exception '...' in destructor 'NetworkInterfaceList::~NetworkInterfaceList(void)';
- Info 1775: catch block does not catch any declared exception.

Changed notify message to error message.

Make explicit that the address from ICMPv6 can not be retrieved.

Fix: handling packets with options length equal to zero.

Fix: the TCP's ttl was being printed as a char instead of as a integer.

PC-Lint warnings fixed:
- Info 737: Loss of sign in promotion from int to unsigned int;
- Info 713: Loss of precision (arg. no. 2) (unsigned int to int) in 'is.read( ..., PayloadSizeInBytes )' and 'os.write( ..., PayloadSizeInBytes )'.

PC-Lint warnings fixed:
- Info 1736: Redundant access specifier (private);

PC-Lint warnings fixed:
- Warning 1551: Function may throw exception '...' in destructor 'NetworkInterfaceList::~NetworkInterfaceList(void)';
- Info 774: Boolean within 'if' always evaluates to True.

PC-Lint warnings fixed:
- Info 1786: Implicit conversion to Boolean (return) (int to bool);
- Info 1705: static class members may be accessed by the scoping operator;
- Info 713: Loss of precision (arg. no. 2) (unsigned int to int);
- Info 737: Loss of sign in promotion from int to unsigned int in statement 'size -= sizeof(word_array[ 0 ])';
- Info 713: Loss of precision (assignment) (unsigned int to int) in statement 'size -= sizeof(word_array[ 0 ])';
- Info 1773: Attempt to cast away const (or volatile) in statement 'cksum += *(uint8_t*) word_array';
- Replaced all C-style casts by C++ style casts.

PC-Lint warnings fixed:
- Info 1705: static class members may be accessed by the scoping operator;
- Info 732: Loss of sign (arg. no. 1) (int to unsigned int) in 'byte = Payload[ header_byte_index ]';
- Warning 534: Ignoring return value of function 'MessagePayload::read(...)'.

PC-Lint warnings fixed:
- Info 766: Header file 'limits' not used in module;
- Warning 534: Ignoring return value of functions 'MessagePayload::read(...)' and 'std::basic_istream<char>::read(char *, int)';
- Info 737: Loss of sign in promotion from int to unsigned int;
- Info 713: Loss of precision (initialization) (unsigned int to int);
- Info 732: Loss of sign (arg. no. 2) (int to unsigned int) in 'header.Payload.append( options_data.get(), options_length )'.

PC-Lint warnings fixed:
- Info 734: Loss of precision (arg. no. 1) (9 bits to 8 bits) in src\icmp\icmppacketfactory.cpp:170;
- Info 788: enum constant 'Icmpv6Type::Icmpv6Type_First' not used within defaulted switch;

Made copy constructor private to disable object copies.

PC-Lint warnings fixed:
- Warning 574: Signed-unsigned mix with relational;
- Info 737: Loss of sign in promotion from int to unsigned int;
- Warning 534: Ignoring return value of function 'IcmpPacket/TcpSegment::write(std::basic_ostream<char> &) const';
- Info 731: Boolean argument to equal/not equal;

PC-Lint warnings fixed:
- Warning 574: Signed-unsigned mix with relational;
- Info 737: Loss of sign in promotion from int to unsigned int;

Changed 'ttl' field from signed to unsigned, avoiding loss of precision.

PC-Lint warnings fixed:
- Info 713: Loss of precision (initialization) (unsigned int to int) of 'resolved_ip_count' and 'current_ttl';
- Info 1762: Member function 'DnsResolver::fill_answers_list(...)' could be made const;
- Info 1762: Member function 'DnsResolver::fill_resolved_ip_list(...)' could be made const;
- Warning 578: Declaration of symbol 'ip' hides symbol 'boost::asio::ip';
- Warning 613: Possible use of null pointer 'a_rr' in left argument to operator '->';

PC-Lint warnings fixed:
- Warning 1529: Symbol 'Class::operator=(const Class &)' not first checking for assignment to this

PC-Lint warnings fixed:
- Info 732: Loss of sign (arg. no. 1) (int to unsigned int)

Final adjustments for cleaner PC-lint output

Silence more bogus PC-Lint warnings

Use return value of the function instead of a pointer to the target.

C++ will automatically create a copy of the object upon return.

Silence more PC-Lint warnings

Ignore unused parameter

More constification

Hide "xyz could be made const" linter message
for modifications done through a boost::shared_ptr.
(Pc-Lint can't detect those properly).

Hide one warning emitted by BOOST_FOREACH

Constification

Fix type: Boost::shared_ptr supports conversion to bool but not to char*

Add script for easier Pc-Lint invocation

Add include directories to PC-lint config

Changes introduced in the ipv6 branch.

Message to require sudo access to open raw sockets.

The IP version used to transport TCP segments can be choosen, IPv4 or IPv6.

Changed old-style C casts by C++ style casts.

TCP checksum for IPv6

Documentation for TCP Segment Factory.

Small documentation change in IP header classes: specifying the unit.

TODO: move DNS library to its own project.

Some grammatical corrections.

Fixed include guards name

Bring aboard ChangeLog file.

Documentation for ICMP Packet Factory.

Just hardening the code with asserts.

Bug: the match() method cannot consider the source address from IP header,
once the IPHeader object is not initialized in the read() method.
We must figure out why the IPv6 data does not come in the istream like IPv4 data.

It is working, and can be configurated, the ICMP ping over IPv6

Small documentation change

Small code change

Documentation for IPv6 header.

TCP checksum source and destination addresses use Boost.Asio objects instead of primitive types

TCP pinger IP version is chosen by the pinger factory

Bring aboard TCP header class for IPv6

Updated "Type of Service" field name to "Differentiated Services"

Using Boost.Asio address objects instead of primitive integer types
- This allows switching to 128 bits IPv6 address

Simplifying Network Interface class and using more Boost.Asio objects

Using Boost.Asio protocol type instead of the enumeration for TCP
- Also, changed the IP address to Boost type

Bring aboard a class that encapsulate the list of network interfaces.

Adjusting some comments

New method to get the network interface IP address
- Using Boost.Asio objects instead of a primitive data type (under development).

Using Boost.Asio protocol type instead of the enumeration and adding ICMPv6 ping

Delegating the segment creation and printing to more specialized classes

Ephemeral source port calculation moved to a method(), mirroring the destination port.

Renamed variable, most significant noun first

The IcmpPacket's write() method returns a boolean, mirroring the read() method.

Bring aboard TCP segment factory.
- This class handles the TCP segment creation for IPv4 and IPv6

Bring aboard TCP segment (IP and TCP headers)
- This allows to polymorphically change IPv4 and IPv6

Documentation for IPv4 header.

High Integrity CPP Rule 3.3.14: Declare the copy assignment operator protected in an abstract class.

High Integrity CPP Rule 3.3.16: Explicitly declare polymorphic member functions virtual in a derived class.

High Integrity CPP Guideline 3.2.4: An abstract class shall have no public constructors.

Avoid 'unused variable' warning when compiling a non-debug version.

Delegating the ICMP packet creation to a specialized class and using a polymorphic packet type

Bring aboard ICMP packet factory.
- This class handles the packet creation for ICMPv4 and ICMPv6

Providing read() and write() methods, once operators >> and << does work properly with smart pointers

Renamed methods, changed verb 'is' to 'match'

The packet's type check is performed by methods in IcmpPacket
- Because ICMP v4 and v6 have different values to echo reply

The packet printing is polymorphically performed by the IcmpPackets instead of the IcmpPinger
- Due to different fields, the printing methods were moved to the packet classes

Changed assert(false) to assert(!"message") as recommended by C++ Coding Standards, chapter 68.

Increased the assertion check required to construct a PingScheduler object

DNS resolver now detects IPv6 IPs too.
- Also changed the detection mechanism from C fashioned to C++ boost style

Added const qualifier to some parameters

Bring aboard ICMPv6 packet class

Bring aboard ICMPv6 header class

Providing different methods to set the types in ICMP v4 and v6
- it was not provided a polymorphic approach

Added ICMPv6 messages types, as defined at iana.org/assignments/icmpv6-parameters

Renamed IcmpType enumeration to Icmpv4Type, because ICMP type values differ between v4 and v6 of the protocol

Updated classes comments

Renamed IcmpPacket class to Icmpv4Packet, in order to distinguish from the ICMPv6 classes

Renamed IcmpHeader class to Icmpv4Header, in order to distinguish from ICMP v6

Small documentation change

Bring aboard IPv6 header class (not tested yet)

Solved operator precedence mismatch reported by PC-lint, where the '|' has higher precedence than ternary '?:'
- Warning 514: Unusual use of a Boolean expression

Implementing the assignment operator, also checking for self-assignment reported by PC-lint
- Warning 1529: Symbol 'MessagePayload::operator=(const MessagePayload &)' not first checking for assignment to this
- Warning 1539: member 'MessagePayload::PayloadSizeInBytes' not assigned by assignment operator
- Warning 1539: member 'MessagePayload::Payload' not assigned by assignment operator

Solving two issues reported by PC-lint
- Ignoring return value of function 'sigfillset(__sigset_t *)'
- Declaration of symbol 'signal' hides symbol 'signal(int, void (*)(int))'

Renamed parameter to avoid hiding symbol 'socket(int, int, int)' (issued by PC-lint)

Added a note about a TODO fix

Fixed broken assert in copy constructor
- The Payload member was initialized in the initializer list, so asserting Payload.get() == NULL was wrong!