Relintai/pandemonium_engine
1
0
Fork 0
mirror of https://github.com/Relintai/pandemonium_engine.git synced 2025-02-07 08:35:55 +01:00
Code Issues Packages Projects Releases Wiki Activity

Ported: iOS: Fix memory leak on touch input

Browse Source 
Replaces iOS gesture with touch implementation
Fixes 66422
Remove godot_view_gesture_recognizer
It's now unused.
Remove input_devices/pointing/ios/touch_delay
Unused with removal of gesture.
Remove unused methods from interface
Implementation made obsolete in prior commit
Style conformance
- tbveralrud
28b11a0785
This commit is contained in:
Relintai 2023-03-12 14:08:29 +01:00
parent 304c45fbf3
commit 75ed28f2b3
7 changed files with 41 additions and 289 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
doc/classes/ProjectSettings.xml
View File

@ -779,9 +779,6 @@
<member name="input_devices/pointing/emulate_touch_from_mouse" type="bool" setter="" getter="" default="false"> <member name="input_devices/pointing/emulate_touch_from_mouse" type="bool" setter="" getter="" default="false">
If [code]true[/code], sends touch input events when clicking or dragging the mouse. If [code]true[/code], sends touch input events when clicking or dragging the mouse.
</member> </member>
<member name="input_devices/pointing/ios/touch_delay" type="float" setter="" getter="" default="0.15">
Default delay for touch events. This only affects iOS devices.
</member>
<member name="layer_names/2d_navigation/layer_1" type="String" setter="" getter="" default="&quot;&quot;"> <member name="layer_names/2d_navigation/layer_1" type="String" setter="" getter="" default="&quot;&quot;">
Optional name for the 2D navigation layer 1. If left empty, the layer will display as "Layer 1". Optional name for the 2D navigation layer 1. If left empty, the layer will display as "Layer 1".
</member> </member>

5
main/main.cpp
View File

