Rework blobstamper code. We do not need both size and end count. So leave only one (and it is end that will be left)

Author: nataraj-hates-MS-for-stealing-github

blobstamper/blob.cpp

@@ -27,9 +27,8 @@
 Blob::Blob (char * data_in, size_t size_in)
 {
     data = data_in;
-    size = size_in;
     begin = 0;
-    end = size -1; /* i.e. size=1 means begin=0 && end=0 */
+    end = size_in - 1; /* i.e. size=1 means begin=0 && end=0 */
 }
 
 bool
@@ -77,7 +76,7 @@ Blob::Chop(size_t min_size, size_t max_size)
 std::vector<char>
 Blob::AsByteVector()
 {
-    std::vector<char> res(data + begin, data + begin + size);
+    std::vector<char> res(data + begin, data + end + 1);
     return res;
 }
 
@@ -100,8 +99,6 @@ std::vector<char>
 Blob::asVector()
 {
   std::vector<char> res( (char *)data + begin, (char*)data + begin + Size());
-
-//  memcpy(&res[0], data + begin, Size());
   return res;
 }
 

blobstamper/blob.h

@@ -29,9 +29,8 @@ class StampBase;
 
 class Blob
 {
-    protected:
+    private:
         char*  data;
-        size_t    size;
         size_t    begin;
         size_t    end;
     public:
@@ -49,12 +48,12 @@ class Blob
         std::string asString(); /* Should not be used in prod, for tests and examples only*/
 };
 
-class OutOfData  /*An exeption. Experemental for now*/
+class OutOfData  /*An exception. Experimental for now*/
 {
 
 };
 
-class NotImplemented  /*An exeption */
+class NotImplemented  /*An exception */
 {
 
 };