Correct changelog

ChangeLog

@@ -1,9 +1,16 @@
-2020-06-09 Gregory John Casamento <greg.casamento@gmail.com>
+2020-06-14 Gregory John Casamento <greg.casamento@gmail.com>
 
 	* Headers/Foundation/NSKeyedArchiver.h
 	* Source/NSKeyedArchiver.m
 	* Source/NSKeyedUnarchiver.m: Add setter for requiresSecureCoding.
 
+2020-06-11  Frederik Seiffert <frederik@algoriddim.com>
+
+	* Source/NSValueTransformer.m:
+	Fix recursive creation of value transformers.
+	* Tests/base/NSValueTransformer/basic.m:
+	Add tests for NSValueTransformer.
+
 2020-06-06  Frederik Seiffert <frederik@algoriddim.com>
 
 	* Source/NSBundle.m: Extend NSBundle resources support to handle

Source/NSData.m

@@ -1787,9 +1787,11 @@ failure:
 	{
 	  /*
 	   * We have created a new file - so we attempt to make it's
-	   * attributes match that of the original.
+	   * attributes match that of the original (except for those
+	   * we can't reasonably set).
 	   */
 	  [att removeObjectForKey: NSFileSize];
+	  [att removeObjectForKey: NSFileCreationDate];
 	  [att removeObjectForKey: NSFileModificationDate];
 	  [att removeObjectForKey: NSFileReferenceCount];
 	  [att removeObjectForKey: NSFileSystemNumber];

Source/NSValueTransformer.m

@@ -52,7 +52,7 @@
 // non-abstract methods
 
 static NSMutableDictionary *registry = nil;
-static NSLock *lock = nil;
+static NSRecursiveLock *lock = nil;
 
 + (void) initialize
 {
@@ -60,7 +60,7 @@ static NSLock *lock = nil;
     {
       NSValueTransformer	*t;
 
-      lock = [NSLock new];
+      lock = [NSRecursiveLock new];
       [[NSObject leakAt: &lock] release];
       registry = [[NSMutableDictionary alloc] init];
       [[NSObject leakAt: &registry] release];

Tests/base/GSTLS/basic.m

@@ -27,9 +27,12 @@ START_SET("TLS support")
   expiresAt = [c expiresAt: 0];
   dateFormatter = [[NSDateFormatter alloc] init];
   [dateFormatter setDateFormat: @"yyyy-MM-dd HH:mm:ss zzz"];
+  /* Test guaranteed to fail on 32-bit architectures.  */
+#if __LP64__
   PASS_EQUAL(expiresAt,
     [dateFormatter dateFromString: @"2118-12-14 15:35:11 +0000"],
     "Expiration date can be retrieved");
+#endif
   [dateFormatter release];
   PASS_EQUAL([c expiresAt], expiresAt,
     "Expiration for entire list is that of the single item")

Tests/base/NSValueTransformer/basic.m

@@ -0,0 +1,99 @@
+#import "ObjectTesting.h"
+#import <Foundation/NSAutoreleasePool.h>
+#import <Foundation/NSValueTransformer.h>
+#import <Foundation/NSValue.h>
+
+@interface YesTransformer : NSValueTransformer
+@end
+
+@implementation YesTransformer
+
++ (BOOL) allowsReverseTransformation
+{
+  return NO;
+}
+
++ (Class) transformedValueClass
+{
+  return [NSNumber class];
+}
+
+- (id) transformedValue: (id)value
+{
+  return [NSNumber numberWithBool: YES];
+}
+
+@end
+
+@interface NestedTransformer : NSValueTransformer
+{
+  NSValueTransformer *_backingTransformer;
+}
+@end
+
+@implementation NestedTransformer
+
+- (id) init
+{
+  if (self = [super init]) {
+    _backingTransformer = RETAIN([NSValueTransformer valueTransformerForName:
+      NSStringFromClass([YesTransformer class])]);
+  }
+
+  return self;
+}
+
+- (void) dealloc
+{
+  RELEASE(_backingTransformer);
+  DEALLOC
+}
+
++ (BOOL) allowsReverseTransformation
+{
+  return NO;
+}
+
++ (Class) transformedValueClass
+{
+  return [YesTransformer transformedValueClass];
+}
+
+- (id) transformedValue: (id)value
+{
+  return [_backingTransformer transformedValue: value];
+}
+
+@end
+
+int main()
+{
+  NSAutoreleasePool   *arp = [NSAutoreleasePool new];
+  NSValueTransformer  *transformer;
+  
+  transformer = [NSValueTransformer valueTransformerForName:NSNegateBooleanTransformerName];
+  PASS([[transformer transformedValue:[NSNumber numberWithBool:NO]] boolValue] == YES
+    && [[transformer transformedValue:[NSNumber numberWithBool:YES]] boolValue] == NO,
+    "NSNegateBooleanTransformer transforms correctly");
+  PASS([[transformer reverseTransformedValue:[NSNumber numberWithBool:NO]] boolValue] == YES
+    && [[transformer reverseTransformedValue:[NSNumber numberWithBool:YES]] boolValue] == NO,
+    "NSNegateBooleanTransformer reverse transforms correctly");
+  
+  transformer = [NSValueTransformer valueTransformerForName:NSIsNilTransformerName];
+  PASS([[transformer transformedValue:nil] boolValue] == YES
+    && [[transformer transformedValue:@""] boolValue] == NO,
+    "NSIsNilTransformer transforms correctly");
+  
+  transformer = [NSValueTransformer valueTransformerForName:NSIsNotNilTransformerName];
+  PASS([[transformer transformedValue:@""] boolValue] == YES
+    && [[transformer transformedValue:nil] boolValue] == NO,
+    "NSIsNotNilTransformer transforms correctly");
+  
+  transformer = [NSValueTransformer valueTransformerForName:NSStringFromClass([NestedTransformer class])];
+  PASS([[transformer transformedValue:nil] boolValue] == YES
+    && [[transformer transformedValue:@""] boolValue] == YES,
+    "Custom transformer transforms correctly");
+
+  [arp release]; arp = nil;
+  return 0;
+}