@ -1249,11 +1249,6 @@ Error Main::setup(const char *execpath, int argc, char *argv[], bool p_second_ph
GLOBAL_DEF("display/window/ios/hide_home_indicator", true); GLOBAL_DEF("display/window/ios/hide_home_indicator", true);
GLOBAL_DEF("display/window/ios/hide_status_bar", true); GLOBAL_DEF("display/window/ios/hide_status_bar", true);
GLOBAL_DEF("display/window/ios/suppress_ui_gesture", true); GLOBAL_DEF("display/window/ios/suppress_ui_gesture", true);
GLOBAL_DEF("input_devices/pointing/ios/touch_delay", 0.15);
ProjectSettings::get_singleton()->set_custom_property_info("input_devices/pointing/ios/touch_delay",
PropertyInfo(Variant::REAL,
"input_devices/pointing/ios/touch_delay",
PROPERTY_HINT_RANGE, "0,1,0.001"));
Engine::get_singleton()->set_frame_delay(frame_delay); Engine::get_singleton()->set_frame_delay(frame_delay);

1
platform/iphone/SCsub
View File

@ -14,7 +14,6 @@ iphone_lib = [
"display_layer.mm", "display_layer.mm",
"pandemonium_app_delegate.m", "pandemonium_app_delegate.m",
"pandemonium_view_renderer.mm", "pandemonium_view_renderer.mm",
"pandemonium_view_gesture_recognizer.mm",
"device_metrics.m", "device_metrics.m",
"keyboard_input_view.mm", "keyboard_input_view.mm",
"native_video_view.m", "native_video_view.m",

5
platform/iphone/pandemonium_view.h
View File

@ -65,9 +65,4 @@ class String;
@property(nonatomic, assign) BOOL useCADisplayLink; @property(nonatomic, assign) BOOL useCADisplayLink;
- (void)pandemoniumTouchesBegan:(NSSet *)touches withEvent:(UIEvent *)event;
- (void)pandemoniumTouchesMoved:(NSSet *)touches withEvent:(UIEvent *)event;
- (void)pandemoniumTouchesEnded:(NSSet *)touches withEvent:(UIEvent *)event;
- (void)pandemoniumTouchesCancelled:(NSSet *)touches withEvent:(UIEvent *)event;
@end @end

108
platform/iphone/pandemonium_view.mm
View File

@ -38,7 +38,6 @@
#import <QuartzCore/QuartzCore.h> #import <QuartzCore/QuartzCore.h>
#import "display_layer.h" #import "display_layer.h"
#import "pandemonium_view_gesture_recognizer.h"
#import "pandemonium_view_renderer.h" #import "pandemonium_view_renderer.h"
#import <CoreMotion/CoreMotion.h> #import <CoreMotion/CoreMotion.h>
@ -62,8 +61,6 @@ static const int max_touches = 8;
@property(strong, nonatomic) CMMotionManager *motionManager; @property(strong, nonatomic) CMMotionManager *motionManager;
@property(strong, nonatomic) PandemoniumViewGestureRecognizer *delayGestureRecognizer;
@end @end
@implementation PandemoniumView @implementation PandemoniumView
@ -134,10 +131,6 @@ static const int max_touches = 8;
[self.animationTimer invalidate]; [self.animationTimer invalidate];
self.animationTimer = nil; self.animationTimer = nil;
} }
if (self.delayGestureRecognizer) {
self.delayGestureRecognizer = nil;
}
} }
- (void)pandemonium_commonInit { - (void)pandemonium_commonInit {
@ -157,11 +150,6 @@ static const int max_touches = 8;
self.motionManager = nil; self.motionManager = nil;
} }
} }
// Initialize delay gesture recognizer
PandemoniumViewGestureRecognizer *gestureRecognizer = [[PandemoniumViewGestureRecognizer alloc] init];
self.delayGestureRecognizer = gestureRecognizer;
[self addGestureRecognizer:self.delayGestureRecognizer];
} }
- (void)startRendering { - (void)startRendering {
@ -333,75 +321,61 @@ static const int max_touches = 8;
} }
} }
- (void)pandemoniumTouchesBegan:(NSSet *)touchesSet withEvent:(UIEvent *)event { - (void)touchesBegan:(NSSet *)touches withEvent:(UIEvent *)event {
NSArray *tlist = [event.allTouches allObjects]; for (UITouch *touch in touches) {
for (unsigned int i = 0; i < [tlist count]; i++) { int tid = [self getTouchIDForTouch:touch];
if ([touchesSet containsObject:[tlist objectAtIndex:i]]) { ERR_FAIL_COND(tid == -1);
UITouch *touch = [tlist objectAtIndex:i]; CGPoint touchPoint = [touch locationInView:self];
int tid = [self getTouchIDForTouch:touch];
ERR_FAIL_COND(tid == -1);
CGPoint touchPoint = [touch locationInView:self];
if (touch.type == UITouchTypeStylus) { if (touch.type == UITouchTypeStylus) {
OSIPhone::get_singleton()->pencil_press(tid, touchPoint.x * self.contentScaleFactor, touchPoint.y * self.contentScaleFactor, true, touch.tapCount > 1); OSIPhone::get_singleton()->pencil_press(tid, touchPoint.x * self.contentScaleFactor, touchPoint.y * self.contentScaleFactor, true, touch.tapCount > 1);
} else { } else {
OSIPhone::get_singleton()->touch_press(tid, touchPoint.x * self.contentScaleFactor, touchPoint.y * self.contentScaleFactor, true, touch.tapCount > 1); OSIPhone::get_singleton()->touch_press(tid, touchPoint.x * self.contentScaleFactor, touchPoint.y * self.contentScaleFactor, true, touch.tapCount > 1);
}
} }
} }
} }
- (void)pandemoniumTouchesMoved:(NSSet *)touches withEvent:(UIEvent *)event { - (void)touchesMoved:(NSSet *)touches withEvent:(UIEvent *)event {
NSArray *tlist = [event.allTouches allObjects]; for (UITouch *touch in touches) {
for (unsigned int i = 0; i < [tlist count]; i++) { int tid = [self getTouchIDForTouch:touch];
if ([touches containsObject:[tlist objectAtIndex:i]]) { ERR_FAIL_COND(tid == -1);
UITouch *touch = [tlist objectAtIndex:i]; CGPoint touchPoint = [touch locationInView:self];
int tid = [self getTouchIDForTouch:touch]; CGPoint prev_point = [touch previousLocationInView:self];
ERR_FAIL_COND(tid == -1); CGFloat force = touch.force;
CGPoint touchPoint = [touch locationInView:self];
CGPoint prev_point = [touch previousLocationInView:self];
CGFloat force = touch.force;
// Vector2 tilt = touch.azimuthUnitVector;
if (touch.type == UITouchTypeStylus) { if (touch.type == UITouchTypeStylus) {
OSIPhone::get_singleton()->pencil_drag(tid, prev_point.x * self.contentScaleFactor, prev_point.y * self.contentScaleFactor, touchPoint.x * self.contentScaleFactor, touchPoint.y * self.contentScaleFactor, force); OSIPhone::get_singleton()->pencil_drag(tid, prev_point.x * self.contentScaleFactor, prev_point.y * self.contentScaleFactor, touchPoint.x * self.contentScaleFactor, touchPoint.y * self.contentScaleFactor, force);
} else { } else {
OSIPhone::get_singleton()->touch_drag(tid, prev_point.x * self.contentScaleFactor, prev_point.y * self.contentScaleFactor, touchPoint.x * self.contentScaleFactor, touchPoint.y * self.contentScaleFactor); OSIPhone::get_singleton()->touch_drag(tid, prev_point.x * self.contentScaleFactor, prev_point.y * self.contentScaleFactor, touchPoint.x * self.contentScaleFactor, touchPoint.y * self.contentScaleFactor);
}
} }
} }
} }
- (void)pandemoniumTouchesEnded:(NSSet *)touches withEvent:(UIEvent *)event { - (void)touchesEnded:(NSSet *)touches withEvent:(UIEvent *)event {
NSArray *tlist = [event.allTouches allObjects]; for (UITouch *touch in touches) {
for (unsigned int i = 0; i < [tlist count]; i++) { int tid = [self getTouchIDForTouch:touch];
if ([touches containsObject:[tlist objectAtIndex:i]]) { ERR_FAIL_COND(tid == -1);
UITouch *touch = [tlist objectAtIndex:i]; [self removeTouch:touch];
int tid = [self getTouchIDForTouch:touch];
ERR_FAIL_COND(tid == -1); CGPoint touchPoint = [touch locationInView:self];
[self removeTouch:touch];
CGPoint touchPoint = [touch locationInView:self]; if (touch.type == UITouchTypeStylus) {
if (touch.type == UITouchTypeStylus) { OSIPhone::get_singleton()->pencil_press(tid, touchPoint.x * self.contentScaleFactor, touchPoint.y * self.contentScaleFactor, false, false);
OSIPhone::get_singleton()->pencil_press(tid, touchPoint.x * self.contentScaleFactor, touchPoint.y * self.contentScaleFactor, false, false); } else {
} else { OSIPhone::get_singleton()->touch_press(tid, touchPoint.x * self.contentScaleFactor, touchPoint.y * self.contentScaleFactor, false, false);
OSIPhone::get_singleton()->touch_press(tid, touchPoint.x * self.contentScaleFactor, touchPoint.y * self.contentScaleFactor, false, false);
}
} }
} }
} }
- (void)pandemoniumTouchesCancelled:(NSSet *)touches withEvent:(UIEvent *)event { - (void)touchesCancelled:(NSSet *)touches withEvent:(UIEvent *)event {
NSArray *tlist = [event.allTouches allObjects]; for (UITouch *touch in touches) {
for (unsigned int i = 0; i < [tlist count]; i++) { int tid = [self getTouchIDForTouch:touch];
if ([touches containsObject:[tlist objectAtIndex:i]]) { ERR_FAIL_COND(tid == -1);
UITouch *touch = [tlist objectAtIndex:i];
int tid = [self getTouchIDForTouch:touch]; if (touch.type == UITouchTypeStylus) {
ERR_FAIL_COND(tid == -1); OSIPhone::get_singleton()->pencil_cancelled(tid);
if (touch.type == UITouchTypeStylus) { } else {
OSIPhone::get_singleton()->pencil_cancelled(tid); OSIPhone::get_singleton()->touches_cancelled(tid);
} else {
OSIPhone::get_singleton()->touches_cancelled(tid);
}
} }
} }
[self clearTouches]; [self clearTouches];

56
platform/iphone/pandemonium_view_gesture_recognizer.h
View File

@ -1,56 +0,0 @@
/*************************************************************************/
/* pandemonium_view_gesture_recognizer.h */
/*************************************************************************/
/* This file is part of: */
/* GODOT ENGINE */
/* https://godotengine.org */
/*************************************************************************/
/* Copyright (c) 2007-2022 Juan Linietsky, Ariel Manzur. */
/* Copyright (c) 2014-2022 Godot Engine contributors (cf. AUTHORS.md). */
/* */
/* Permission is hereby granted, free of charge, to any person obtaining */
/* a copy of this software and associated documentation files (the */
/* "Software"), to deal in the Software without restriction, including */
/* without limitation the rights to use, copy, modify, merge, publish, */
/* distribute, sublicense, and/or sell copies of the Software, and to */
/* permit persons to whom the Software is furnished to do so, subject to */
/* the following conditions: */
/* */
/* The above copyright notice and this permission notice shall be */
/* included in all copies or substantial portions of the Software. */
/* */
/* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, */
/* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF */
/* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.*/
/* IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY */
/* CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, */
/* TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE */
/* SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. */
/*************************************************************************/
// PandemoniumViewGestureRecognizer allows iOS gestures to work correctly by
// emulating UIScrollView's UIScrollViewDelayedTouchesBeganGestureRecognizer.
// It catches all gestures incoming to UIView and delays them for 150ms
// (the same value used by UIScrollViewDelayedTouchesBeganGestureRecognizer)
// If touch cancellation or end message is fired it fires delayed
// begin touch immediately as well as last touch signal
#import <UIKit/UIKit.h>
@interface PandemoniumViewGestureRecognizer : UIGestureRecognizer {
@private
// Timer used to delay begin touch message.
// Should work as simple emulation of UIDelayedAction
NSTimer *delayTimer;
// Delayed touch parameters
NSSet *delayedTouches;
UIEvent *delayedEvent;
}
@property(nonatomic, readonly, assign) NSTimeInterval delayTimeInterval;
- (instancetype)init;
@end

152
platform/iphone/pandemonium_view_gesture_recognizer.mm
View File

@ -1,152 +0,0 @@
/*************************************************************************/
/* pandemonium_view_gesture_recognizer.mm */
/*************************************************************************/
/* This file is part of: */
/* GODOT ENGINE */
/* https://godotengine.org */
/*************************************************************************/
/* Copyright (c) 2007-2022 Juan Linietsky, Ariel Manzur. */
/* Copyright (c) 2014-2022 Godot Engine contributors (cf. AUTHORS.md). */
/* */
/* Permission is hereby granted, free of charge, to any person obtaining */
/* a copy of this software and associated documentation files (the */
/* "Software"), to deal in the Software without restriction, including */
/* without limitation the rights to use, copy, modify, merge, publish, */
/* distribute, sublicense, and/or sell copies of the Software, and to */
/* permit persons to whom the Software is furnished to do so, subject to */
/* the following conditions: */
/* */
/* The above copyright notice and this permission notice shall be */
/* included in all copies or substantial portions of the Software. */
/* */
/* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, */
/* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF */
/* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.*/
/* IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY */
/* CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, */
/* TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE */
/* SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. */
/*************************************************************************/
#import "pandemonium_view_gesture_recognizer.h"
#import "pandemonium_view.h"
#include "core/config/project_settings.h"
// Minimum distance for touches to move to fire
// a delay timer before scheduled time.
// Should be the low enough to not cause issues with dragging
// but big enough to allow click to work.
const CGFloat kGLGestureMovementDistance = 0.5;
@interface PandemoniumViewGestureRecognizer ()
@property(nonatomic, readwrite, assign) NSTimeInterval delayTimeInterval;
@end
@implementation PandemoniumViewGestureRecognizer
- (PandemoniumView *)pandemoniumView {
return (PandemoniumView *)self.view;
}
- (instancetype)init {
self = [super init];
self.cancelsTouchesInView = YES;
self.delaysTouchesBegan = YES;
self.delaysTouchesEnded = YES;
self.requiresExclusiveTouchType = NO;
self.delayTimeInterval = GLOBAL_GET("input_devices/pointing/ios/touch_delay");
return self;
}
- (void)delayTouches:(NSSet *)touches andEvent:(UIEvent *)event {
[delayTimer fire];
delayedTouches = touches;
delayedEvent = event;
delayTimer = [NSTimer scheduledTimerWithTimeInterval:self.delayTimeInterval target:self selector:@selector(fireDelayedTouches:) userInfo:nil repeats:NO];
}
- (void)fireDelayedTouches:(id)timer {
[delayTimer invalidate];
delayTimer = nil;
if (delayedTouches) {
[self.pandemoniumView pandemoniumTouchesBegan:delayedTouches withEvent:delayedEvent];
}
delayedTouches = nil;
delayedEvent = nil;
}
- (void)touchesBegan:(NSSet *)touches withEvent:(UIEvent *)event {
NSSet *cleared = [self copyClearedTouches:touches phase:UITouchPhaseBegan];
[self delayTouches:cleared andEvent:event];
[super touchesBegan:touches withEvent:event];
}
- (void)touchesMoved:(NSSet *)touches withEvent:(UIEvent *)event {
NSSet *cleared = [self copyClearedTouches:touches phase:UITouchPhaseMoved];
if (delayTimer) {
// We should check if movement was significant enough to fire an event
// for dragging to work correctly.
for (UITouch *touch in cleared) {
CGPoint from = [touch locationInView:self.pandemoniumView];
CGPoint to = [touch previousLocationInView:self.pandemoniumView];
CGFloat xDistance = from.x - to.x;
CGFloat yDistance = from.y - to.y;
CGFloat distance = sqrt(xDistance * xDistance + yDistance * yDistance);
// Early exit, since one of touches has moved enough to fire a drag event.
if (distance > kGLGestureMovementDistance) {
[delayTimer fire];
[self.pandemoniumView pandemoniumTouchesMoved:cleared withEvent:event];
return;
}
}
return;
}
[self.pandemoniumView pandemoniumTouchesMoved:cleared withEvent:event];
[super touchesMoved:touches withEvent:event];
}
- (void)touchesEnded:(NSSet *)touches withEvent:(UIEvent *)event {
[delayTimer fire];
NSSet *cleared = [self copyClearedTouches:touches phase:UITouchPhaseEnded];
[self.pandemoniumView pandemoniumTouchesEnded:cleared withEvent:event];
[super touchesEnded:touches withEvent:event];
}
- (void)touchesCancelled:(NSSet *)touches withEvent:(UIEvent *)event {
[delayTimer fire];
[self.pandemoniumView pandemoniumTouchesCancelled:touches withEvent:event];
[super touchesCancelled:touches withEvent:event];
}
- (NSSet *)copyClearedTouches:(NSSet *)touches phase:(UITouchPhase)phaseToSave {
NSMutableSet *cleared = [touches mutableCopy];
for (UITouch *touch in touches) {
if (touch.view != self.view || touch.phase != phaseToSave) {
[cleared removeObject:touch];
}
}
return cleared;
}
@end