Merge pull request #36 from bruvzg/mac_rename

Author: akien-mga

.github/workflows/ci.yml

@@ -26,7 +26,7 @@ jobs:
 
       - name: Generate Headers
         run: |
-          ./scripts/generate_headers.sh || true
+          ./scripts/generate_headers.sh 3.x || true
 
       - name: Compile Plugins
         run: |

README.md

@@ -36,10 +36,18 @@ If you choose this option, you can skip next the step which generates Godot head
 
 - To generate Godot headers, you need to run the compilation command inside the `godot` submodule directory:
 
+### For Godot 3.x:
+
 ```bash
 scons platform=iphone target=debug
 ```
 
+### For Godot 4.x:
+
+```bash
+scons platform=ios target=debug
+```
+
 You don't have to wait for full engine compilation, as header files are generated first.
 Once the actual compilation starts, you can stop this command by pressing <kbd>Ctrl + C</kbd>.
 

SConstruct

@@ -82,11 +82,11 @@ env.Append(CCFLAGS=[
     # '-Wextra',
 ])
 
-env.Append(CCFLAGS=['-arch', env['arch'], "-isysroot", "$IPHONESDK", "-stdlib=libc++", '-isysroot', sdk_path])
+env.Append(CCFLAGS=['-arch', env['arch'], "-isysroot", "$IOS_SDK_PATH", "-stdlib=libc++", '-isysroot', sdk_path])
 env.Append(CCFLAGS=['-DPTRCALL_ENABLED'])
 env.Prepend(CXXFLAGS=[
     '-DNEED_LONG_INT', '-DLIBYUV_DISABLE_NEON', 
-    '-DIPHONE_ENABLED', '-DUNIX_ENABLED', '-DCOREAUDIO_ENABLED'
+    '-DIOS_ENABLED', '-DUNIX_ENABLED', '-DCOREAUDIO_ENABLED'
 ])
 env.Append(LINKFLAGS=["-arch", env['arch'], '-isysroot', sdk_path, '-F' + sdk_path])
 
@@ -152,19 +152,26 @@ else:
     quit();
 
 # Adding header files
-env.Append(CPPPATH=[
-    '.', 
-    'godot', 
-    'godot/platform/iphone',
-])
+if env['version'] == '3.x':
+    env.Append(CPPPATH=[
+        '.', 
+        'godot', 
+        'godot/platform/iphone',
+    ])
+else:
+       env.Append(CPPPATH=[
+        '.', 
+        'godot', 
+        'godot/platform/ios',
+    ])
 
 # tweak this if you want to use different folders, or more folders, to store your source code in.
 sources = Glob('plugins/' + env['plugin'] + '/*.cpp')
 sources.append(Glob('plugins/' + env['plugin'] + '/*.mm'))
 sources.append(Glob('plugins/' + env['plugin'] + '/*.m'))
 
-# lib<plugin>.<arch>-<simulator|iphone>.<release|debug|release_debug>.a
-library_platform = env["arch"] + "-" + ("simulator" if env["simulator"] else "iphone")
+# lib<plugin>.<arch>-<simulator|ios>.<release|debug|release_debug>.a
+library_platform = env["arch"] + "-" + ("simulator" if env["simulator"] else "ios")
 library_name = env['plugin'] + "." + library_platform + "." + env["target"] + ".a"
 library = env.StaticLibrary(target=env['target_path'] + library_name, source=sources)
 

plugins/apn/apn.h

@@ -31,7 +31,13 @@
 #ifndef godot_apn_implementation_h
 #define godot_apn_implementation_h
 
+#include "core/version.h"
+
+#if VERSION_MAJOR == 4
+#include "core/object/class_db.h"
+#else
 #include "core/object.h"
