MacGui: annotate the type of some arrays, and check the count of the input jobs in...

diff --git a/macosx/HBDistributedArray.h b/macosx/HBDistributedArray.h
index 483f1ac572f77cb595c3b5ebff39fcd7299ebcf9..dee8bfc12729746bf1975f6e0bcac02b248d7014 100644 (file)
--- a/macosx/HBDistributedArray.h
+++ b/macosx/HBDistributedArray.h
@@ -33,7 +33,7 @@ typedef NS_ENUM(NSUInteger, HBDistributedArrayContent) {
  *
  *  It is safe to keep a reference to an array object.
  */
-@interface HBDistributedArray : NSMutableArray
+@interface HBDistributedArray<ObjectType> : NSMutableArray
 
 - (instancetype)initWithURL:(NSURL *)fileURL;
 

diff --git a/macosx/HBDistributedArray.m b/macosx/HBDistributedArray.m
index 867ee4ffbf8790bc3f049827a9b5b141eac9913c..97b8378c3a950f7d93ee27fd06ff6462dac3d0d0 100644 (file)
--- a/macosx/HBDistributedArray.m
+++ b/macosx/HBDistributedArray.m
@@ -52,9 +52,9 @@
 NSString *HBDistributedArrayChanged = @"HBDistributedArrayChanged";
 NSString *HBDistributedArraWrittenToDisk = @"HBDistributedArraWrittenToDisk";
 
-@interface HBDistributedArray ()
+@interface HBDistributedArray<ObjectType> ()
 
-@property (nonatomic, readonly) NSMutableArray *array;
+@property (nonatomic, readonly) NSMutableArray<ObjectType> *array;
 @property (nonatomic, readonly) NSURL *fileURL;
 @property (nonatomic, readwrite) NSTimeInterval modifiedTime;
 

diff --git a/macosx/HBPictureController.m b/macosx/HBPictureController.m
index 083bc23400942010dfb6d8d8db8796c4433036af..8dee131872f01f02aaaa373384a187a5cf91116d 100644 (file)
--- a/macosx/HBPictureController.m
+++ b/macosx/HBPictureController.m
@@ -54,7 +54,7 @@ static void *HBPictureControllerContext = &HBPictureControllerContext;
     {
         for (NSString *keyPath in observerdKeyPaths)
         {
-            [self removeObserver:self forKeyPath:keyPath];
+            [self removeObserver:self forKeyPath:keyPath context:HBPictureControllerContext];
         }
 
     } @catch (NSException * __unused exception) {}

diff --git a/macosx/HBPictureViewController.m b/macosx/HBPictureViewController.m
index 344580ec6b2915604419f681df28d0ce5b3c75c8..87618cb97fc94b1d4c68c19d540a2813f57e55f3 100644 (file)
--- a/macosx/HBPictureViewController.m
+++ b/macosx/HBPictureViewController.m
@@ -38,7 +38,7 @@ static void *HBPictureViewControllerContext = &HBPictureViewControllerContext;
 {
     @try
     {
-        [self removeObserver:self forKeyPath:@"self.picture.modulus"];
+        [self removeObserver:self forKeyPath:@"self.picture.modulus" context:HBPictureViewControllerContext];
     }
     @catch (NSException * __unused exception) {}
 }
@@ -64,7 +64,7 @@ static void *HBPictureViewControllerContext = &HBPictureViewControllerContext;
 {
     if (context == HBPictureViewControllerContext)
     {
-        // Set the increment here, it's not possible with bidings.
+        // Set the increment here, it's not possible with bindings.
         if ([keyPath isEqualToString:@"self.picture.modulus"])
         {
             [self.widthStepper setIncrement:self.picture.modulus];

diff --git a/macosx/HBQueueController.h b/macosx/HBQueueController.h
index ff8ca5aafa0e7cfb5b22c6d2bf640915d83904f2..3584f7737a466f9826db51d9cca6f95eaebbbb76 100644 (file)
--- a/macosx/HBQueueController.h
+++ b/macosx/HBQueueController.h
@@ -29,7 +29,7 @@ NS_ASSUME_NONNULL_BEGIN
 @property (nonatomic, readonly) NSUInteger pendingItemsCount;
 
 - (void)addJob:(HBJob *)item;
-- (void)addJobsFromArray:(NSArray *)items;
+- (void)addJobsFromArray:(NSArray<HBJob *> *)items;
 
 - (BOOL)jobExistAtURL:(NSURL *)url;
 

diff --git a/macosx/HBQueueController.m b/macosx/HBQueueController.m
index 624d037001494a070c6813a1761e09c48d974ce5..537e8c86baa9d1c455973d1ce5c60c584151a53b 100644 (file)
--- a/macosx/HBQueueController.m
+++ b/macosx/HBQueueController.m
@@ -44,7 +44,7 @@
 
 @property (nonatomic, readonly) NSMutableDictionary *descriptions;
 
-@property (nonatomic, readonly) HBDistributedArray *jobs;
+@property (nonatomic, readonly) HBDistributedArray<HBJob *> *jobs;
 @property (nonatomic)   HBJob *currentJob;
 @property (nonatomic)   HBJobOutputFileWriter *currentLog;
 
@@ -53,7 +53,7 @@
 @property (nonatomic, readwrite) NSUInteger pendingItemsCount;
 @property (nonatomic, readwrite) NSUInteger completedItemsCount;
 
-@property (nonatomic) NSArray *dragNodesArray;
+@property (nonatomic) NSArray<HBJob *> *dragNodesArray;
 
 @end
 
@@ -213,10 +213,13 @@
     [self addJobsFromArray:@[item]];
 }
 
-- (void)addJobsFromArray:(NSArray *)items;
+- (void)addJobsFromArray:(NSArray<HBJob *> *)items;
 {
     NSParameterAssert(items);
-    [self addQueueItems:items];
+    if (items.count)
+    {
+        [self addQueueItems:items];
+    }
 }
 
 - (BOOL)jobExistAtURL:(NSURL *)url
@@ -937,7 +940,7 @@
         if ([targetedRows containsIndexes:workingIndexes])
         {
             [targetedRows removeIndexes:workingIndexes];
-            NSArray *workingJobs = [self.jobs filteredArrayUsingBlock:^BOOL(HBJob *item) {
+            NSArray<HBJob *> *workingJobs = [self.jobs filteredArrayUsingBlock:^BOOL(HBJob *item) {
                 return item.state == HBJobStateWorking;
             }];
 

diff --git a/macosx/HBSubtitlesDefaultsController.m b/macosx/HBSubtitlesDefaultsController.m
index 678ceb20432e7423b92e83843ff6b12ecc9708ef..a78a14b48b72424061272ca96929069761cb634e 100644 (file)
--- a/macosx/HBSubtitlesDefaultsController.m
+++ b/macosx/HBSubtitlesDefaultsController.m
@@ -8,7 +8,7 @@
 #import "HBSubtitlesDefaults.h"
 #import "HBLanguagesSelection.h"
 
-static void *HBSubtitlesDefaultsContex = &HBSubtitlesDefaultsContex;
+static void *HBSubtitlesDefaultsContext = &HBSubtitlesDefaultsContext;
 
 @interface HBSubtitlesDefaultsController ()
 
@@ -36,7 +36,7 @@ static void *HBSubtitlesDefaultsContex = &HBSubtitlesDefaultsContex;
 
 - (void)windowDidLoad
 {
-    [self addObserver:self forKeyPath:@"tableController.showSelectedOnly" options:0 context:HBSubtitlesDefaultsContex];
+    [self addObserver:self forKeyPath:@"tableController.showSelectedOnly" options:0 context:HBSubtitlesDefaultsContext];
 
     if (self.settings.trackSelectionLanguages.count)
     {
@@ -46,7 +46,7 @@ static void *HBSubtitlesDefaultsContex = &HBSubtitlesDefaultsContex;
 
 - (void)observeValueForKeyPath:(NSString *)keyPath ofObject:(id)object change:(NSDictionary *)change context:(void *)context
 {
-    if (context == HBSubtitlesDefaultsContex)
+    if (context == HBSubtitlesDefaultsContext)
     {
         if ([keyPath isEqualToString:@"tableController.showSelectedOnly"])
         {
@@ -77,7 +77,7 @@ static void *HBSubtitlesDefaultsContex = &HBSubtitlesDefaultsContex;
 - (void)dealloc
 {
     @try {
-        [self removeObserver:self forKeyPath:@"tableController.showSelectedOnly"];
+        [self removeObserver:self forKeyPath:@"tableController.showSelectedOnly" context:HBSubtitlesDefaultsContext];
     } @catch (NSException * __unused exception) {}
 }