Merge pull request #3607 from phausler/UndoManager_registerUndo_naming_update

[Foundation] Update UndoManager registerUndo to adopt new naming rules

Author: parkera

stdlib/public/SDK/Foundation/Foundation.swift

@@ -1259,10 +1259,13 @@ internal func NS_Swift_NSUndoManager_registerUndoWithTargetHandler(
   _ handler: @convention(block) (AnyObject) -> Void)
 
 extension UndoManager {
+  @available(*, unavailable, renamed: "registerUndo(withTarget:handler:)")
+  public func registerUndoWithTarget<TargetType : AnyObject>(_ target: TargetType, handler: (TargetType) -> Void) {
+    fatalError("This API has been renamed")
+  }
+
   @available(OSX 10.11, iOS 9.0, *)
-  public func registerUndoWithTarget<TargetType : AnyObject>(
-    _ target: TargetType, handler: (TargetType) -> Void
-  ) {
+  public func registerUndo<TargetType : AnyObject>(withTarget target: TargetType, handler: (TargetType) -> Void) {
     // The generic blocks use a different ABI, so we need to wrap the provided
     // handler in something ObjC compatible.
     let objcCompatibleHandler: (AnyObject) -> Void = { internalTarget in

test/Interpreter/SDK/Foundation_test.swift

@@ -230,7 +230,7 @@ if #available(OSX 10.11, iOS 9.0, *) {
       var someProperty: String = ""
     }
     let f = ObjCClass()
-    UM.registerUndoWithTarget(f) { target in
+    UM.registerUndo(withTarget: f) { target in
       target.someProperty = "expected"
     }
     UM.undo()
@@ -245,7 +245,7 @@ if #available(OSX 10.11, iOS 9.0, *) {
       var someOtherProperty: String = ""
     }
     var b = SwiftClass()
-    UM.registerUndoWithTarget(b) { target in
+    UM.registerUndo(withTarget:b) { target in
       target.someOtherProperty = "expected"
     }
     UM.undo()