diff --git a/sensor_msgs/include/sensor_msgs/image_encodings.hpp b/sensor_msgs/include/sensor_msgs/image_encodings.hpp index 63ff15d5..c3274700 100644 --- a/sensor_msgs/include/sensor_msgs/image_encodings.hpp +++ b/sensor_msgs/include/sensor_msgs/image_encodings.hpp @@ -97,11 +97,9 @@ const char BAYER_GRBG16[] = "bayer_grbg16"; // https://www.kernel.org/doc/html/latest/userspace-api/media/v4l/pixfmt-packed-yuv.html#id1 // fourcc: UYVY const char UYVY[] = "uyvy"; -[[deprecated("use sensor_msgs::image_encodings::UYVY")]] const char YUV422[] = "yuv422"; // deprecated // fourcc: YUYV const char YUYV[] = "yuyv"; -[[deprecated("use sensor_msgs::image_encodings::YUYV")]] const char YUV422_YUY2[] = "yuv422_yuy2"; // deprecated // YUV 4:2:0 encodings with an 8-bit depth @@ -122,8 +120,6 @@ const std::regex cv_type_regex("(8|16|32|64)(U|S|F)C([0-9]*)"); // Utility functions for inspecting an encoding string static inline bool isColor(const std::string & encoding) { -#pragma GCC diagnostic push -#pragma GCC diagnostic ignored "-Wdeprecated-declarations" return encoding == RGB8 || encoding == BGR8 || encoding == RGBA8 || encoding == BGRA8 || encoding == RGB16 || encoding == BGR16 || @@ -131,7 +127,6 @@ static inline bool isColor(const std::string & encoding) encoding == YUV422 || encoding == YUV422_YUY2 || encoding == UYVY || encoding == YUYV || encoding == NV21 || encoding == NV24; -#pragma GCC diagnostic pop } static inline bool isMono(const std::string & encoding) @@ -194,8 +189,6 @@ static inline int numChannels(const std::string & encoding) return (m[3] == "") ? 1 : std::atoi(m[3].str().c_str()); } -#pragma GCC diagnostic push -#pragma GCC diagnostic ignored "-Wdeprecated-declarations" if (encoding == YUV422 || encoding == YUV422_YUY2 || encoding == UYVY || @@ -205,7 +198,6 @@ static inline int numChannels(const std::string & encoding) { return 2; } -#pragma GCC diagnostic pop throw std::runtime_error("Unknown encoding " + encoding); return -1; @@ -249,8 +241,6 @@ static inline int bitDepth(const std::string & encoding) return std::atoi(m[0].str().c_str()); } -#pragma GCC diagnostic push -#pragma GCC diagnostic ignored "-Wdeprecated-declarations" if (encoding == YUV422 || encoding == YUV422_YUY2 || encoding == UYVY || @@ -260,7 +250,6 @@ static inline int bitDepth(const std::string & encoding) { return 8; } -#pragma GCC diagnostic pop throw std::runtime_error("Unknown encoding " + encoding); return -1; diff --git a/sensor_msgs/test/test_image_encodings.cpp b/sensor_msgs/test/test_image_encodings.cpp index c2aefd5f..9cfef93e 100644 --- a/sensor_msgs/test/test_image_encodings.cpp +++ b/sensor_msgs/test/test_image_encodings.cpp @@ -48,8 +48,8 @@ TEST(sensor_msgs, NumChannels) ASSERT_EQ(sensor_msgs::image_encodings::numChannels("64FC"), 1); ASSERT_EQ(sensor_msgs::image_encodings::numChannels("64FC3"), 3); ASSERT_EQ(sensor_msgs::image_encodings::numChannels("64FC10"), 10); - ASSERT_EQ(sensor_msgs::image_encodings::numChannels("uyvy"), 2); - ASSERT_EQ(sensor_msgs::image_encodings::numChannels("yuyv"), 2); + ASSERT_EQ(sensor_msgs::image_encodings::numChannels("yuv422"), 2); + ASSERT_EQ(sensor_msgs::image_encodings::numChannels("yuv422_yuy2"), 2); } TEST(sensor_msgs, bitDepth) @@ -68,6 +68,6 @@ TEST(sensor_msgs, bitDepth) ASSERT_EQ(sensor_msgs::image_encodings::bitDepth("64FC"), 64); ASSERT_EQ(sensor_msgs::image_encodings::bitDepth("64FC3"), 64); ASSERT_EQ(sensor_msgs::image_encodings::bitDepth("64FC10"), 64); - ASSERT_EQ(sensor_msgs::image_encodings::bitDepth("uyvy"), 8); - ASSERT_EQ(sensor_msgs::image_encodings::bitDepth("yuyv"), 8); + ASSERT_EQ(sensor_msgs::image_encodings::bitDepth("yuv422"), 8); + ASSERT_EQ(sensor_msgs::image_encodings::bitDepth("yuv422_yuy2"), 8); }