macOS: Clarify useless-looking code in VLCDocumentController
authorMarvin Scholz <epirat07@gmail.com>
Fri, 2 Jun 2017 16:12:02 +0000 (18:12 +0200)
committerMarvin Scholz <epirat07@gmail.com>
Fri, 2 Jun 2017 16:12:08 +0000 (18:12 +0200)
Clarify why the VLCDocumentController is allocated and initialized in
the load method, so other developers are not confused by or accidentally
delete it, as it looks useless.
Additionally do not assign to an unused variable but just cast the
result to void, to make clear it is unused on purpose.

modules/gui/macosx/VLCDocumentController.m

index a752c6b..e97991b 100644 (file)
 
 + (void)load
 {
-    id controller = [[VLCDocumentController alloc] init];
+    /* For the custom document controller to be used by the app,
+     * it needs to be created before the default NSDocumentController
+     * is created.
+     */
+    (void)[[VLCDocumentController alloc] init];
 }
 
 - (IBAction)clearRecentDocuments:(id)sender