fixup! [Sema] Add fix-it for property in protocol

Author: mzp

lib/Sema/CodeSynthesis.cpp

@@ -2023,21 +2023,15 @@ void swift::maybeAddAccessorsToStorage(TypeChecker &TC,
   } else if (isa<ProtocolDecl>(dc)) {
     if (storage->hasStorage()) {
       auto var = cast<VarDecl>(storage);
-      auto braces = var->getBracesRange();
 
       if (var->isLet()) {
-        auto diag = TC.diagnose(var->getLoc(),
-                                diag::protocol_property_must_be_computed_var);
-        if (braces.isValid())
-          diag
-            .fixItReplace(var->getParentPatternBinding()->getLoc(), "var")
-            .fixItReplace(braces, " { get }");
-        else
-          diag
-            .fixItReplace(var->getParentPatternBinding()->getLoc(), "var")
-            .fixItInsertAfter(var->getTypeLoc().getLoc(), " { get }");
+        TC.diagnose(var->getLoc(),
+                   diag::protocol_property_must_be_computed_var)
+          .fixItReplace(var->getParentPatternBinding()->getLoc(), "var")
+          .fixItInsertAfter(var->getTypeLoc().getLoc(), " { get }");
       } else {
         auto diag = TC.diagnose(var->getLoc(), diag::protocol_property_must_be_computed);
+        auto braces = var->getBracesRange();
 
         if (braces.isValid())
           diag.fixItReplace(braces, "{ get <#set#> }");