[IDE] Tweak replace.bodies.with.fatalError.

The menu item should be capitalised, and it's nicer if the fatalError is on its
own line in the new function body.

Author: huonw

include/swift/IDE/RefactoringKinds.def

@@ -60,7 +60,7 @@ RANGE_REFACTORING(ExpandTernaryExpr, "Expand Ternary Expression", expand.ternary
 
 RANGE_REFACTORING(ConvertToTernaryExpr, "Convert To Ternary Expression", convert.ternary.expr)
 
-RANGE_REFACTORING(ReplaceBodiesWithFatalError, "Replace function bodies with fatalError", replace.bodies.with.fatalError)
+RANGE_REFACTORING(ReplaceBodiesWithFatalError, "Replace Function Bodies With 'fatalError()'", replace.bodies.with.fatalError)
 
 #undef CURSOR_REFACTORING
 #undef RANGE_REFACTORING

lib/IDE/Refactoring.cpp

@@ -1639,7 +1639,7 @@ bool RefactoringActionReplaceBodiesWithFatalError::isApplicable(
 }
 
 bool RefactoringActionReplaceBodiesWithFatalError::performChange() {
-  const StringRef replacement = "{ fatalError() }";
+  const StringRef replacement = "{\nfatalError()\n}";
   llvm::SmallPtrSet<Decl *, 16> Found;
   for (auto decl : RangeInfo.DeclaredDecls) {
     FindAllSubDecls(Found).walk(decl.VD);

test/refactoring/ReplaceBodiesWithFatalError/Outputs/basic/L12-14.swift.expected

@@ -9,7 +9,9 @@ var qux: Int = 0 {
     didSet { print("bye") }
 }
 
-func generic<T>(_: T) -> Int { fatalError() }
+func generic<T>(_: T) -> Int {
+fatalError()
+}
 
 class Foo {
     var foo: Int

test/refactoring/ReplaceBodiesWithFatalError/Outputs/basic/L16-49.swift.expected

@@ -16,25 +16,43 @@ func generic<T>(_: T) -> Int {
 class Foo {
     var foo: Int
 
-    var bar: Int { fatalError() }
+    var bar: Int {
+fatalError()
+}
 
     var baz: Int {
-        get { fatalError() }
-        set { fatalError() }
+        get {
+fatalError()
+}
+        set {
+fatalError()
+}
     }
 
     var qux: Int {
-        willSet { fatalError() }
-        didSet { fatalError() }
+        willSet {
+fatalError()
+}
+        didSet {
+fatalError()
+}
     }
 
-    init() { fatalError() }
+    init() {
+fatalError()
+}
 
-    func nonGeneric() -> Int { fatalError() }
+    func nonGeneric() -> Int {
+fatalError()
+}
 
-    func generic<T>(_: T) -> Int { fatalError() }
+    func generic<T>(_: T) -> Int {
+fatalError()
+}
 
-    deinit { fatalError() }
+    deinit {
+fatalError()
+}
 }
 
 extension Foo {

test/refactoring/ReplaceBodiesWithFatalError/Outputs/basic/L19-22.swift.expected

@@ -16,7 +16,9 @@ func generic<T>(_: T) -> Int {
 class Foo {
     var foo: Int
 
-    var bar: Int { fatalError() }
+    var bar: Int {
+fatalError()
+}
 
     var baz: Int {
         get { return 0 }

test/refactoring/ReplaceBodiesWithFatalError/Outputs/basic/L3-5.swift.expected

@@ -1,6 +1,8 @@
 var foo: Int = 0
 
-var bar: Int { fatalError() }
+var bar: Int {
+fatalError()
+}
 
 var qux: Int = 0 {
     willSet { print("hi") }

test/refactoring/ReplaceBodiesWithFatalError/Outputs/basic/L30-44.swift.expected

@@ -27,14 +27,22 @@ class Foo {
 
     var qux: Int {
         willSet { print("hi") }
-        didSet { fatalError() }
+        didSet {
+fatalError()
+}
     }
 
-    init() { fatalError() }
+    init() {
+fatalError()
+}
 
-    func nonGeneric() -> Int { fatalError() }
+    func nonGeneric() -> Int {
+fatalError()
+}
 
-    func generic<T>(_: T) -> Int { fatalError() }
+    func generic<T>(_: T) -> Int {
+fatalError()
+}
 
     deinit {
         print("bye")

test/refactoring/ReplaceBodiesWithFatalError/Outputs/basic/L51-59.swift.expected

@@ -49,9 +49,13 @@ class Foo {
 }
 
 extension Foo {
-    func nonGeneric2() -> Int { fatalError() }
+    func nonGeneric2() -> Int {
+fatalError()
+}
 
-    func generic2<T>(_: T) -> Int { fatalError() }
+    func generic2<T>(_: T) -> Int {
+fatalError()
+}
 }
 
 

test/refactoring/ReplaceBodiesWithFatalError/Outputs/basic/L7-10.swift.expected

@@ -5,8 +5,12 @@ var bar: Int {
 }
 
 var qux: Int = 0 {
-    willSet { fatalError() }
-    didSet { fatalError() }
+    willSet {
+fatalError()
+}
+    didSet {
+fatalError()
+}
 }
 
 func generic<T>(_: T) -> Int {

test/refactoring/ReplaceBodiesWithFatalError/Outputs/basic/L8.swift.expected

@@ -5,7 +5,9 @@ var bar: Int {
 }
 
 var qux: Int = 0 {
-    willSet { fatalError() }
+    willSet {
+fatalError()
+}
     didSet { print("bye") }
 }