Fix for non-unity file and double declaration of WCHAR (#3657)

Signed-off-by: Esteban Papp <81431996+amznestebanpapp@users.noreply.github.com>

Gems/Atom/Asset/ImageProcessingAtom/Code/CMakeLists.txt

@@ -48,7 +48,7 @@ ly_add_target(
     PLATFORM_INCLUDE_FILES
         ${pal_source_dir}/platform_${PAL_PLATFORM_NAME_LOWERCASE}.cmake
         ${common_source_dir}/${PAL_TRAIT_COMPILER_ID}/imageprocessingatom_editor_static_${PAL_TRAIT_COMPILER_ID_LOWERCASE}.cmake
-         ${platform_tools_files}
+        ${platform_tools_files}
     INCLUDE_DIRECTORIES
         PUBLIC
             Include

Gems/Atom/Asset/ImageProcessingAtom/Code/Source/Compressors/Compressor.cpp

@@ -7,6 +7,7 @@
  */
 
 
+#include <AzCore/PlatformIncl.h>
 #include <Compressors/CTSquisher.h>
 #include <Compressors/PVRTC.h>
 #include <Compressors/ETC2.h>

Gems/Atom/Asset/ImageProcessingAtom/Code/Source/Compressors/PVRTC.cpp

@@ -7,20 +7,12 @@
  */
 
 #include <ImageProcessing_Traits_Platform.h>
+#include <AzCore/PlatformIncl.h>
 #include <Atom/ImageProcessing/ImageObject.h>
 #include <Processing/ImageFlags.h>
 #include <Processing/PixelFormatInfo.h>
 #include <Compressors/PVRTC.h>
 
-#if AZ_TRAIT_IMAGEPROCESSING_PVRTEXLIB_USE_WINDLL_IMPORT
-//_WINDLL_IMPORT need to be defined before including PVRTexLib header files to avoid linking error on windows.
-#define _WINDLL_IMPORT
-// NOMINMAX needs to be defined before including PVRTexLib header files (which include Windows.h)
-// so that Windows.h doesn't define min/max. Otherwise, a compile error may arise in Uber builds
-#ifndef NOMINMAX
-#define NOMINMAX
-#endif
-#endif
 #include <PVRTexture.h>
 #include <PVRTextureUtilities.h>
 

Gems/Atom/Asset/ImageProcessingAtom/Code/imageprocessing_files.cmake

@@ -136,7 +136,3 @@ set(FILES
     Source/Thumbnail/ImageThumbnailSystemComponent.cpp
     Source/Thumbnail/ImageThumbnailSystemComponent.h
 )
-
-set(SKIP_UNITY_BUILD_INCLUSION_FILES
-    Source/Compressors/PVRTC.cpp
-)

Gems/Atom/Asset/ImageProcessingAtom/External/CubeMapGen/CCubeMapProcessor.h

@@ -14,6 +14,7 @@
 #include <stdio.h>
 #include <AzCore/std/parallel/thread.h>
 #include <AzCore/std/parallel/atomic.h>
+#include <AzCore/PlatformIncl.h>
 
 #include "VectorMacros.h"
 #include "CBBoxInt32.h"
@@ -22,11 +23,9 @@
 //has routines for saving .rgbe files
 #define CG_RGBE_SUPPORT
 
-
-#ifndef WCHAR 
+#ifndef WCHAR // For non-windows platforms, for Windows-based platforms it will be defined through PlatformIncl.h
 #define WCHAR wchar_t
-#endif //WCHAR 
-
+#endif // WCHAR
 
 //used to index cube faces
 #define CP_FACE_X_POS 0

Gems/Atom/Asset/ImageProcessingAtom/External/CubeMapGen/CImageSurface.h

@@ -14,10 +14,11 @@
 
 #include "VectorMacros.h"
 #include <ImageBuilderBaseType.h>
+#include <AzCore/PlatformIncl.h>
 
-#ifndef WCHAR 
+#ifndef WCHAR // For non-windows platforms, for Windows-based platforms it will be defined through PlatformIncl.h
 #define WCHAR wchar_t
-#endif //WCHAR 
+#endif // WCHAR
 
 #ifndef SAFE_DELETE 
 #define SAFE_DELETE(p)       { if(p) { delete (p);   (p)=NULL; } }