Adding namespaces on image and moving private methods to anonymous.

datumbox · datumbox · commit 7fdc7ba39d22 · 2021-01-27T20:27:00.000Z
diff --git a/torchvision/csrc/io/image/cpu/jpegcommon.cpp b/torchvision/csrc/io/image/cpu/jpegcommon.cpp
@@ -1,5 +1,9 @@
 #include "jpegcommon.h"
 
+namespace vision {
+namespace image {
+namespace detail {
+
 #if JPEG_FOUND
 void torch_jpeg_error_exit(j_common_ptr cinfo) {
   /* cinfo->err really points to a torch_jpeg_error_mgr struct, so coerce
@@ -16,3 +20,7 @@ void torch_jpeg_error_exit(j_common_ptr cinfo) {
   longjmp(myerr->setjmp_buffer, 1);
 }
 #endif
+
+} // namespace detail
+} // namespace image
+} // namespace vision
diff --git a/torchvision/csrc/io/image/cpu/jpegcommon.h b/torchvision/csrc/io/image/cpu/jpegcommon.h
@@ -3,6 +3,13 @@
 #if JPEG_FOUND
 #include <jpeglib.h>
 #include <setjmp.h>
+#endif
+
+namespace vision {
+namespace image {
+namespace detail {
+
+#if JPEG_FOUND
 
 static const JOCTET EOI_BUFFER[1] = {JPEG_EOI};
 struct torch_jpeg_error_mgr {
@@ -15,3 +22,7 @@ using torch_jpeg_error_ptr = struct torch_jpeg_error_mgr*;
 void torch_jpeg_error_exit(j_common_ptr cinfo);
 
 #endif
+
+} // namespace detail
+} // namespace image
+} // namespace vision
diff --git a/torchvision/csrc/io/image/cpu/pngcommon.h b/torchvision/csrc/io/image/cpu/pngcommon.h
@@ -0,0 +1,6 @@
+#pragma once
+
+#if PNG_FOUND
+#include <png.h>
+#include <setjmp.h>
+#endif
diff --git a/torchvision/csrc/io/image/cpu/read_image_impl.cpp b/torchvision/csrc/io/image/cpu/read_image_impl.cpp
@@ -3,6 +3,9 @@
 #include "readjpeg_impl.h"
 #include "readpng_impl.h"
 
+namespace vision {
+namespace image {
+
 torch::Tensor decode_image(const torch::Tensor& data, ImageReadMode mode) {
   // Check that the input tensor dtype is uint8
   TORCH_CHECK(data.dtype() == torch::kU8, "Expected a torch.uint8 tensor");
@@ -27,3 +30,6 @@ torch::Tensor decode_image(const torch::Tensor& data, ImageReadMode mode) {
         "are currently supported.");
   }
 }
+
+} // namespace image
+} // namespace vision
diff --git a/torchvision/csrc/io/image/cpu/read_image_impl.h b/torchvision/csrc/io/image/cpu/read_image_impl.h
@@ -3,6 +3,12 @@
 #include <torch/types.h>
 #include "../image_read_mode.h"
 
+namespace vision {
+namespace image {
+
 C10_EXPORT torch::Tensor decode_image(
     const torch::Tensor& data,
     ImageReadMode mode = IMAGE_READ_MODE_UNCHANGED);
+
+} // namespace image
+} // namespace vision
diff --git a/torchvision/csrc/io/image/cpu/read_write_file_impl.cpp b/torchvision/csrc/io/image/cpu/read_write_file_impl.cpp
@@ -1,9 +1,17 @@
 #include "read_write_file_impl.h"
 
+#include <sys/stat.h>
+
 #ifdef _WIN32
 #define WIN32_LEAN_AND_MEAN
 #include <Windows.h>
+#endif
+
+namespace vision {
+namespace image {
 
+#ifdef _WIN32
+namespace {
 std::wstring utf8_decode(const std::string& str) {
   if (str.empty()) {
     return std::wstring();
@@ -21,6 +29,7 @@ std::wstring utf8_decode(const std::string& str) {
       size_needed);
   return wstrTo;
 }
+} // namespace
 #endif
 
 torch::Tensor read_file(const std::string& filename) {
@@ -90,3 +99,6 @@ void write_file(const std::string& filename, torch::Tensor& data) {
   fwrite(fileBytes, sizeof(uint8_t), data.numel(), outfile);
   fclose(outfile);
 }
+
+} // namespace image
+} // namespace vision
diff --git a/torchvision/csrc/io/image/cpu/read_write_file_impl.h b/torchvision/csrc/io/image/cpu/read_write_file_impl.h
@@ -1,8 +1,13 @@
 #pragma once
 
-#include <sys/stat.h>
 #include <torch/types.h>
 
+namespace vision {
+namespace image {
+
 C10_EXPORT torch::Tensor read_file(const std::string& filename);
 
 C10_EXPORT void write_file(const std::string& filename, torch::Tensor& data);
+
+} // namespace image
+} // namespace vision
diff --git a/torchvision/csrc/io/image/cpu/readjpeg_impl.cpp b/torchvision/csrc/io/image/cpu/readjpeg_impl.cpp
@@ -1,12 +1,19 @@
 #include "readjpeg_impl.h"
+#include "jpegcommon.h"
+
+namespace vision {
+namespace image {
+
+using namespace detail;
 
 #if !JPEG_FOUND
 torch::Tensor decodeJPEG(const torch::Tensor& data, ImageReadMode mode) {
   TORCH_CHECK(
       false, "decodeJPEG: torchvision not compiled with libjpeg support");
 }
 #else
-#include "jpegcommon.h"
+
+namespace {
 
 struct torch_jpeg_mgr {
   struct jpeg_source_mgr pub;
@@ -64,6 +71,8 @@ static void torch_jpeg_set_source_mgr(
   src->pub.next_input_byte = src->data;
 }
 
+} // namespace
+
 torch::Tensor decodeJPEG(const torch::Tensor& data, ImageReadMode mode) {
   // Check that the input tensor dtype is uint8
   TORCH_CHECK(data.dtype() == torch::kU8, "Expected a torch.uint8 tensor");
@@ -146,4 +155,7 @@ torch::Tensor decodeJPEG(const torch::Tensor& data, ImageReadMode mode) {
   return tensor.permute({2, 0, 1});
 }
 
-#endif // JPEG_FOUND
+#endif
+
+} // namespace image
+} // namespace vision
diff --git a/torchvision/csrc/io/image/cpu/readjpeg_impl.h b/torchvision/csrc/io/image/cpu/readjpeg_impl.h
@@ -3,6 +3,12 @@
 #include <torch/types.h>
 #include "../image_read_mode.h"
 
+namespace vision {
+namespace image {
+
 C10_EXPORT torch::Tensor decodeJPEG(
     const torch::Tensor& data,
     ImageReadMode mode = IMAGE_READ_MODE_UNCHANGED);
+
+} // namespace image
+} // namespace vision
diff --git a/torchvision/csrc/io/image/cpu/readpng_impl.cpp b/torchvision/csrc/io/image/cpu/readpng_impl.cpp
@@ -1,12 +1,14 @@
 #include "readpng_impl.h"
+#include "pngcommon.h"
+
+namespace vision {
+namespace image {
 
 #if !PNG_FOUND
 torch::Tensor decodePNG(const torch::Tensor& data, ImageReadMode mode) {
   TORCH_CHECK(false, "decodePNG: torchvision not compiled with libPNG support");
 }
 #else
-#include <png.h>
-#include <setjmp.h>
 
 torch::Tensor decodePNG(const torch::Tensor& data, ImageReadMode mode) {
   // Check that the input tensor dtype is uint8
@@ -160,4 +162,7 @@ torch::Tensor decodePNG(const torch::Tensor& data, ImageReadMode mode) {
   png_destroy_read_struct(&png_ptr, &info_ptr, nullptr);
   return tensor.permute({2, 0, 1});
 }
-#endif // PNG_FOUND
+#endif
+
+} // namespace image
+} // namespace vision
diff --git a/torchvision/csrc/io/image/cpu/readpng_impl.h b/torchvision/csrc/io/image/cpu/readpng_impl.h
@@ -3,6 +3,12 @@
 #include <torch/types.h>
 #include "../image_read_mode.h"
 
+namespace vision {
+namespace image {
+
 C10_EXPORT torch::Tensor decodePNG(
     const torch::Tensor& data,
     ImageReadMode mode = IMAGE_READ_MODE_UNCHANGED);
+
+} // namespace image
+} // namespace vision
diff --git a/torchvision/csrc/io/image/cpu/writejpeg_impl.cpp b/torchvision/csrc/io/image/cpu/writejpeg_impl.cpp
@@ -1,4 +1,10 @@
 #include "writejpeg_impl.h"
+#include "jpegcommon.h"
+
+namespace vision {
+namespace image {
+
+using namespace detail;
 
 #if !JPEG_FOUND
 
@@ -8,7 +14,6 @@ torch::Tensor encodeJPEG(const torch::Tensor& data, int64_t quality) {
 }
 
 #else
-#include "jpegcommon.h"
 
 torch::Tensor encodeJPEG(const torch::Tensor& data, int64_t quality) {
   // Define compression structures and error handling
@@ -98,3 +103,6 @@ torch::Tensor encodeJPEG(const torch::Tensor& data, int64_t quality) {
   return outTensor;
 }
 #endif
+
+} // namespace image
+} // namespace vision
diff --git a/torchvision/csrc/io/image/cpu/writejpeg_impl.h b/torchvision/csrc/io/image/cpu/writejpeg_impl.h
@@ -2,4 +2,10 @@
 
 #include <torch/types.h>
 
+namespace vision {
+namespace image {
+
 C10_EXPORT torch::Tensor encodeJPEG(const torch::Tensor& data, int64_t quality);
+
+} // namespace image
+} // namespace vision
diff --git a/torchvision/csrc/io/image/cpu/writepng_impl.cpp b/torchvision/csrc/io/image/cpu/writepng_impl.cpp
@@ -1,4 +1,8 @@
 #include "writejpeg_impl.h"
+#include "pngcommon.h"
+
+namespace vision {
+namespace image {
 
 #if !PNG_FOUND
 
@@ -7,8 +11,8 @@ torch::Tensor encodePNG(const torch::Tensor& data, int64_t compression_level) {
 }
 
 #else
-#include <png.h>
-#include <setjmp.h>
+
+namespace {
 
 struct torch_mem_encode {
   char* buffer;
@@ -59,6 +63,8 @@ void torch_png_write_data(
   p->size += length;
 }
 
+} // namespace
+
 torch::Tensor encodePNG(const torch::Tensor& data, int64_t compression_level) {
   // Define compression structures and error handling
   png_structp png_write;
@@ -171,3 +177,6 @@ torch::Tensor encodePNG(const torch::Tensor& data, int64_t compression_level) {
 }
 
 #endif
+
+} // namespace image
+} // namespace vision
diff --git a/torchvision/csrc/io/image/cpu/writepng_impl.h b/torchvision/csrc/io/image/cpu/writepng_impl.h
@@ -2,6 +2,12 @@
 
 #include <torch/types.h>
 
+namespace vision {
+namespace image {
+
 C10_EXPORT torch::Tensor encodePNG(
     const torch::Tensor& data,
     int64_t compression_level);
+
+} // namespace image
+} // namespace vision
diff --git a/torchvision/csrc/io/image/image.cpp b/torchvision/csrc/io/image/image.cpp
@@ -11,6 +11,9 @@ PyMODINIT_FUNC PyInit_image(void) {
 }
 #endif
 
+namespace vision {
+namespace image {
+
 static auto registry = torch::RegisterOperators()
                            .op("image::decode_png", &decodePNG)
                            .op("image::encode_png", &encodePNG)
@@ -19,3 +22,6 @@ static auto registry = torch::RegisterOperators()
                            .op("image::read_file", &read_file)
                            .op("image::write_file", &write_file)
                            .op("image::decode_image", &decode_image);
+
+} // namespace image
+} // namespace vision
diff --git a/torchvision/csrc/io/image/image_read_mode.h b/torchvision/csrc/io/image/image_read_mode.h
@@ -1,9 +1,17 @@
 #pragma once
 
+#include <stdint.h>
+
+namespace vision {
+namespace image {
+
 /* Should be kept in-sync with Python ImageReadMode enum */
 using ImageReadMode = int64_t;
 const ImageReadMode IMAGE_READ_MODE_UNCHANGED = 0;
 const ImageReadMode IMAGE_READ_MODE_GRAY = 1;
 const ImageReadMode IMAGE_READ_MODE_GRAY_ALPHA = 2;
 const ImageReadMode IMAGE_READ_MODE_RGB = 3;
 const ImageReadMode IMAGE_READ_MODE_RGB_ALPHA = 4;
+
+} // namespace image
+} // namespace vision
