Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit5f2eb3e

Browse files
fix notUsed logic
1 parent9e51546 commit5f2eb3e

File tree

1 file changed

+11
-2
lines changed

1 file changed

+11
-2
lines changed

‎vsintegration/src/FSharp.Editor/Diagnostics/UnusedOpensDiagnosticAnalyzer.fs‎

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -145,8 +145,17 @@ module private UnusedOpens =
145145
let recfilterInner acc(list:OpenStatement list)(seenOpenStatements:OpenStatement list)=
146146

147147
letnotUsed(os:OpenStatement)=
148-
not(namespacesInUse|> List.exists(fun nsu-> rangeContainsRange os.ModuleRange nsu.Location&& os.Names|> Set.contains nsu.Ident))
149-
|| seenOpenStatements|> List.contains os
148+
letnotUsedAnywhere=not(namespacesInUse|> List.exists(fun nsu-> rangeContainsRange os.ModuleRange nsu.Location&& os.Names|> Set.contains nsu.Ident))
149+
if notUsedAnywherethentrue
150+
else
151+
letalreadySeen=
152+
seenOpenStatements
153+
|> List.exists(fun seenNs->
154+
// if such open statement has already been marked as used in this or outer module, we skip it
155+
// (that is, do not mark as used so far)
156+
(seenNs.ModuleRange= os.ModuleRange|| rangeContainsRange seenNs.ModuleRange os.ModuleRange)&&
157+
not(os.Names|> Set.intersect seenNs.Names|> Set.isEmpty))
158+
alreadySeen
150159

151160
match listwith
152161
| os:: xswhen notUsed os->

0 commit comments

Comments
 (0)

[8]ページ先頭

©2009-2025 Movatter.jp