Header call moved to first line of each cpp

doc/release_notes.md

@@ -388,6 +388,7 @@ OpenPose Library - Release Notes
     1. Headers do not contain any 3rd-party library includes nor functions. This way, OpenPose can be exported without needing 3rd-party includes nor static library files (e.g., lib files in Windows), allowing people to use their own versions of OpenCV, Eigen, etc. without conflicting with OpenPose. Dynamic library files (e.g., `dll` files in Windows, `so` in Ubuntu) are still required.
     2. Created the `openpose_private` directory with some internal headers that, if exported with OpenPose, would require including 3rd-party headers and static library files.
     3. Default OpenCV version for Windows upgraded to version 4.1.1, extracted from their oficial website: section `Releases`, subsection `OpenCV - 4.1.1`, `Windows` version.
+    4. In all `*.cpp` files, their include of their analog `*.hpp` file has been moved to the first line of those `*.cpp` files to slightly speed up compiling time.
 2. Functions or parameters renamed:
     1. All headers moved into `openpose_private` and all 3rd-party library calls in headers.
 3. Main bugs fixed:

include/openpose/pose/poseGpuRenderer.hpp

@@ -6,6 +6,7 @@
 #include <openpose/pose/enumClasses.hpp>
 #include <openpose/pose/poseExtractorNet.hpp>
 #include <openpose/pose/poseParameters.hpp>
+#include <openpose/pose/poseParametersRender.hpp>
 #include <openpose/pose/poseRenderer.hpp>
 
 namespace op

src/openpose/3d/jointAngleEstimation.cpp

 #ifdef USE_3D_ADAM_MODEL
+#include <openpose/3d/jointAngleEstimation.hpp>
 #ifdef USE_3D_ADAM_MODEL
     #include <adam/FitToBody.h>
     #include <adam/totalmodel.h>
 #endif
