Removed all sign warnings

gineshidalgo99 · gineshidalgo99 · commit 25d021e74bb6 · 2017-09-18T11:48:12.000-04:00
diff --git a/3rdparty/caffe/include/caffe/blob.hpp b/3rdparty/caffe/include/caffe/blob.hpp
@@ -53,7 +53,7 @@ class Blob {
   void ReshapeLike(const Blob& other);
   inline string shape_string() const {
     ostringstream stream;
-    for (int i = 0; i < shape_.size(); ++i) {
+    for (auto i = 0u; i < shape_.size(); ++i) {
       stream << shape_[i] << " ";
     }
     stream << "(" << count_ << ")";
diff --git a/Makefile b/Makefile
@@ -169,7 +169,10 @@ ifeq ($(USE_OPENCV), 1)
 	endif
 
 endif
-WARNINGS := -Wall -Wno-sign-compare
+##############################
+# OpenPose extra code: commented
+##############################
+# WARNINGS := -Wall -Wno-sign-compare
 
 ##############################
 # Set build directories
diff --git a/doc/release_notes.md b/doc/release_notes.md
@@ -105,5 +105,6 @@ OpenPose Library - Release Notes
 ## Current version (future OpenPose 1.0.3)
 1. Main improvements:
     1. Added how to use keypoint data in `examples/tutorial_wrapper/`.
+    2. Added flag for warnings of type `-Wsign-compare` and removed in code.
 2. Main bugs fixed:
     1. Windows version crashing with std::map copy.
diff --git a/examples/tests/wrapperHandFromJsonTest.hpp b/examples/tests/wrapperHandFromJsonTest.hpp
@@ -185,7 +185,7 @@ namespace op
             {
                 spWPoses.resize(gpuNumber);
                 const auto handDetector = std::make_shared<HandDetectorFromTxt>(handGroundTruth);
-                for (auto gpuId = 0; gpuId < spWPoses.size(); gpuId++)
+                for (auto gpuId = 0u; gpuId < spWPoses.size(); gpuId++)
                 {
                     // Hand detector
                     // If tracking
diff --git a/include/openpose/filestream/wFaceSaver.hpp b/include/openpose/filestream/wFaceSaver.hpp
@@ -59,7 +59,7 @@ namespace op
                 auto& tDatumsNoPtr = *tDatums;
                 // Record people face keypoint data
                 std::vector<Array<float>> keypointVector(tDatumsNoPtr.size());
-                for (auto i = 0; i < tDatumsNoPtr.size(); i++)
+                for (auto i = 0u; i < tDatumsNoPtr.size(); i++)
                     keypointVector[i] = tDatumsNoPtr[i].faceKeypoints;
                 const auto fileName = (!tDatumsNoPtr[0].name.empty() ? tDatumsNoPtr[0].name : std::to_string(tDatumsNoPtr[0].id));
                 spKeypointSaver->saveKeypoints(keypointVector, fileName, "face");
diff --git a/include/openpose/filestream/wHandSaver.hpp b/include/openpose/filestream/wHandSaver.hpp
@@ -61,11 +61,11 @@ namespace op
                 const auto fileName = (!tDatumsNoPtr[0].name.empty() ? tDatumsNoPtr[0].name : std::to_string(tDatumsNoPtr[0].id));
                 std::vector<Array<float>> keypointVector(tDatumsNoPtr.size());
                 // Left hand
-                for (auto i = 0; i < tDatumsNoPtr.size(); i++)
+                for (auto i = 0u; i < tDatumsNoPtr.size(); i++)
                     keypointVector[i] = tDatumsNoPtr[i].handKeypoints[0];
                 spKeypointSaver->saveKeypoints(keypointVector, fileName, "hand_left");
                 // Right hand
-                for (auto i = 0; i < tDatumsNoPtr.size(); i++)
+                for (auto i = 0u; i < tDatumsNoPtr.size(); i++)
                     keypointVector[i] = tDatumsNoPtr[i].handKeypoints[1];
                 spKeypointSaver->saveKeypoints(keypointVector, fileName, "hand_right");
                 // Profiling speed
diff --git a/include/openpose/filestream/wHeatMapSaver.hpp b/include/openpose/filestream/wHeatMapSaver.hpp
@@ -58,7 +58,7 @@ namespace op
                 auto& tDatumsNoPtr = *tDatums;
                 // Record image(s) on disk
                 std::vector<Array<float>> poseHeatMaps(tDatumsNoPtr.size());
-                for (auto i = 0; i < tDatumsNoPtr.size(); i++)
+                for (auto i = 0u; i < tDatumsNoPtr.size(); i++)
                     poseHeatMaps[i] = tDatumsNoPtr[i].poseHeatMaps;
                 const auto fileName = (!tDatumsNoPtr[0].name.empty() ? tDatumsNoPtr[0].name : std::to_string(tDatumsNoPtr[0].id));
                 spHeatMapSaver->saveHeatMaps(poseHeatMaps, fileName);
diff --git a/include/openpose/filestream/wImageSaver.hpp b/include/openpose/filestream/wImageSaver.hpp
@@ -58,7 +58,7 @@ namespace op
                 auto& tDatumsNoPtr = *tDatums;
                 // Record image(s) on disk
                 std::vector<cv::Mat> cvOutputDatas(tDatumsNoPtr.size());
-                for (auto i = 0; i < tDatumsNoPtr.size(); i++)
+                for (auto i = 0u; i < tDatumsNoPtr.size(); i++)
                     cvOutputDatas[i] = tDatumsNoPtr[i].cvOutputData;
                 const auto fileName = (!tDatumsNoPtr[0].name.empty() ? tDatumsNoPtr[0].name : std::to_string(tDatumsNoPtr[0].id));
                 spImageSaver->saveImages(cvOutputDatas, fileName);
diff --git a/include/openpose/filestream/wKeypointJsonSaver.hpp b/include/openpose/filestream/wKeypointJsonSaver.hpp
@@ -59,7 +59,7 @@ namespace op
                 const auto baseFileName = (!tDatumFirst.name.empty() ? tDatumFirst.name
                                             : std::to_string(tDatumFirst.id)) + "_keypoints";
                 const bool humanReadable = true;
-                for (auto i = 0 ; i < tDatums->size() ; i++)
+                for (auto i = 0u ; i < tDatums->size() ; i++)
                 {
                     const auto& tDatum = (*tDatums)[i];
                     // const auto fileName = baseFileName;
diff --git a/include/openpose/filestream/wPoseSaver.hpp b/include/openpose/filestream/wPoseSaver.hpp
@@ -59,7 +59,7 @@ namespace op
                 auto& tDatumsNoPtr = *tDatums;
                 // Record people pose keypoint data
                 std::vector<Array<float>> keypointVector(tDatumsNoPtr.size());
-                for (auto i = 0; i < tDatumsNoPtr.size(); i++)
+                for (auto i = 0u; i < tDatumsNoPtr.size(); i++)
                     keypointVector[i] = tDatumsNoPtr[i].poseKeypoints;
                 const auto fileName = (!tDatumsNoPtr[0].name.empty() ? tDatumsNoPtr[0].name : std::to_string(tDatumsNoPtr[0].id));
                 spKeypointSaver->saveKeypoints(keypointVector, fileName, "pose");
diff --git a/include/openpose/filestream/wVideoSaver.hpp b/include/openpose/filestream/wVideoSaver.hpp
@@ -58,7 +58,7 @@ namespace op
                 auto& tDatumsNoPtr = *tDatums;
                 // Record video(s)
                 std::vector<cv::Mat> cvOutputDatas(tDatumsNoPtr.size());
-                for (auto i = 0 ; i < cvOutputDatas.size() ; i++)
+                for (auto i = 0u ; i < cvOutputDatas.size() ; i++)
                     cvOutputDatas[i] = tDatumsNoPtr[i].cvOutputData;
                 spVideoSaver->write(cvOutputDatas);
                 // Profiling speed
diff --git a/include/openpose/thread/wQueueOrderer.hpp b/include/openpose/thread/wQueueOrderer.hpp
@@ -12,7 +12,7 @@ namespace op
     class WQueueOrderer : public Worker<TDatums>
     {
     public:
-        explicit WQueueOrderer(const int maxBufferSize = 64);
+        explicit WQueueOrderer(const unsigned int maxBufferSize = 64u);
 
         void initializationOnThread();
 
@@ -21,7 +21,7 @@ namespace op
         void tryStop();
 
     private:
-        const int mMaxBufferSize;
+        const unsigned int mMaxBufferSize;
         bool mStopWhenEmpty;
         unsigned long long mNextExpectedId;
         std::priority_queue<TDatums, std::vector<TDatums>, PointerContainerGreater<TDatums>> mPriorityQueueBuffer;
@@ -40,7 +40,7 @@ namespace op
 namespace op
 {
     template<typename TDatums>
-    WQueueOrderer<TDatums>::WQueueOrderer(const int maxBufferSize) :
+    WQueueOrderer<TDatums>::WQueueOrderer(const unsigned int maxBufferSize) :
         mMaxBufferSize{maxBufferSize},
         mStopWhenEmpty{false},
         mNextExpectedId{0}
@@ -99,7 +99,7 @@ namespace op
                 mNextExpectedId = tDatumsNoPtr[0].id + 1;
             }
             // Sleep if no new tDatums to either pop
-            if (!checkNoNullNorEmpty(tDatums) && mPriorityQueueBuffer.size() < mMaxBufferSize / 2)
+            if (!checkNoNullNorEmpty(tDatums) && mPriorityQueueBuffer.size() < mMaxBufferSize / 2u)
                 std::this_thread::sleep_for(std::chrono::milliseconds{1});
             // If TDatum popped and/or pushed
             if (profileSpeed || tDatums != nullptr)
diff --git a/include/openpose/wrapper/wrapper.hpp b/include/openpose/wrapper/wrapper.hpp
@@ -589,7 +589,7 @@ namespace op
                 // GPU rendering
                 if (renderOutputGpu)
                 {
-                    for (auto gpuId = 0; gpuId < poseExtractors.size(); gpuId++)
+                    for (auto gpuId = 0u; gpuId < poseExtractors.size(); gpuId++)
                     {
                         poseRenderers.emplace_back(std::make_shared<PoseRenderer>(
                             poseNetOutputSize, finalOutputSize, wrapperStructPose.poseModel, poseExtractors[gpuId],
@@ -621,14 +621,14 @@ namespace op
 
             // Pose extractor(s)
             spWPoses.resize(poseExtractors.size());
-            for (auto i = 0; i < spWPoses.size(); i++)
+            for (auto i = 0u; i < spWPoses.size(); i++)
                 spWPoses.at(i) = {std::make_shared<WPoseExtractor<TDatumsPtr>>(poseExtractors.at(i))};
 
             // Face extractor(s)
             if (wrapperStructFace.enable)
             {
                 const auto faceDetector = std::make_shared<FaceDetector>(wrapperStructPose.poseModel);
-                for (auto gpuId = 0; gpuId < spWPoses.size(); gpuId++)
+                for (auto gpuId = 0u; gpuId < spWPoses.size(); gpuId++)
                 {
                     // Face detector
                     spWPoses.at(gpuId).emplace_back(std::make_shared<WFaceDetector<TDatumsPtr>>(faceDetector));
@@ -645,7 +645,7 @@ namespace op
             if (wrapperStructHand.enable)
             {
                 const auto handDetector = std::make_shared<HandDetector>(wrapperStructPose.poseModel);
-                for (auto gpuId = 0; gpuId < spWPoses.size(); gpuId++)
+                for (auto gpuId = 0u; gpuId < spWPoses.size(); gpuId++)
                 {
                     // Hand detector
                     // If tracking
@@ -669,7 +669,7 @@ namespace op
 
             // Pose renderer(s)
             if (!poseRenderers.empty())
-                for (auto i = 0; i < spWPoses.size(); i++)
+                for (auto i = 0u; i < spWPoses.size(); i++)
                     spWPoses.at(i).emplace_back(std::make_shared<WPoseRenderer<TDatumsPtr>>(poseRenderers.at(i)));
 
             // Face renderer(s)
@@ -689,7 +689,7 @@ namespace op
                 // GPU rendering
                 else if (wrapperStructFace.renderMode == RenderMode::Gpu)
                 {
-                    for (auto i = 0; i < spWPoses.size(); i++)
+                    for (auto i = 0u; i < spWPoses.size(); i++)
                     {
                         // Construct face renderer
                         const auto faceRenderer = std::make_shared<FaceRenderer>(finalOutputSize, wrapperStructFace.renderThreshold,
@@ -727,7 +727,7 @@ namespace op
                 // GPU rendering
                 else if (wrapperStructHand.renderMode == RenderMode::Gpu)
                 {
-                    for (auto i = 0; i < spWPoses.size(); i++)
+                    for (auto i = 0u; i < spWPoses.size(); i++)
                     {
                         // Construct hands renderer
                         const auto handRenderer = std::make_shared<HandRenderer>(finalOutputSize, wrapperStructHand.renderThreshold,
@@ -751,7 +751,7 @@ namespace op
             // Itermediate workers (e.g. OpenPose format to cv::Mat, json & frames recorder, ...)
             mPostProcessingWs.clear();
             // Frame buffer and ordering
-            if (spWPoses.size() > 1)
+            if (spWPoses.size() > 1u)
                 mPostProcessingWs.emplace_back(std::make_shared<WQueueOrderer<TDatumsPtr>>());
             // Frames processor (OpenPose format -> cv::Mat format)
             if (renderOutput)
diff --git a/src/openpose/core/array.cpp b/src/openpose/core/array.cpp
@@ -173,7 +173,7 @@ namespace op
             else
             {
                 mSize = {};
-                mVolume = 0;
+                mVolume = 0ul;
                 spData.reset();
                 // cv::Mat available but empty
                 mCvMatData = std::make_pair(true, cv::Mat());
@@ -222,7 +222,7 @@ namespace op
             {
                 // New size
                 std::vector<int> newSize(cvMat.dims,0);
-                for (auto i = 0 ; i < newSize.size() ; i++)
+                for (auto i = 0u ; i < newSize.size() ; i++)
                     newSize[i] = cvMat.size[i];
                 // Reset data & volume
                 reset(newSize);
@@ -252,7 +252,7 @@ namespace op
                 if (mCvMatData.first)
                     mCvMatData.second.setTo((double)value);
                 else
-                    for (auto i = 0 ; i < mVolume ; i++)
+                    for (auto i = 0u ; i < mVolume ; i++)
                         operator[](i) = value;
             }
         }
@@ -270,12 +270,12 @@ namespace op
             // Matlab style:
                 // If empty -> return 0
                 // If index >= # dimensions -> return 1
-            if (index < mSize.size() && 0 <= index)
+            if ((unsigned int)index < mSize.size() && 0 <= index)
                 return mSize[index];
             // Long version:
             // else if (mSize.empty())
             //     return 0;
-            // else // if mSize.size() <= index 
+            // else // if mSize.size() <= (unsigned int)index
             //     return 1;
             // Equivalent to:
             else
@@ -295,7 +295,7 @@ namespace op
         {
             if (indexA < indexB)
             {
-                if (0 <= indexA && indexB < mSize.size()) // 0 <= indexA < indexB < mSize.size()
+                if (0 <= indexA && (unsigned int)indexB < mSize.size()) // 0 <= indexA < indexB < mSize.size()
                     return std::accumulate(mSize.begin()+indexA, mSize.begin()+indexB+1, 1ul, std::multiplies<size_t>());
                 else
                 {
@@ -358,14 +358,14 @@ namespace op
             // Initial value
             std::string string{"Array<T>::toString():\n"};
             // Add each element
-            for (auto i = 0 ; i < mVolume ; i++)
+            for (auto i = 0u ; i < mVolume ; i++)
             {
                 // Adding element sepearted by an space
                 string += std::to_string(spData[i]) + " ";
                 // Introduce an enter for each dimension change
                 // If comented, all values will be printed in the same line
                 auto multiplier = 1;
-                for (auto dimension = (int)(mSize.size() - 1u) ; dimension > 0 && ((i/multiplier) % getSize(dimension) == getSize(dimension)-1) ; dimension--)
+                for (auto dimension = (int)(mSize.size() - 1u) ; dimension > 0 && (int(i/multiplier) % getSize(dimension) == getSize(dimension)-1) ; dimension--)
                 {
                     string += "\n";
                     multiplier *= getSize(dimension);
@@ -423,7 +423,7 @@ namespace op
     {
         try
         {
-            if (0 <= index && index < mVolume)
+            if (0 <= index && (size_t)index < mVolume)
                 return spData[index];
             else
             {
diff --git a/src/openpose/filestream/cocoJsonSaver.cpp b/src/openpose/filestream/cocoJsonSaver.cpp
@@ -63,15 +63,15 @@ namespace op
                 mJsonOfstream.key("keypoints");
                 mJsonOfstream.arrayOpen();
                 const std::vector<int> indexesInCocoOrder{0, 15, 14, 17, 16,        5, 2, 6, 3, 7,        4, 11, 8, 12, 9,        13, 10};
-                for (auto bodyPart = 0 ; bodyPart < indexesInCocoOrder.size() ; bodyPart++)
+                for (auto bodyPart = 0u ; bodyPart < indexesInCocoOrder.size() ; bodyPart++)
                 {
                     const auto finalIndex = 3*(person*numberBodyParts + indexesInCocoOrder.at(bodyPart));
                     mJsonOfstream.plainText(poseKeypoints[finalIndex]);
                     mJsonOfstream.comma();
                     mJsonOfstream.plainText(poseKeypoints[finalIndex+1]);
                     mJsonOfstream.comma();
                     mJsonOfstream.plainText(1);
-                    if (bodyPart < indexesInCocoOrder.size() - 1)
+                    if (bodyPart < indexesInCocoOrder.size() - 1u)
                         mJsonOfstream.comma();
                 }
                 mJsonOfstream.arrayClose();
diff --git a/src/openpose/filestream/fileStream.cpp b/src/openpose/filestream/fileStream.cpp
@@ -82,7 +82,7 @@ namespace op
                 error("cvMats.size() != cvMatNames.size()", __LINE__, __FUNCTION__, __FILE__);
             // Save cv::Mat data
             cv::FileStorage fileStorage{getFullName(fileNameNoExtension, format), cv::FileStorage::WRITE};
-            for (auto i = 0 ; i < cvMats.size() ; i++)
+            for (auto i = 0u ; i < cvMats.size() ; i++)
                 fileStorage << cvMatNames[i] << (cvMats[i].empty() ? cv::Mat() : cvMats[i]);
             // Release file
             fileStorage.release();
@@ -114,7 +114,7 @@ namespace op
 
             cv::FileStorage fileStorage{getFullName(fileNameNoExtension, format), cv::FileStorage::READ};
             std::vector<cv::Mat> cvMats(cvMatNames.size());
-            for (auto i = 0 ; i < cvMats.size() ; i++)
+            for (auto i = 0u ; i < cvMats.size() ; i++)
                 fileStorage[cvMatNames[i]] >> cvMats[i];
             fileStorage.release();
             return cvMats;
@@ -171,12 +171,12 @@ namespace op
             jsonOfstream.arrayOpen();
             // Ger max numberPeople
             auto numberPeople = 0;
-            for (auto vectorIndex = 0 ; vectorIndex < keypointVector.size() ; vectorIndex++)
+            for (auto vectorIndex = 0u ; vectorIndex < keypointVector.size() ; vectorIndex++)
                 numberPeople = fastMax(numberPeople, keypointVector[vectorIndex].first.getSize(0));
             for (auto person = 0 ; person < numberPeople ; person++)
             {
                 jsonOfstream.objectOpen();
-                for (auto vectorIndex = 0 ; vectorIndex < keypointVector.size() ; vectorIndex++)
+                for (auto vectorIndex = 0u ; vectorIndex < keypointVector.size() ; vectorIndex++)
                 {
                     const auto& keypoints = keypointVector[vectorIndex].first;
                     const auto& keypointName = keypointVector[vectorIndex].second;
@@ -262,7 +262,7 @@ namespace op
                     std::vector<float> splittedInts;
                     for (auto splittedString : splittedStrings)
                         splittedInts.emplace_back(std::stof(splittedString));
-                    if (splittedInts.size() != 4)
+                    if (splittedInts.size() != 4u)
                         error("splittedInts.size() != 4, but splittedInts.size() = "
                               + std::to_string(splittedInts.size()) + ".", __LINE__, __FUNCTION__, __FILE__);
                     const Rectangle<float> handRectangleZero;
diff --git a/src/openpose/filestream/heatMapSaver.cpp b/src/openpose/filestream/heatMapSaver.cpp
@@ -31,16 +31,16 @@ namespace op
 
                 // Get names for each heatMap
                 std::vector<std::string> fileNames(heatMaps.size());
-                for (auto i = 0; i < fileNames.size(); i++)
+                for (auto i = 0u; i < fileNames.size(); i++)
                     fileNames[i] = {fileNameNoExtension + (i != 0 ? "_" + std::to_string(i) : "") + "." + mImageFormat};
 
                 // heatMaps -> cvOutputDatas
                 std::vector<cv::Mat> cvOutputDatas(heatMaps.size());
-                for (auto i = 0; i < cvOutputDatas.size(); i++)
+                for (auto i = 0u; i < cvOutputDatas.size(); i++)
                     unrollArrayToUCharCvMat(cvOutputDatas[i], heatMaps[i]);
 
                 // Save each heatMap
-                for (auto i = 0; i < cvOutputDatas.size(); i++)
+                for (auto i = 0u; i < cvOutputDatas.size(); i++)
                     saveImage(cvOutputDatas[i], fileNames[i]);
             }
         }
diff --git a/src/openpose/filestream/imageSaver.cpp b/src/openpose/filestream/imageSaver.cpp
@@ -30,11 +30,11 @@ namespace op
 
                 // Get names for each image
                 std::vector<std::string> fileNames(cvOutputDatas.size());
-                for (auto i = 0; i < fileNames.size(); i++)
+                for (auto i = 0u; i < fileNames.size(); i++)
                     fileNames[i] = {fileNameNoExtension + (i != 0 ? "_" + std::to_string(i) : "") + "." + mImageFormat};
 
                 // Save each image
-                for (auto i = 0; i < cvOutputDatas.size(); i++)
+                for (auto i = 0u; i < cvOutputDatas.size(); i++)
                     saveImage(cvOutputDatas[i], fileNames[i]);
             }
         }
diff --git a/src/openpose/filestream/keypointSaver.cpp b/src/openpose/filestream/keypointSaver.cpp
@@ -20,12 +20,12 @@ namespace op
 
                 // Get vector of people poses
                 std::vector<cv::Mat> cvMatPoses(keypointVector.size());
-                for (auto i = 0; i < keypointVector.size(); i++)
+                for (auto i = 0u; i < keypointVector.size(); i++)
                     cvMatPoses[i] = keypointVector[i].getConstCvMat();
 
                 // Get names inside file
                 std::vector<std::string> keypointVectorNames(cvMatPoses.size());
-                for (auto i = 0; i < cvMatPoses.size(); i++)
+                for (auto i = 0u; i < cvMatPoses.size(); i++)
                     keypointVectorNames[i] = {keypointName + "_" + std::to_string(i)};
 
                 // Record people poses in desired format
diff --git a/src/openpose/filestream/videoSaver.cpp b/src/openpose/filestream/videoSaver.cpp
diff --git a/src/openpose/hand/handDetector.cpp b/src/openpose/hand/handDetector.cpp
diff --git a/src/openpose/pose/bodyPartConnectorBase.cpp b/src/openpose/pose/bodyPartConnectorBase.cpp
diff --git a/src/openpose/pose/poseRenderer.cpp b/src/openpose/pose/poseRenderer.cpp
diff --git a/src/openpose/utilities/keypoint.cpp b/src/openpose/utilities/keypoint.cpp

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ class Blob {`
`53`	`53`	`void ReshapeLike(const Blob& other);`
`54`	`54`	`inline string shape_string() const {`
`55`	`55`	`ostringstream stream;`
`56`		`- for (int i = 0; i < shape_.size(); ++i) {`
	`56`	`+ for (auto i = 0u; i < shape_.size(); ++i) {`
`57`	`57`	`stream << shape_[i] << " ";`
`58`	`58`	`}`
`59`	`59`	`stream << "(" << count_ << ")";`
Original file line number	Diff line number	Diff line change
`@@ -185,7 +185,7 @@ namespace op`
`185`	`185`	`{`
`186`	`186`	`spWPoses.resize(gpuNumber);`
`187`	`187`	`const auto handDetector = std::make_shared<HandDetectorFromTxt>(handGroundTruth);`
`188`		`- for (auto gpuId = 0; gpuId < spWPoses.size(); gpuId++)`
	`188`	`+ for (auto gpuId = 0u; gpuId < spWPoses.size(); gpuId++)`
`189`	`189`	`{`
`190`	`190`	`// Hand detector`
`191`	`191`	`// If tracking`
Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ namespace op`
`59`	`59`	`const auto baseFileName = (!tDatumFirst.name.empty() ? tDatumFirst.name`
`60`	`60`	`: std::to_string(tDatumFirst.id)) + "_keypoints";`
`61`	`61`	`const bool humanReadable = true;`
`62`		`- for (auto i = 0 ; i < tDatums->size() ; i++)`
	`62`	`+ for (auto i = 0u ; i < tDatums->size() ; i++)`
`63`	`63`	`{`
`64`	`64`	`const auto& tDatum = (*tDatums)[i];`
`65`	`65`	`// const auto fileName = baseFileName;`