Skip to content

Commit 161dc2e

Browse files
committed
Sema: Rename getUnmetDeclAvailabilityRequirement().
Name it `getUnsatisfiedAvailabilityConstraint()` to match `AvailabilityConstraint`.
1 parent 4fa02cd commit 161dc2e

File tree

6 files changed

+29
-29
lines changed

6 files changed

+29
-29
lines changed

lib/Sema/BuilderTransform.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1322,7 +1322,7 @@ ResultBuilderOpSupport TypeChecker::checkBuilderOpSupport(
13221322

13231323
auto isUnavailable = [&](Decl *D) -> bool {
13241324
auto loc = extractNearestSourceLoc(dc);
1325-
return getUnmetDeclAvailabilityRequirement(D, dc, loc).has_value();
1325+
return getUnsatisfiedAvailabilityConstraint(D, dc, loc).has_value();
13261326
};
13271327

13281328
bool foundMatch = false;

lib/Sema/CSSyntacticElement.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2475,9 +2475,9 @@ class ResultBuilderRewriter : public SyntacticElementSolutionApplication {
24752475
if (!nominal)
24762476
return false;
24772477

2478-
auto unmetRequirement = getUnmetDeclAvailabilityRequirement(
2478+
auto constraint = getUnsatisfiedAvailabilityConstraint(
24792479
nominal, context.getAsDeclContext(), loc);
2480-
if (unmetRequirement && unmetRequirement->isConditionallySatisfiable()) {
2480+
if (constraint && constraint->isConditionallySatisfiable()) {
24812481
auto &ctx = getASTContext();
24822482
ctx.Diags.diagnose(loc,
24832483
diag::result_builder_missing_limited_availability,

lib/Sema/ConstraintSystem.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4682,7 +4682,7 @@ bool ConstraintSystem::isDeclUnavailable(const Decl *D,
46824682
loc = getLoc(anchor);
46834683
}
46844684

4685-
return getUnmetDeclAvailabilityRequirement(D, DC, loc).has_value();
4685+
return getUnsatisfiedAvailabilityConstraint(D, DC, loc).has_value();
46864686
}
46874687

46884688
bool ConstraintSystem::isConformanceUnavailable(ProtocolConformanceRef conformance,
@@ -4881,9 +4881,9 @@ bool ConstraintSystem::isReadOnlyKeyPathComponent(
48814881
if (auto setter = storage->getOpaqueAccessor(AccessorKind::Set)) {
48824882
// FIXME: Fully unavailable setters should cause the key path to be
48834883
// readonly too.
4884-
auto unmetRequirement =
4885-
getUnmetDeclAvailabilityRequirement(setter, DC, referenceLoc);
4886-
if (unmetRequirement && unmetRequirement->isConditionallySatisfiable())
4884+
auto constraint =
4885+
getUnsatisfiedAvailabilityConstraint(setter, DC, referenceLoc);
4886+
if (constraint && constraint->isConditionallySatisfiable())
48874887
return true;
48884888
}
48894889

lib/Sema/TypeCheckAvailability.cpp

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -3033,7 +3033,7 @@ bool diagnoseExplicitUnavailability(
30333033
}
30343034

30353035
std::optional<AvailabilityConstraint>
3036-
swift::getUnmetDeclAvailabilityRequirement(
3036+
swift::getUnsatisfiedAvailabilityConstraint(
30373037
const Decl *decl, const DeclContext *declContext,
30383038
AvailabilityContext availabilityContext) {
30393039
auto &ctx = declContext->getASTContext();
@@ -3075,10 +3075,10 @@ swift::getUnmetDeclAvailabilityRequirement(
30753075
}
30763076

30773077
std::optional<AvailabilityConstraint>
3078-
swift::getUnmetDeclAvailabilityRequirement(const Decl *decl,
3079-
const DeclContext *referenceDC,
3080-
SourceLoc referenceLoc) {
3081-
return getUnmetDeclAvailabilityRequirement(
3078+
swift::getUnsatisfiedAvailabilityConstraint(const Decl *decl,
3079+
const DeclContext *referenceDC,
3080+
SourceLoc referenceLoc) {
3081+
return getUnsatisfiedAvailabilityConstraint(
30823082
decl, referenceDC,
30833083
TypeChecker::availabilityAtLocation(referenceLoc, referenceDC));
30843084
}
@@ -4094,10 +4094,10 @@ bool swift::diagnoseDeclAvailability(const ValueDecl *D, SourceRange R,
40944094
auto *DC = Where.getDeclContext();
40954095
auto &ctx = DC->getASTContext();
40964096

4097-
auto unmetRequirement =
4098-
getUnmetDeclAvailabilityRequirement(D, DC, Where.getAvailability());
4097+
auto constraint =
4098+
getUnsatisfiedAvailabilityConstraint(D, DC, Where.getAvailability());
40994099

4100-
if (unmetRequirement && !unmetRequirement->isConditionallySatisfiable()) {
4100+
if (constraint && !constraint->isConditionallySatisfiable()) {
41014101
// FIXME: diagnoseExplicitUnavailability should take an unmet requirement
41024102
if (diagnoseExplicitUnavailability(D, R, Where, call, Flags))
41034103
return true;
@@ -4121,10 +4121,10 @@ bool swift::diagnoseDeclAvailability(const ValueDecl *D, SourceRange R,
41214121
&& isa<ProtocolDecl>(D))
41224122
return false;
41234123

4124-
if (!unmetRequirement)
4124+
if (!constraint)
41254125
return false;
41264126

4127-
auto requiredRange = unmetRequirement->getRequiredNewerAvailabilityRange(ctx);
4127+
auto requiredRange = constraint->getRequiredNewerAvailabilityRange(ctx);
41284128

41294129
// Diagnose (and possibly signal) for potential unavailability
41304130
if (!requiredRange)
@@ -4601,9 +4601,9 @@ swift::diagnoseConformanceAvailability(SourceLoc loc,
46014601
return true;
46024602
}
46034603

4604-
auto unmetRequirement = getUnmetDeclAvailabilityRequirement(
4604+
auto constraint = getUnsatisfiedAvailabilityConstraint(
46054605
ext, where.getDeclContext(), where.getAvailability());
4606-
if (unmetRequirement) {
4606+
if (constraint) {
46074607
// FIXME: diagnoseExplicitUnavailability() should take unmet requirement
46084608
if (diagnoseExplicitUnavailability(
46094609
loc, rootConf, ext, where,
@@ -4614,7 +4614,7 @@ swift::diagnoseConformanceAvailability(SourceLoc loc,
46144614

46154615
// Diagnose (and possibly signal) for potential unavailability
46164616
if (auto requiredRange =
4617-
unmetRequirement->getRequiredNewerAvailabilityRange(ctx)) {
4617+
constraint->getRequiredNewerAvailabilityRange(ctx)) {
46184618
if (diagnosePotentialUnavailability(rootConf, ext, loc, DC,
46194619
*requiredRange)) {
46204620
maybeEmitAssociatedTypeNote();

lib/Sema/TypeCheckAvailability.h

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -246,18 +246,18 @@ void diagnoseOverrideOfUnavailableDecl(ValueDecl *override,
246246

247247
/// Checks whether a declaration should be considered unavailable when referred
248248
/// to in the given declaration context and availability context and, if so,
249-
/// returns a result that describes the unmet availability requirements.
249+
/// returns a result that describes the unsatisfied constraint.
250250
/// Returns `std::nullopt` if the declaration is available.
251251
std::optional<AvailabilityConstraint>
252-
getUnmetDeclAvailabilityRequirement(const Decl *decl,
253-
const DeclContext *declContext,
254-
AvailabilityContext availabilityContext);
252+
getUnsatisfiedAvailabilityConstraint(const Decl *decl,
253+
const DeclContext *declContext,
254+
AvailabilityContext availabilityContext);
255255

256256
/// Checks whether a declaration should be considered unavailable when referred
257257
/// to at the given source location in the given decl context and, if so,
258-
/// returns a result that describes the unmet availability requirements.
258+
/// returns a result that describes the unsatisfied constraint.
259259
/// Returns `std::nullopt` if the declaration is available.
260-
std::optional<AvailabilityConstraint> getUnmetDeclAvailabilityRequirement(
260+
std::optional<AvailabilityConstraint> getUnsatisfiedAvailabilityConstraint(
261261
const Decl *decl, const DeclContext *referenceDC, SourceLoc referenceLoc);
262262

263263
/// Diagnose uses of the runtime support of the given type, such as

lib/Sema/TypeCheckPattern.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,10 +38,10 @@ using namespace swift;
3838
/// be kept in sync with importEnumCaseAlias in the ClangImporter library.
3939
static EnumElementDecl *extractEnumElement(DeclContext *DC, SourceLoc UseLoc,
4040
const VarDecl *constant) {
41-
if (auto requirement =
42-
getUnmetDeclAvailabilityRequirement(constant, DC, UseLoc)) {
41+
if (auto constraint =
42+
getUnsatisfiedAvailabilityConstraint(constant, DC, UseLoc)) {
4343
// Only diagnose explicit unavailability.
44-
if (!requirement->isConditionallySatisfiable())
44+
if (!constraint->isConditionallySatisfiable())
4545
diagnoseDeclAvailability(constant, UseLoc, nullptr,
4646
ExportContext::forFunctionBody(DC, UseLoc));
4747
}

0 commit comments

Comments
 (0)