Skip to content

Commit 91da39a

Browse files
committed
Remove warning
1 parent b667378 commit 91da39a

File tree

3 files changed

+0
-21
lines changed

3 files changed

+0
-21
lines changed

lib/src/model/model_element.dart

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -539,20 +539,6 @@ abstract class ModelElement extends Canonicalization
539539
return topLevelElement == lookup;
540540
}).toList(growable: true);
541541

542-
// Avoid claiming canonicalization for elements outside of this element's
543-
// defining package.
544-
// TODO(srawlins): Make the else block unconditional, so that we always
545-
// exclude cross-package exports.
546-
if (candidateLibraries.isNotEmpty &&
547-
candidateLibraries.any((l) => l.package != definingLibrary.package)) {
548-
warn(PackageWarning.reexportedPrivateApiAcrossPackages,
549-
message: definingLibrary.package.fullyQualifiedName,
550-
referredFrom: candidateLibraries);
551-
} else {
552-
candidateLibraries
553-
.removeWhere((l) => l.package != definingLibrary.package);
554-
}
555-
556542
if (candidateLibraries.isEmpty) {
557543
return null;
558544
}

lib/src/model/package_graph.dart

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -353,7 +353,6 @@ class PackageGraph with CommentReferable, Nameable, ModelBuilder {
353353
PackageWarning.ambiguousDocReference ||
354354
PackageWarning.ignoredCanonicalFor ||
355355
PackageWarning.packageOrderGivesMissingPackageName ||
356-
PackageWarning.reexportedPrivateApiAcrossPackages ||
357356
PackageWarning.notImplemented ||
358357
PackageWarning.unresolvedDocReference ||
359358
PackageWarning.unknownDirective ||

lib/src/warnings.dart

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -206,10 +206,6 @@ const Map<PackageWarning, PackageWarningDefinition> packageWarningDefinitions =
206206
PackageWarning.packageOrderGivesMissingPackageName,
207207
'category-order-gives-missing-package-name',
208208
'The category-order flag on the command line was given the name of a nonexistent package'),
209-
PackageWarning.reexportedPrivateApiAcrossPackages: PackageWarningDefinition(
210-
PackageWarning.reexportedPrivateApiAcrossPackages,
211-
'reexported-private-api-across-packages',
212-
'One or more libraries reexports private API members from outside its own package'),
213209
PackageWarning.unresolvedDocReference: PackageWarningDefinition(
214210
PackageWarning.unresolvedDocReference,
215211
'unresolved-doc-reference',
@@ -339,8 +335,6 @@ enum PackageWarning {
339335
noLibraryLevelDocs('{0} has no library level documentation comments'),
340336
packageOrderGivesMissingPackageName(
341337
"--package-order gives invalid package name: '{0}'"),
342-
reexportedPrivateApiAcrossPackages(
343-
'private API of {0} is reexported by libraries in other packages: '),
344338
unresolvedDocReference('unresolved doc reference [{0}]',
345339
referredFromPrefix: 'in documentation inherited from'),
346340
unknownDirective('undefined directive: {0}'),

0 commit comments

Comments
 (0)