Merge pull request #9 from MerrickSapsford/protocol-fix

Fix incorrect strong delegates
uias · Jun 21, 2016 · c01e9fc · c01e9fc
2 parents cae6a5d + 6fb4126
commit c01e9fc
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/Sources/CocoaBar.swift b/Sources/CocoaBar.swift
@@ -14,7 +14,7 @@ private let CocoaBarAnimatedKey: String =       "animated"
 public typealias CocoaBarPopulationClosure = (layout: CocoaBarLayout) -> Void
 public typealias CocoaBarAnimationCompletionClosure = (animated: Bool, completed: Bool, visible: Bool) -> Void
 
-public protocol CocoaBarDelegate: Any {
+public protocol CocoaBarDelegate: class {
 
     /**
      The action button on the CocoaBar has been pressed.
@@ -168,7 +168,7 @@ public class CocoaBar: UIView, CocoaBarLayoutDelegate {
     /**
      The object that acts as a delegate to the CocoaBar
      */
-    public var delegate: CocoaBarDelegate?
+    public weak var delegate: CocoaBarDelegate?
 
     /**
      The CocoaBar that is attached to the key window.

diff --git a/Sources/CocoaBarLayout.swift b/Sources/CocoaBarLayout.swift
@@ -73,7 +73,7 @@ public class CocoaBarLayout: DropShadowView {
      The object that acts as a delegate to the layout.
      This should always be the CocoaBar
     */
-    internal var delegate: CocoaBarLayoutDelegate?
+    internal weak var delegate: CocoaBarLayoutDelegate?
 
     /**
      The dismiss button on the layout
@@ -314,7 +314,7 @@ public class CocoaBarLayout: DropShadowView {
     }
 }
 
-internal protocol CocoaBarLayoutDelegate: Any {
+internal protocol CocoaBarLayoutDelegate: class {
 
     /**
      The dismiss button has been pressed on the layout.