You cannot select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
9876d86d5d
* Various fixes and empty boilerplate files required for restricted platforms. Signed-off-by: bosnichd <bosnichd@amazon.com> * Add comments to address review feedback. Signed-off-by: bosnichd <bosnichd@amazon.com> * Fix two compilation errors exposed when enabling tools support for any restricted platform. - The simple one: remove menu commands that call OnChangeGameSpec (which has since been removed) from CryEdit.cpp - The "I almost threw my computer out the window" one: pull PVRTC.cpp out of unity builds, because it indirectly #includes winnt.h, which typedefs wchar_t WCHAR, which causes a complilation error (Error C2632 'wchar_t' followed by 'wchar_t' is illegal ImageProcessingAtom.Editor.Static C:\Program Files (x86)\Windows Kits\10\Include\10.0.19041.0\um\winnt.h 471) if something else in the unity file has also happened to define WCHAR as wchar_t. Enabling tools support for any restricted platform resulted in this happening due to different compile definitions being set for ImageConvert.cpp and BuilderSettingManager.cpp (see Gems/Atom/Asset/ImageProcessingAtom/Code/CMakeLists.txt), which resulted in those two files being pulled out of unity builds, which resulted in the entirely unrelated PVRTC.cpp file being moved from the first thing included by a unity file to the second last thing included by a different unity file, that just happened to include something else (prior to PVRTC.cpp) which was defining WCHAR as wchar_t. Signed-off-by: bosnichd <bosnichd@amazon.com> |
5 years ago | |
|---|---|---|
| .. | ||
| Editor | 5 years ago | |
| Framework | 5 years ago | |
| LauncherUnified | 5 years ago | |
| Legacy | 5 years ago | |
| Tools | 5 years ago | |
| CMakeLists.txt | 5 years ago | |