Merge pull request #17083 from naru-jpn/fixit-dosent-confirm-to-public-protocol

[SR-7629] Fix wrong fixit when the type doesn't conform to a public protocol

Author: jrose-apple

lib/Sema/TypeCheckProtocol.cpp

@@ -2778,6 +2778,14 @@ ConformanceChecker::resolveWitnessViaLookup(ValueDecl *requirement) {
                        requiredAccess,
                        protoAccessScope.accessLevelForDiagnostics(),
                        proto->getName());
+        if (auto *decl = dyn_cast<AbstractFunctionDecl>(witness)) {
+          auto isMemberwiseInitializer =
+              decl->getBodyKind() ==
+              AbstractFunctionDecl::BodyKind::MemberwiseInitializer;
+          if (isMemberwiseInitializer) {
+            return;
+          }
+        }
         auto fixItDiag = diags.diagnose(witness, diag::witness_fix_access,
                                         witness->getDescriptiveKind(),
                                         requiredAccess);

test/attr/accessibility.swift

@@ -252,3 +252,20 @@ public struct AdoptViaCombinedProtocol : ProtoWithReqs, ReqProvider2 {
   // expected-error@-1 {{method 'foo()' must be declared public because it matches a requirement in public protocol 'ProtoWithReqs'}} {{none}}
   public typealias Assoc = Int
 }
+
+public protocol PublicInitProto {
+  var value: Int { get }
+  init(value: Int)
+}
+public struct NonPublicInitStruct: PublicInitProto {
+  public var value: Int
+  init(value: Int) {
+  // expected-error@-1 {{initializer 'init(value:)' must be declared public because it matches a requirement in public protocol 'PublicInitProto'}}
+  // expected-note@-2 {{mark the initializer as 'public' to satisfy the requirement}}
+    self.value = value
+  }
+}
+public struct NonPublicMemberwiseInitStruct: PublicInitProto {
+// expected-error@-1 {{initializer 'init(value:)' must be declared public because it matches a requirement in public protocol 'PublicInitProto'}}
+  public var value: Int
+}