Refactoring

src/LargeFileStream.cpp

@@ -19,7 +19,7 @@ void LargeFileStream::FromString(std::string content) {
 
 void LargeFileStream::ReadNextChunk() {
   chuck_count++;
-  file.read(buffer, BUFFER_SIZE);
+  file.read(buffer, FILE_BUFFER_SIZE);
   last_read_size = file.gcount();
 }
 
@@ -42,11 +42,11 @@ char LargeFileStream::operator[](loctype loc) {
   }
 
   // Goto the right memory chuck
-  loctype loc_chunk_count = loc / BUFFER_SIZE;
+  loctype loc_chunk_count = loc / FILE_BUFFER_SIZE;
   while (chuck_count < loc_chunk_count) {
     ReadNextChunk();
   }
-  loctype offset = loc - (loc_chunk_count * BUFFER_SIZE);
+  loctype offset = loc - (loc_chunk_count * FILE_BUFFER_SIZE);
   
   // Ensure for EOF
   if (!file && offset >= last_read_size) {

src/LargeFileStream.hpp

@@ -5,8 +5,6 @@
 #include <iostream>
 #include <string>
 
-#define BUFFER_SIZE (1024 * 1024 / 2)
-
 namespace pgnp {
 using namespace std;
 
@@ -14,7 +12,7 @@ class LargeFileStream {
   /// @brief File to load
   ifstream file;
   /// @brief In memory buffer
-  char buffer[BUFFER_SIZE];
+  char buffer[FILE_BUFFER_SIZE];
   /// @brief Number of chuck read minus 1
   loctype chuck_count;
   /// @brief Number of byte read during the last file access

src/Types.hpp

@@ -1,7 +1,10 @@
 #pragma once
 
+// The corner stone of memory usage
+#define FILE_BUFFER_SIZE (1024 * 1024 / 2)
+
 namespace pgnp {
 typedef unsigned long long ull;
 typedef unsigned int uint;
 typedef ull loctype; // Choose location pointer type
-}
+} // namespace pgnp