-#include <openpose/3d/jointAngleEstimation.hpp>
 
 namespace op
 {

src/openpose/core/arrayCpuGpu.cpp

+#include <openpose/core/arrayCpuGpu.hpp>
 #ifdef USE_CAFFE
     #include <caffe/blob.hpp>
 #endif
 #include <openpose/utilities/errorAndLog.hpp>
-#include <openpose/core/arrayCpuGpu.hpp>
 
 namespace op
 {

src/openpose/core/datum.cpp

-#include <openpose/utilities/errorAndLog.hpp>
 #include <openpose/core/datum.hpp>
+#include <openpose/utilities/errorAndLog.hpp>
 
 namespace op
 {

src/openpose/core/gpuRenderer.cpp

+#include <openpose/core/gpuRenderer.hpp>
 #ifdef USE_CUDA
     #include <cuda.h>
     #include <cuda_runtime_api.h>
 #endif
-#include <openpose/core/gpuRenderer.hpp>
 
 namespace op
 {

src/openpose/core/keypointScaler.cpp

-#include <openpose/utilities/keypoint.hpp>
 #include <openpose/core/keypointScaler.hpp>
+#include <openpose/utilities/keypoint.hpp>
 
 namespace op
 {

src/openpose/core/point.cpp

-#include <openpose/utilities/errorAndLog.hpp>
 #include <openpose/core/point.hpp>
 #include <ostream>
+#include <openpose/utilities/errorAndLog.hpp>
 
 namespace op
 {

src/openpose/core/rectangle.cpp

-#include <openpose/utilities/errorAndLog.hpp>
 #include <openpose/core/rectangle.hpp>
+#include <openpose/utilities/errorAndLog.hpp>
 
 namespace op
 {

src/openpose/core/scaleAndSizeExtractor.cpp

+#include <openpose/core/scaleAndSizeExtractor.hpp>
 #include <openpose/utilities/fastMath.hpp>
 #include <openpose/utilities/openCv.hpp> // resizeGetScaleFactor
-#include <openpose/core/scaleAndSizeExtractor.hpp>
 
 namespace op
 {

src/openpose/core/verbosePrinter.cpp

-#include <openpose/utilities/fastMath.hpp>
 #include <openpose/core/verbosePrinter.hpp>
+#include <openpose/utilities/fastMath.hpp>
 
 namespace op
 {

src/openpose/face/faceCpuRenderer.cpp

-#include <openpose/face/renderFace.hpp>
 #include <openpose/face/faceCpuRenderer.hpp>
+#include <openpose/face/renderFace.hpp>
 
 namespace op
 {

src/openpose/face/faceDetector.cpp

+#include <openpose/face/faceDetector.hpp>
 #include <openpose/pose/poseParameters.hpp>
 #include <openpose/utilities/check.hpp>
 #include <openpose/utilities/keypoint.hpp>
-#include <openpose/face/faceDetector.hpp>
 
 namespace op
 {

src/openpose/face/faceExtractorNet.cpp

-#include <openpose/utilities/check.hpp>
 #include <openpose/face/faceExtractorNet.hpp>
+#include <openpose/utilities/check.hpp>
 
 namespace op
 {

src/openpose/face/faceGpuRenderer.cpp

+#include <openpose/face/faceGpuRenderer.hpp>
 #ifdef USE_CUDA
     #include <cuda.h>
     #include <cuda_runtime_api.h>
 #endif
 #include <openpose/face/renderFace.hpp>
 #include <openpose/gpu/cuda.hpp>
-#include <openpose/face/faceGpuRenderer.hpp>
 
 namespace op
 {

src/openpose/face/faceRenderer.cpp

+#include <openpose/face/faceRenderer.hpp>
 #include <openpose/face/renderFace.hpp>
 #include <openpose/utilities/keypoint.hpp>
-#include <openpose/face/faceRenderer.hpp>
 
 namespace op
 {

src/openpose/face/renderFace.cpp

+#include <openpose/face/renderFace.hpp>
 #include <openpose/face/faceParameters.hpp>
 #include <openpose/utilities/fastMath.hpp>
 #include <openpose/utilities/keypoint.hpp>
-#include <openpose/face/renderFace.hpp>
 
 namespace op
 {

src/openpose/filestream/bvhSaver.cpp

 #ifdef USE_3D_ADAM_MODEL
+#include <openpose/filestream/bvhSaver.hpp>
 #ifdef USE_3D_ADAM_MODEL
     #include <adam/BVHWriter.h>
 #endif
-#include <openpose/filestream/bvhSaver.hpp>
 
 namespace op
 {

src/openpose/filestream/cocoJsonSaver.cpp

+#include <openpose/filestream/cocoJsonSaver.hpp>
 #include <numeric> // std::iota
 #include <openpose/pose/poseParametersRender.hpp>
 #include <openpose/utilities/fileSystem.hpp>
 #include <openpose/utilities/string.hpp>
-#include <openpose/filestream/cocoJsonSaver.hpp>
 
 namespace op
 {

src/openpose/filestream/fileSaver.cpp

-#include <openpose/utilities/fileSystem.hpp>
 #include <openpose/filestream/fileSaver.hpp>
+#include <openpose/utilities/fileSystem.hpp>
 
 namespace op
 {

src/openpose/filestream/fileStream.cpp

+#include <openpose/filestream/fileStream.hpp>
 #include <fstream> // std::ifstream, std::ofstream
 #include <opencv2/highgui/highgui.hpp> // cv::imread
 #include <openpose/utilities/fastMath.hpp>
 #include <openpose/utilities/fileSystem.hpp>
 #include <openpose/utilities/string.hpp>
 #include <openpose/filestream/jsonOfstream.hpp>
-#include <openpose/filestream/fileStream.hpp>
 
 namespace op
 {

src/openpose/filestream/heatMapSaver.cpp

+#include <openpose/filestream/heatMapSaver.hpp>
 #include <openpose/utilities/openCv.hpp>
 #include <openpose/filestream/fileStream.hpp>
-#include <openpose/filestream/heatMapSaver.hpp>
 
 namespace op
 {

src/openpose/filestream/imageSaver.cpp

-#include <openpose/filestream/fileStream.hpp>
 #include <openpose/filestream/imageSaver.hpp>
+#include <openpose/filestream/fileStream.hpp>
 
 namespace op
 {

src/openpose/filestream/keypointSaver.cpp

-#include <openpose/filestream/fileStream.hpp>
 #include <openpose/filestream/keypointSaver.hpp>
+#include <openpose/filestream/fileStream.hpp>
 
 namespace op
 {

src/openpose/filestream/peopleJsonSaver.cpp

-#include <openpose/filestream/fileStream.hpp>
 #include <openpose/filestream/peopleJsonSaver.hpp>
+#include <openpose/filestream/fileStream.hpp>
 
 namespace op
 {

src/openpose/filestream/udpSender.cpp

+#include <openpose/filestream/udpSender.hpp>
 #ifdef USE_ASIO
     #include <iostream>
     #include <asio.hpp>
@@ -6,7 +7,6 @@
     #include <Eigen/Core>
 #endif
 #include <openpose/filestream/fileStream.hpp>
-#include <openpose/filestream/udpSender.hpp>
 
 namespace op
 {

src/openpose/filestream/videoSaver.cpp

+#include <openpose/filestream/videoSaver.hpp>
 #include <opencv2/highgui/highgui.hpp> // cv::VideoWriter
 #include <openpose/filestream/imageSaver.hpp>
 #include <openpose/utilities/fileSystem.hpp>
 #include <openpose/utilities/string.hpp>
-#include <openpose/filestream/videoSaver.hpp>
 
 namespace op
 {

src/openpose/gpu/cuda.cpp

+#include <openpose/gpu/cuda.hpp>
 #ifdef USE_CUDA
     #include <cuda.h>
     #include <cuda_runtime.h>
     #include <openpose/utilities/fastMath.hpp>
 #endif
-#include <openpose/gpu/cuda.hpp>
 
 namespace op
 {

src/openpose/gpu/cuda.cu

+#include <openpose/gpu/cuda.hpp>
 #ifdef USE_CUDA
     #include <cuda.h>
     #include <cuda_runtime.h>
     #include <openpose_private/gpu/cuda.hu>
 #endif
-#include <openpose/gpu/cuda.hpp>
 
 namespace op
 {

src/openpose/gui/gui.cpp

+#include <openpose/gui/gui.hpp>
 #include <opencv2/highgui/highgui.hpp> // cv::waitKey
 #include <openpose/filestream/fileStream.hpp>
 #include <openpose/utilities/check.hpp>
-#include <openpose/gui/gui.hpp>
 
 namespace op
 {

src/openpose/gui/gui3D.cpp

-#include <atomic>
+#include <openpose/gui/gui3D.hpp>
+#include <atomic>
 #include <mutex>
 #include <stdio.h>
 #ifdef USE_3D_RENDERER
@@ -11,7 +12,6 @@
 #include <openpose/hand/handParameters.hpp>
 #include <openpose/pose/poseParameters.hpp>
 #include <openpose/utilities/keypoint.hpp>
-#include <openpose/gui/gui3D.hpp>
 
 namespace op
 {

src/openpose/gui/guiAdam.cpp

 #ifdef USE_3D_ADAM_MODEL
+#include <openpose/gui/guiAdam.hpp>
 #ifdef USE_3D_ADAM_MODEL
     #include <adam/KinematicModel.h>
     #include <adam/Renderer.h>
@@ -8,7 +9,6 @@
 #endif
 #include <openpose/3d/jointAngleEstimation.hpp>
 #include <openpose/filestream/videoSaver.hpp>
-#include <openpose/gui/guiAdam.hpp>
 
 namespace op
 {

src/openpose/hand/handCpuRenderer.cpp

-#include <openpose/hand/renderHand.hpp>
 #include <openpose/hand/handCpuRenderer.hpp>
+#include <openpose/hand/renderHand.hpp>
 
 namespace op
 {

src/openpose/hand/handDetector.cpp

+#include <openpose/hand/handDetector.hpp>
 #include <openpose/pose/poseParameters.hpp>
 #include <openpose/utilities/check.hpp>
 #include <openpose/utilities/fastMath.hpp>
 #include <openpose/utilities/keypoint.hpp>
-#include <openpose/hand/handDetector.hpp>
 
 namespace op
 {

src/openpose/hand/handDetectorFromTxt.cpp

+#include <openpose/hand/handDetectorFromTxt.hpp>
 #include <openpose/filestream/fileStream.hpp>
 #include <openpose/utilities/fileSystem.hpp>
-#include <openpose/hand/handDetectorFromTxt.hpp>
 
 namespace op
 {

src/openpose/hand/handExtractorNet.cpp

-#include <openpose/utilities/check.hpp>
 #include <openpose/hand/handExtractorNet.hpp>
+#include <openpose/utilities/check.hpp>
 
 namespace op
 {

src/openpose/hand/handGpuRenderer.cpp

+#include <openpose/hand/handGpuRenderer.hpp>
 #ifdef USE_CUDA
     #include <cuda.h>
     #include <cuda_runtime_api.h>
 #endif
 #include <openpose/gpu/cuda.hpp>
 #include <openpose/hand/renderHand.hpp>
-#include <openpose/hand/handGpuRenderer.hpp>
 
 namespace op
 {

src/openpose/hand/handRenderer.cpp

-#include <openpose/utilities/keypoint.hpp>
 #include <openpose/hand/handRenderer.hpp>
+#include <openpose/utilities/keypoint.hpp>
 
 namespace op
 {

src/openpose/hand/renderHand.cpp

+#include <openpose/hand/renderHand.hpp>
 #include <openpose/hand/handParameters.hpp>
 #include <openpose/utilities/fastMath.hpp>
 #include <openpose/utilities/keypoint.hpp>
-#include <openpose/hand/renderHand.hpp>
 
 namespace op
 {

src/openpose/net/bodyPartConnectorBase.cu

+#include <openpose/net/bodyPartConnectorBase.hpp>
 #include <openpose/gpu/cuda.hpp>
 #include <openpose/pose/poseParameters.hpp>
 #include <openpose/utilities/fastMath.hpp>
-#include <openpose/net/bodyPartConnectorBase.hpp>
 
 namespace op
 {

src/openpose/net/maximumBase.cu

+#include <openpose/net/maximumBase.hpp>
 #include <thrust/device_ptr.h>
 #include <thrust/extrema.h>
 #include <openpose/gpu/cuda.hpp>
 #include <openpose/gpu/cuda.hpp>
-#include <openpose/net/maximumBase.hpp>
 
 namespace op
 {

src/openpose/net/maximumCaffe.cpp

+#include <openpose/net/maximumCaffe.hpp>
 #ifdef USE_CAFFE
     #include <caffe/blob.hpp>
 #endif
 #include <openpose/net/maximumBase.hpp>
-#include <openpose/net/maximumCaffe.hpp>
 
 namespace op
 {

src/openpose/net/nmsBase.cpp

-#include <opencv2/opencv.hpp>
 #include <openpose/net/nmsBase.hpp>
+#include <opencv2/opencv.hpp>
 
 namespace op
 {

src/openpose/pose/poseCpuRenderer.cpp

+#include <openpose/pose/poseCpuRenderer.hpp>
 #include <openpose/pose/renderPose.hpp>
 #include <openpose/utilities/keypoint.hpp>
-#include <openpose/pose/poseCpuRenderer.hpp>
 
 namespace op
 {

src/openpose/pose/poseGpuRenderer.cpp

+#include <openpose/pose/poseGpuRenderer.hpp>
 #ifdef USE_CUDA
     #include <cuda.h>
     #include <cuda_runtime_api.h>
@@ -6,7 +7,6 @@
 #include <openpose/pose/renderPose.hpp>
 #include <openpose/gpu/cuda.hpp>
 #include <openpose/utilities/keypoint.hpp>
-#include <openpose/pose/poseGpuRenderer.hpp>
 
 namespace op
 {

src/openpose/pose/poseParameters.cpp

-#include <openpose/pose/poseParametersRender.hpp>
 #include <openpose/pose/poseParameters.hpp>
+#include <openpose/pose/poseParametersRender.hpp>
 
 namespace op
 {

src/openpose/pose/poseParametersRender.cpp

-#include <openpose/pose/poseParameters.hpp>
 #include <openpose/pose/poseParametersRender.hpp>
+#include <openpose/pose/poseParameters.hpp>
 
 namespace op
 {

src/openpose/pose/poseRenderer.cpp

+#include <openpose/pose/poseRenderer.hpp>
 #include <openpose/pose/poseParameters.hpp>
 #include <openpose/pose/poseParametersRender.hpp>
 #include <openpose/pose/renderPose.hpp>
 #include <openpose/utilities/fastMath.hpp>
-#include <openpose/pose/poseRenderer.hpp>
 
 namespace op
 {

src/openpose/pose/renderPose.cpp

+#include <openpose/pose/renderPose.hpp>
 #include <openpose/pose/poseParameters.hpp>
 #include <openpose/utilities/fastMath.hpp>
 #include <openpose/utilities/keypoint.hpp>
-#include <openpose/pose/renderPose.hpp>
 
 namespace op
 {

src/openpose/utilities/errorAndLog.cpp

+#include <openpose/utilities/errorAndLog.hpp>
 #include <atomic>
 #include <mutex>
 #include <ctime> // std::tm, std::time_t
 #include <fstream> // std::ifstream, std::ofstream
 #include <iostream> // std::cout, std::endl
 #include <stdexcept> // std::runtime_error
-#include <openpose/utilities/errorAndLog.hpp>
 
 namespace op
 {

src/openpose/utilities/flagsToOpenPose.cpp

+#include <openpose/utilities/flagsToOpenPose.hpp>
 #include <cstdio> // sscanf
 #include <openpose/utilities/check.hpp>
-#include <openpose/utilities/flagsToOpenPose.hpp>
 
 namespace op
 {

src/openpose/utilities/keypoint.cpp

+#include <openpose/utilities/keypoint.hpp>
 #include <limits> // std::numeric_limits
 #include <opencv2/imgproc/imgproc.hpp> // cv::line, cv::circle
 #include <openpose/utilities/fastMath.hpp>
-#include <openpose/utilities/keypoint.hpp>
 
 namespace op
 {

src/openpose/utilities/profiler.cpp

+#include <openpose/utilities/profiler.hpp>
 #include <map>
 #include <mutex>
 #include <openpose/utilities/errorAndLog.hpp>
-#include <openpose/utilities/profiler.hpp>
 
 // First, I apologize for the ugliness of the code of this function. Nevertheless, it has been made
 // in this way so that it has no computational impact at all if PROFILER_ENABLED is not defined.

src/openpose/wrapper/wrapperAuxiliary.cpp

+#include <openpose/wrapper/wrapperAuxiliary.hpp>
 #include <openpose/gpu/gpu.hpp>
 #include <openpose/thread/enumClasses.hpp>
-#include <openpose/wrapper/wrapperAuxiliary.hpp>
 
 namespace op
 {