Fixed Issue #24 (assert statements)

The C assert.h header was erroneously included within a namespace
in the NanoLog library, causing compilation errors for outside users.
This commit fixes the issue by moving the include statement.

Author: syang0

runtime/Common.h

@@ -13,11 +13,7 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
-// Unfortunately, unit tests based on gtest can't access private members
-// of classes.  If the following uppercase versions of "private" and
-// "protected" are used instead, it works around the problem:  when
-// compiling unit test files (anything that includes TestUtil.h)
-// everything becomes public.
+#include <cassert>
 
 #ifndef COMMON_H
 #define COMMON_H
@@ -27,6 +23,12 @@
 
 namespace NanoLogInternal {
 
+// Unfortunately, unit tests based on gtest can't access private members
+// of classes.  If the following uppercase versions of "private" and
+// "protected" are used instead, it works around the problem:  when
+// compiling unit test files (anything that includes TestUtil.h)
+// everything becomes public.
+
 #ifdef EXPOSE_PRIVATES
 #define PRIVATE public
 #define PROTECTED public
@@ -44,7 +46,6 @@ namespace NanoLogInternal {
     TypeName& operator=(const TypeName&) = delete;
 #endif
 
-#include <cassert>
 /**
  * Cast one size of int down to another one.
  * Asserts that no precision is lost at runtime.

runtime/Log.h

@@ -16,7 +16,7 @@
 #include <ctime>
 #include <vector>
 
-#include <assert.h>
+#include <cassert>
 #include <stdio.h>
 
 #include "Config.h"

runtime/PackerTest.cc

@@ -13,9 +13,9 @@
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
 
-#include <assert.h>
+#include <cassert>
 #include <cstdlib>
-#include <stdio.h>
+#include <cstdio>
 #include <fstream>
 
 #include "TestUtil.h"

runtime/Util.h

@@ -19,7 +19,7 @@
 #include <time.h>
 #include <stdint.h>
 #include <sched.h>
-#include <assert.h>
+#include <cassert>
 #include <sys/syscall.h>
 #include <unistd.h>
 #include <sys/ioctl.h>