MacOSX/Framework/VLCTime: Code cleanup and use Objective-C 2.0 @property. (Patch...
authorPierre d'Herbemont <pdherbemont@videolan.org>
Wed, 2 Jan 2008 02:51:42 +0000 (02:51 +0000)
committerPierre d'Herbemont <pdherbemont@videolan.org>
Wed, 2 Jan 2008 02:51:42 +0000 (02:51 +0000)
extras/MacOSX/Framework/Headers/Public/VLCTime.h
extras/MacOSX/Framework/Sources/VLCTime.m

index c885e27..6153741 100644 (file)
 
 #import <Cocoa/Cocoa.h>
 
+/**
+ * Provides an object to define VLCMedia's ti   me.
+ */
 @interface VLCTime : NSObject
 {
-    NSNumber *value;
+    NSNumber * value;       //< Holds, in seconds, the VLCTime value
 }
+
+/* Factories */
 + (VLCTime *)nullTime;
 + (VLCTime *)timeWithNumber:(NSNumber *)aNumber;
 //+ (VLCTime *)timeWithString:(NSString *)aString;
 
+/* Initializers */
 - (id)initWithNumber:(NSNumber *)aNumber;
 //- (id)initWithString:(NSString *)aString;
 
-- (NSNumber *)numberValue;
-- (NSString *)stringValue;
+/* Properties */
+@property (readonly) NSNumber * numberValue;
+@property (readonly) NSString * stringValue;
 
+/* Comparitors */
 - (NSComparisonResult)compare:(VLCTime *)aTime;
 @end
index 5e897cf..68f3197 100644 (file)
 
 #import <VLCTime.h>
 
-static VLCTime *nullTime = nil;
-
 @implementation VLCTime
 + (VLCTime *)nullTime
 {
+    static VLCTime * nullTime = nil;
     if (!nullTime)
         nullTime = [[VLCTime timeWithNumber:nil] retain];
     return nullTime;
@@ -76,7 +75,7 @@ static VLCTime *nullTime = nil;
 
 - (NSNumber *)numberValue
 {
-    return value ? [value copy] : nil;
+    return value ? [[value copy] autorelease] : nil;
 }
 
 - (NSString *)stringValue
@@ -103,11 +102,11 @@ static VLCTime *nullTime = nil;
     else if (!aTime)
         return NSOrderedDescending;
     else
-        return [value compare:[aTime numberValue]];
+        return [value compare:aTime.numberValue];
 }
 
 - (NSString *)description
 {
-    return [self stringValue];
+    return self.stringValue;
 }
 @end