From: Guilherme Maciel Ferreira <guilherme.maciel.ferreira@intra2net.com>
Date: Tue, 22 Feb 2011 14:59:30 +0000 (+0100)
Subject: Added assertions to document interface assumptions
X-Git-Tag: v1.0~193
X-Git-Url: http://developer.intra2net.com/git/?a=commitdiff_plain;h=8c3f445c00a248a3ddb2d17856fd9db3b0416ad6;p=pingcheck

Added assertions to document interface assumptions
---

diff --git a/src/config/configuration.cpp b/src/config/configuration.cpp
index 3b682d6..c378daf 100644
--- a/src/config/configuration.cpp
+++ b/src/config/configuration.cpp
@@ -1,3 +1,5 @@
+#include <boost/assert.hpp>
+
 #include "configuration.h"
 
 using namespace std;
@@ -25,6 +27,8 @@ string Configuration::get_config_file_name() const
 
 void Configuration::set_config_file_name( std::string config_file_name )
 {
+    BOOST_ASSERT( !config_file_name.empty() );
+
     this->config_file_name = config_file_name;
 }
 
@@ -35,5 +39,7 @@ uint32_t Configuration::get_limit_to_notify() const
 
 void Configuration::set_limit_to_notify( uint32_t limit_to_notify )
 {
+    BOOST_ASSERT( ( MIN_LIMIT_TO_NOTIFY <= limit_to_notify ) && ( limit_to_notify <= MAX_LIMIT_TO_NOTIFY) );
+
     this->limit_to_notify = limit_to_notify;
 }
diff --git a/src/config/configurationreader.cpp b/src/config/configurationreader.cpp
index 9fbaea1..9dc0365 100644
--- a/src/config/configurationreader.cpp
+++ b/src/config/configurationreader.cpp
@@ -1,3 +1,4 @@
+#include <boost/assert.hpp>
 #include <fstream>
 #include <iostream>
 #include <iterator>
@@ -114,6 +115,8 @@ bool ConfigurationReader::parse_configuration_file(
         variables_map& vm
 )
 {
+    BOOST_ASSERT( !config_file_name.empty() );
+
     ifstream ifs( config_file_name.c_str() );
     if ( !ifs )
     {
@@ -138,6 +141,9 @@ bool ConfigurationReader::parse(
         char* argv[]
 )
 {
+    BOOST_ASSERT( argc > 1 );
+    BOOST_ASSERT( argv != NULL );
+
     variables_map vm;
     bool command_line_parsed = parse_command_line( argc, argv, vm );
     bool configuration_file_parsed = parse_configuration_file( vm );