+#endif
 
 class APNPlugin : public Object {
 	GDCLASS(APNPlugin, Object);

plugins/apn/apn.mm

@@ -32,8 +32,13 @@
 
 #import <Foundation/Foundation.h>
 
+#if VERSION_MAJOR == 4
+#include "core/object/class_db.h"
+#include "core/config/project_settings.h"
+#else
 #include "core/class_db.h"
 #include "core/project_settings.h"
+#endif
 
 #import "godot_apn_delegate.h"
 

plugins/apn/apn_plugin.cpp

@@ -31,7 +31,11 @@
 #import "apn_plugin.h"
 #import "apn.h"
 
+#if VERSION_MAJOR == 4
+#import "core/config/engine.h"
+#else
 #import "core/engine.h"
+#endif
 
 APNPlugin *plugin;
 

plugins/apn/godot_apn_delegate.mm

@@ -33,7 +33,12 @@
 #include "apn.h"
 
 #import "godot_user_notification_delegate.h"
+
+#if VERSION_MAJOR == 4
+#import "platform/ios/godot_app_delegate.h"
+#else
 #import "platform/iphone/godot_app_delegate.h"
+#endif
 
 struct APNSInitializer {
 

plugins/apn/godot_app_delegate_extension.h

@@ -28,7 +28,13 @@
 /* SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.                */
 /*************************************************************************/
 
+#include "core/version.h"
+
+#if VERSION_MAJOR == 4
+#import "platform/ios/godot_app_delegate.h"
+#else
 #import "platform/iphone/godot_app_delegate.h"
+#endif
 
 @interface GodotApplicalitionDelegate (PushNotifications)
 

plugins/arkit/arkit_interface.h

@@ -60,7 +60,7 @@ typedef PoolVector<uint8_t> GodotUInt8Vector;
 /**
 	@author Bastiaan Olij <[email protected]>
 
-	ARKit interface between iPhone and Godot
+	ARKit interface between iOS and Godot
 */
 
 // forward declaration for some needed objects
@@ -142,7 +142,7 @@ class ARKitInterface : public GodotBaseARInterface {
 
 	virtual void process() GODOT_ARKIT_OVERRIDE;
 
-	// called by delegate (void * because C++ and Obj-C don't always mix, should really change all platform/iphone/*.cpp files to .mm)
+	// called by delegate (void * because C++ and Obj-C don't always mix, should really change all platform/ios/*.cpp files to .mm)
 	void _add_or_update_anchor(GodotARAnchor *p_anchor);
 	void _remove_anchor(GodotARAnchor *p_anchor);
 

plugins/camera/camera_ios.mm

@@ -28,7 +28,7 @@
 /* SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.                */
 /*************************************************************************/
 
-///@TODO this is a near duplicate of CameraOSX, we should find a way to combine those to minimize code duplication!!!!
+///@TODO this is a near duplicate of CameraMacOS, we should find a way to combine those to minimize code duplication!!!!
 // If you fix something here, make sure you fix it there as wel!
 
 #include "camera_ios.h"
@@ -182,12 +182,12 @@ - (void)captureOutput:(AVCaptureOutput *)captureOutput didOutputSampleBuffer:(CM
 #if VERSION_MAJOR == 4
 			uint8_t *w = img_data[0].ptrw();
 			memcpy(w, dataY, new_width * new_height);
+			img[0].instantiate();
 #else
 			GodotUInt8Vector::Write w = img_data[0].write();
 			memcpy(w.ptr(), dataY, new_width * new_height);
-#endif
-
 			img[0].instance();
+#endif
 			img[0]->create(new_width, new_height, 0, Image::FORMAT_R8, img_data[0]);
 		}
 
@@ -205,13 +205,14 @@ - (void)captureOutput:(AVCaptureOutput *)captureOutput didOutputSampleBuffer:(CM
 #if VERSION_MAJOR == 4
 			uint8_t *w = img_data[1].ptrw();
 			memcpy(w, dataCbCr, 2 * new_width * new_height);
+			img[1].instantiate();
 #else
 			GodotUInt8Vector::Write w = img_data[1].write();
 			memcpy(w.ptr(), dataCbCr, 2 * new_width * new_height);
+			img[1].instance();
 #endif
 
 			///TODO GLES2 doesn't support FORMAT_RG8, need to do some form of conversion
-			img[1].instance();
 			img[1]->create(new_width, new_height, 0, Image::FORMAT_RG8, img_data[1]);
 		}
 
@@ -358,7 +359,7 @@ - (void)dealloc {
 MyDeviceNotifications *device_notifications = nil;
 
 //////////////////////////////////////////////////////////////////////////
-// CameraIOS - Subclass for our camera server on iPhone
+// CameraIOS - Subclass for our camera server on iOS
 
 void CameraIOS::update_feeds() {
 	// this way of doing things is deprecated but still works,
@@ -410,7 +411,11 @@ - (void)dealloc {
 
 		if (!found) {
 			Ref<CameraFeedIOS> newfeed;
+#if VERSION_MAJOR == 4
+			newfeed.instantiate();
+#else
 			newfeed.instance();
+#endif
 			newfeed->set_device(device);
 			add_feed(newfeed);
 		}