Whack all the memory leaks

Author: tiennou

Classes/Controllers/PBGitCommitController.m

@@ -135,6 +135,7 @@ - (void)closeView
 {
 	[[NSNotificationCenter defaultCenter] removeObserver:self];
 	[webController closeView];
+	[super closeView];
 }
 
 - (NSResponder *)firstResponder;

Classes/Controllers/PBGitSidebarController.m

@@ -125,6 +125,11 @@ - (void)awakeFromNib
 	[[NSNotificationCenter defaultCenter] addObserver:self selector:@selector(expandCollapseItem:) name:NSOutlineViewItemWillCollapseNotification object:sourceView];
 }
 
+- (void)closeView
+{
+	[super closeView];
+}
+
 - (void)dealloc
 {
 	[[NSNotificationCenter defaultCenter] removeObserver:self name:NSOutlineViewItemWillExpandNotification object:sourceView];

Classes/Controllers/PBViewController.m

@@ -35,7 +35,9 @@ - (void)closeView
 {
 	[self unbind:@"repository"];
 	if (_hasViewLoaded)
-		[[self view] removeFromSuperview]; // remove the current view
+		[[self view] removeFromSuperview];	// remove the current view
+	repository = nil;
+	superController = nil;
 }
 
 - (void)awakeFromNib

Classes/Views/PBSourceViewItem.h

@@ -40,7 +40,7 @@ NS_ASSUME_NONNULL_BEGIN
 @property (getter=isUncollapsible) BOOL uncollapsible;
 @property (getter=isExpanded) BOOL expanded;
 @property PBGitRevSpecifier *revSpecifier;
-@property PBSourceViewItem *parent;
+@property (weak) PBSourceViewItem *parent;
 @property (readonly) NSString *iconName;
 @property (nullable, readonly) NSImage *icon;