It appears the image conversion functions could do with a clean-up. While they seem to be declared Image/Base/ImageConv.hh, the definitions are scattered over Image/Base/ImageConv*.cc, Image/Base/ImgCnv*.cc and Image/ImageIO/*.cc. The Image/Base filenames are cryptic, which doesn't help searching for individual definitions, and the conversion functions aren't file-related so Image/ImageIO doesn't seem like a suitable home for them either.
My suggestion would be a separate file for each group of related conversions, such as ConvertYUV422.cc, so that to find a implementation you would need to look in at most two files e.g. YUV to RGB would be in either ConvertYUV.cc or ConvertRGB.cc. If we end up with an unreasonable number of conversion files, these could be placed in a separate folder so as not to 'hide' more important files in Image/Base.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
It appears the image conversion functions could do with a clean-up. While they seem to be declared Image/Base/ImageConv.hh, the definitions are scattered over Image/Base/ImageConv*.cc, Image/Base/ImgCnv*.cc and Image/ImageIO/*.cc. The Image/Base filenames are cryptic, which doesn't help searching for individual definitions, and the conversion functions aren't file-related so Image/ImageIO doesn't seem like a suitable home for them either.
My suggestion would be a separate file for each group of related conversions, such as ConvertYUV422.cc, so that to find a implementation you would need to look in at most two files e.g. YUV to RGB would be in either ConvertYUV.cc or ConvertRGB.cc. If we end up with an unreasonable number of conversion files, these could be placed in a separate folder so as not to 'hide' more important files in Image/Base.