-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
remove OF_NO_FMOD #8184
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
remove OF_NO_FMOD #8184
Changes from all commits
ebae252
1060e05
5193705
551e6c8
c35e97b
9787e2a
1a8b6c9
dbda2ab
e0abfcd
88eefa2
44320b8
6229cb2
380a69f
c468077
b8b1550
382a766
a6d5537
9bb39d6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -209,12 +209,8 @@ enum ofTargetPlatform{ | |
| #define TARGET_GLFW_WINDOW | ||
| #define OF_CAIRO | ||
| #define OF_RTAUDIO | ||
| #ifndef __MACOSX_CORE__ | ||
| #define __MACOSX_CORE__ // rtAudio | ||
| #endif | ||
| #ifndef OF_NO_FMOD | ||
| #define OF_NO_FMOD | ||
| #endif | ||
|
|
||
|
|
||
| #include "GL/glew.h" | ||
| #include "OpenGL/OpenGL.h" | ||
|
|
||
|
|
@@ -324,39 +320,31 @@ typedef TESSindex ofIndexType; | |
|
|
||
|
|
||
| #if (defined(_M_ARM64) || defined(_M_ARM64EC)) && defined(TARGET_WIN32) | ||
| #undef USE_FMOD // No FMOD lib for ARM64 yet | ||
| #ifndef OF_NO_FMOD | ||
| #define OF_NO_FMOD | ||
| #endif | ||
| // #undef USE_FMOD // No FMOD lib for ARM64 yet | ||
| #include <arm64_neon.h> // intrinsics SIMD on https://learn.microsoft.com/en-us/cpp/intrinsics/arm64-intrinsics?view=msvc-170 | ||
| #endif | ||
|
|
||
| //------------------------------------------------ soundplayer | ||
| //MAC_OS and IOS uncomment to enable AVEnginePlayer | ||
| #ifdef OF_NO_FMOD | ||
| #undef USE_FMOD | ||
| #if defined(TARGET_OF_IOS) || defined(TARGET_OSX) | ||
| #define OF_SOUND_PLAYER_AV_ENGINE | ||
| #elif defined(TARGET_WIN32) | ||
| #define OF_SOUND_PLAYER_MEDIA_FOUNDATION | ||
| #endif | ||
| #endif | ||
|
|
||
|
|
||
| // check if any soundplayer api is defined from the compiler | ||
|
|
||
| //------------------------------------------------ soundplayer | ||
| #if !defined(TARGET_NO_SOUND) | ||
| #if !defined(OF_SOUND_PLAYER_QUICKTIME) && !defined(OF_SOUND_PLAYER_FMOD) && !defined(OF_SOUND_PLAYER_OPENAL) && !defined(OF_SOUND_PLAYER_EMSCRIPTEN) && !defined(OF_SOUND_PLAYER_AV_ENGINE) && !defined(OF_SOUND_PLAYER_MEDIA_FOUNDATION) | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Removing the checks for the sound player defines won't allow sound player definition from the compiler. Not sure how widely used this functionality is, but seems simple enough to support by keeping the checks. |
||
| #ifdef TARGET_OF_IOS | ||
| #define OF_SOUND_PLAYER_IPHONE | ||
| #elif defined(TARGET_LINUX) || defined(TARGET_MINGW) | ||
| #define OF_SOUND_PLAYER_OPENAL | ||
| #elif defined(TARGET_EMSCRIPTEN) | ||
| #define OF_SOUND_PLAYER_EMSCRIPTEN | ||
| #elif !defined(TARGET_ANDROID) && (!defined(USE_FMOD) || USE_FMOD) | ||
| #define OF_SOUND_PLAYER_FMOD | ||
| #endif | ||
| #endif | ||
|
|
||
| #if defined(USE_FMOD) | ||
| #define OF_SOUND_PLAYER_FMOD | ||
| #else | ||
| #if defined(TARGET_OF_IOS) || defined(TARGET_OSX) | ||
| #define OF_SOUND_PLAYER_AV_ENGINE | ||
| // #elif defined(TARGET_OF_IOS) | ||
| // #define OF_SOUND_PLAYER_IPHONE | ||
| #elif defined(TARGET_LINUX) || defined(TARGET_MINGW) | ||
| #define OF_SOUND_PLAYER_OPENAL | ||
| #elif defined(TARGET_EMSCRIPTEN) | ||
| #define OF_SOUND_PLAYER_EMSCRIPTEN | ||
| #elif defined(TARGET_WIN32) | ||
| #define OF_SOUND_PLAYER_MEDIA_FOUNDATION | ||
| #endif | ||
| #endif | ||
| #endif | ||
|
|
||
| //------------------------------------------------ thread local storage | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,49 +6,56 @@ CLANG_C_LANGUAGE_STANDARD = c17 | |
| MACOSX_DEPLOYMENT_TARGET = 10.15 | ||
|
|
||
|
|
||
| HEADER_OF = "$(OF_PATH)/libs/openFrameworks/**" | ||
| HEADER_FREETYPE = "$(OF_PATH)/libs/freetype/include" | ||
| HEADER_FREETYPE2 = "$(OF_PATH)/libs/freetype/include/freetype2" | ||
| //HEADER_FMOD = "$(OF_PATH)/libs/fmod/include" | ||
| HEADER_GLEW = "$(OF_PATH)/libs/glew/include" | ||
| HEADER_FREEIMAGE = "$(OF_PATH)/libs/FreeImage/include" | ||
| HEADER_TESS2 = "$(OF_PATH)/libs/tess2/include" | ||
| HEADER_CAIRO = "$(OF_PATH)/libs/cairo/include" | ||
| HEADER_RTAUDIO = "$(OF_PATH)/libs/rtAudio/include" | ||
| HEADER_GLFW = "$(OF_PATH)/libs/glfw/include" | ||
| HEADER_UTF8 = "$(OF_PATH)/libs/utf8/include" | ||
| HEADER_JSON = "$(OF_PATH)/libs/json/include" | ||
| HEADER_GLM = "$(OF_PATH)/libs/glm/include" | ||
| HEADER_BROTLI = "$(OF_PATH)/libs/brotli/include" | ||
| HEADER_CURL = "$(OF_PATH)/libs/curl/include" | ||
| HEADER_SSL = "$(OF_PATH)/libs/openssl/include" | ||
| HEADER_URIPARSER = "$(OF_PATH)/libs/uriparser/include" | ||
| HEADER_PUGIXML = "$(OF_PATH)/libs/pugixml/include" | ||
| HEADER_OF = $(OF_PATH)/libs/openFrameworks/** | ||
| HEADER_FREETYPE = $(OF_PATH)/libs/freetype/include | ||
| HEADER_FREETYPE2 = $(OF_PATH)/libs/freetype/include/freetype2 | ||
| //HEADER_FMOD = $(OF_PATH)/libs/fmod/include | ||
| HEADER_GLEW = $(OF_PATH)/libs/glew/include | ||
| HEADER_FREEIMAGE = $(OF_PATH)/libs/FreeImage/include | ||
| HEADER_TESS2 = $(OF_PATH)/libs/tess2/include | ||
| HEADER_CAIRO = $(OF_PATH)/libs/cairo/include | ||
| HEADER_RTAUDIO = $(OF_PATH)/libs/rtAudio/include | ||
| HEADER_GLFW = $(OF_PATH)/libs/glfw/include | ||
| HEADER_UTF8 = $(OF_PATH)/libs/utf8/include | ||
| HEADER_JSON = $(OF_PATH)/libs/json/include | ||
| HEADER_GLM = $(OF_PATH)/libs/glm/include | ||
| HEADER_BROTLI = $(OF_PATH)/libs/brotli/include | ||
| HEADER_CURL = $(OF_PATH)/libs/curl/include | ||
| HEADER_SSL = $(OF_PATH)/libs/openssl/include | ||
| HEADER_URIPARSER = $(OF_PATH)/libs/uriparser/include | ||
| HEADER_PUGIXML = $(OF_PATH)/libs/pugixml/include | ||
|
|
||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Curious about the quote removal - is there any downside to removing? |
||
| //------- Libraries | ||
| LIB_OF_RELEASE = "$(OF_PATH)/libs/openFrameworksCompiled/lib/osx/openFrameworks.a" | ||
| LIB_OF_DEBUG = "$(OF_PATH)/libs/openFrameworksCompiled/lib/osx/openFrameworksDebug.a" | ||
|
|
||
| //LIB_FMOD = "$(OF_PATH)/libs/fmod/lib/macos/libfmod.dylib" | ||
|
|
||
| //LIB_GLFW = "$(OF_PATH)/libs/glfw/lib/osx/glfw3.a" | ||
| //LIB_FREEIMAGE = "$(OF_PATH)/libs/FreeImage/lib/osx/freeimage.a" | ||
| //LIB_FREETYPE = "$(OF_PATH)/libs/freetype/lib/osx/freetype.a" | ||
| //LIB_GLEW = "$(OF_PATH)/libs/glew/lib/osx/glew.a" | ||
| //LIB_RTAUDIO = "$(OF_PATH)/libs/rtAudio/lib/osx/rtaudio.a" | ||
| //LIB_TESS = "$(OF_PATH)/libs/tess2/lib/osx/tess2.a" | ||
| //LIB_CAIRO1 = "$(OF_PATH)/libs/cairo/lib/osx/cairo-script-interpreter.a" | ||
| //LIB_CAIRO2 = "$(OF_PATH)/libs/cairo/lib/osx/cairo.a" | ||
| //LIB_CAIRO3 = "$(OF_PATH)/libs/cairo/lib/osx/pixman-1.a" | ||
| //LIB_FMOD = $(OF_PATH)/libs/fmod/lib/macos/libfmod.dylib | ||
|
|
||
| //LIB_GLFW = $(OF_PATH)/libs/glfw/lib/osx/glfw3.a | ||
| //LIB_FREEIMAGE = $(OF_PATH)/libs/FreeImage/lib/osx/freeimage.a | ||
| //LIB_FREETYPE = $(OF_PATH)/libs/freetype/lib/osx/freetype.a | ||
| //LIB_GLEW = $(OF_PATH)/libs/glew/lib/osx/glew.a | ||
| //LIB_RTAUDIO = $(OF_PATH)/libs/rtAudio/lib/osx/rtaudio.a | ||
| //LIB_TESS = $(OF_PATH)/libs/tess2/lib/osx/tess2.a | ||
| //LIB_CAIRO1 = $(OF_PATH)/libs/cairo/lib/osx/cairo-script-interpreter.a | ||
| //LIB_CAIRO2 = $(OF_PATH)/libs/cairo/lib/osx/cairo.a | ||
| //LIB_CAIRO3 = $(OF_PATH)/libs/cairo/lib/osx/pixman-1.a | ||
| //LIB_CURL = "-lcurl" | ||
| //LIB_URIPARSER = "$(OF_PATH)/libs/uriparser/lib/osx/uriparser.a" | ||
| //LIB_PUGIXML = "$(OF_PATH)/libs/pugixml/lib/osx/pugixml.a" | ||
| //LIB_URIPARSER = $(OF_PATH)/libs/uriparser/lib/osx/uriparser.a | ||
| //LIB_PUGIXML = $(OF_PATH)/libs/pugixml/lib/osx/pugixml.a | ||
|
|
||
|
|
||
|
|
||
|
|
||
| OF_CORE_HEADERS = $(HEADER_OF) $(HEADER_FREETYPE) $(HEADER_FREETYPE2) $(HEADER_GLEW) $(HEADER_FREEIMAGE) $(HEADER_TESS2) $(HEADER_CAIRO) $(HEADER_RTAUDIO) $(HEADER_GLFW) $(HEADER_UTF8) $(HEADER_JSON) $(HEADER_GLM) $(HEADER_CURL) ${HEADER_SSL} $(HEADER_URIPARSER) $(HEADER_PUGIXML) ${HEADER_BROTLI} | ||
|
|
||
| OF_CORE_FRAMEWORKS = -framework Accelerate -framework AppKit -framework ApplicationServices -framework AudioToolbox -framework AVFoundation -framework Cocoa -framework CoreAudio -framework CoreFoundation -framework CoreMedia -framework CoreServices -framework CoreVideo -framework Foundation -framework IOKit -framework OpenGL -framework QuartzCore -framework Security -framework SystemConfiguration -framework Metal | ||
|
|
||
| // FMOD can be enabled in OF Core by uncommenting this block | ||
| //USE_FMOD=1 | ||
| //USER_PREPROCESSOR_DEFINITIONS=USE_FMOD=1 $(inherited) | ||
| //OF_CORE_HEADERS = $(OF_PATH)/libs/fmod/include $(inherited) | ||
| //OF_CORE_LIBS=$(OF_PATH)/libs/fmod/lib/macos/libfmod.dylib $(inherited) | ||
|
|
||
|
|
||
| ALWAYS_SEARCH_USER_PATHS = NO | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Safe to remove even though it mentions rtAudio?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The define might be worth keeping around for legacy purposes. Removing it doesn't seem like it adds much benefit other than being a bit more tidy.