commit 6832a7a4e04c84192f0bafc6ac990fc79cd13882
author Narayan Kamath <narayan@google.com> Mon Feb 23 15:43:35 2015 +0000
committer Narayan Kamath <narayan@google.com> Mon Feb 23 15:49:43 2015 +0000
tree e8c0940ae4a25e96db0ad84985991731f2b1b8f4
parent f1ac6917da172dd3e6829bac41fcbf23e96da079

Remove useless refCounting from FileMap.

Nobody ever called acquire() so release() was always
equivalent to delete. Just use delete instead so that
people can use unique_ptr directly (or shared_ptr if
they really want refcounts).

Change-Id: I9e3ad5e0f6a4fcc4e02e5a2ff7ef9514fe234415

libutils/Tokenizer.cpp

diff --git a/libutils/Tokenizer.cpp b/libutils/Tokenizer.cpp
index 7067533..610002f 100644
--- a/libutils/Tokenizer.cpp
+++ b/libutils/Tokenizer.cpp
@@ -43,9 +43,7 @@
 }
 
 Tokenizer::~Tokenizer() {
-    if (mFileMap) {
-        mFileMap->release();
-    }
+    delete mFileMap;
     if (mOwnBuffer) {
         delete[] mBuffer;
     }
@@ -74,7 +72,7 @@
                 fileMap->advise(FileMap::SEQUENTIAL);
                 buffer = static_cast<char*>(fileMap->getDataPtr());
             } else {
-                fileMap->release();
+                delete fileMap;
                 fileMap = NULL;
 
                 // Fall back to reading into a buffer since we can't mmap files in sysfs.