Skip to content

Commit 6daef7c

Browse files
clean up uses of open
1 parent 20df51e commit 6daef7c

File tree

4 files changed

+21
-10
lines changed

4 files changed

+21
-10
lines changed

Validator/Env/TestEnvTreeParserStateWithMem.lean

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,6 @@ structure TreeParserAndMemTest where
1313
enter: MemEnter.EnterMap
1414
leave: MemLeave.LeaveMap
1515

16-
open EnvTreeParserStateWithMem
17-
1816
abbrev TreeParserStateWithMemTest α := EStateM String TreeParserAndMemTest α
1917

2018
def TreeParserStateWithMemTest.mk (p: TreeParser.TreeParser): TreeParserAndMemTest :=

Validator/Expr/Language.lean

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,19 @@ import Validator.Std.Linter.DetectClassical
44

55
namespace Language
66

7-
open List
7+
open List (
8+
append_assoc
9+
append_eq_nil_iff
10+
append_nil
11+
cons
12+
cons_append
13+
cons.injEq
14+
foldl_nil
15+
nil
16+
nil_append
17+
nil_eq
18+
singleton_append
19+
)
820

921
-- Definitions
1022

Validator/Parser/TreeParser.lean

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -126,8 +126,7 @@ def TreeParser.run (x: StateT TreeParser (Except String) α) (t: ParseTree): Exc
126126
instance : Debug (StateT TreeParser (Except String)) where
127127
debug (_: String) := return ()
128128

129-
open Parser (walk)
130-
open Parser (Action)
129+
open Parser (walk Action)
131130
open ParseTree (node)
132131

133132
#guard TreeParser.run

Validator/Std/Linter/DetectClassical.lean

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,9 @@ The "detectClassical" linter emits a warning on declarations that depend on the
1212
axiom.
1313
-/
1414

15-
open Lean Elab Command
15+
open Lean (Linter collectAxioms logInfoAt addLinter withSetOptionIn)
16+
open Lean.Linter (getLinterOptions getLinterValue logLint)
17+
open Mathlib.Linter (getNamesFrom)
1618

1719
namespace RegexDeriv.Std.Linter
1820

@@ -38,14 +40,14 @@ namespace DetectClassical
3840

3941
@[inherit_doc RegexDeriv.Std.Linter.linter.detectClassical]
4042
def detectClassicalLinter : Linter where run := withSetOptionIn fun stx ↦ do
41-
unless Linter.getLinterValue linter.detectClassical (← Linter.getLinterOptions) do
43+
unless getLinterValue linter.detectClassical (← getLinterOptions) do
4244
return
4345
if (← get).messages.hasErrors then
4446
return
4547
let d := (stx.getPos?.getD default)
46-
let nmsd := (← Mathlib.Linter.getNamesFrom d)
48+
let nmsd := (← getNamesFrom d)
4749
let nms := nmsd.filter (! ·.getId.isInternal)
48-
let verbose? := Linter.getLinterValue linter.verbose.detectClassical (← Linter.getLinterOptions)
50+
let verbose? := getLinterValue linter.verbose.detectClassical (← getLinterOptions)
4951
for constStx in nms do
5052
let constName := constStx.getId
5153
let axioms ← collectAxioms constName
@@ -58,7 +60,7 @@ def detectClassicalLinter : Linter where run := withSetOptionIn fun stx ↦ do
5860
logInfoAt constStx
5961
m!"'{constName}' is non-classical and depends on axioms: {axioms.toList}"
6062
else
61-
Linter.logLint linter.detectClassical constStx
63+
logLint linter.detectClassical constStx
6264
m!"'{constName}' depends on 'Classical.choice' on axioms: {axioms.toList}"
6365

6466
initialize addLinter detectClassicalLinter

0 commit comments

Comments
 (0)