Skip to content

Commit b500347

Browse files
committed
chore: fix non-terminal simps followed by fun_prop (leanprover-community#30048)
In most cases, these can be `dsimp; fun_prop` instead (which shows more clearly what is going on). In two cases where `simp` was actually necessary, squeeze it. Extracted from leanprover-community#28962, but hopefully also makes sense on its own. Co-authored-by: Michael Rothgang <rothgang@math.uni-bonn.de>
1 parent b8f1ced commit b500347

5 files changed

Lines changed: 12 additions & 11 deletions

File tree

Mathlib/Analysis/CStarAlgebra/ContinuousFunctionalCalculus/Unique.lean

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ noncomputable def realContinuousMapOfNNReal (φ : C(X, ℝ≥0) →⋆ₐ[ℝ≥
142142
@[fun_prop]
143143
lemma continuous_realContinuousMapOfNNReal (φ : C(X, ℝ≥0) →⋆ₐ[ℝ≥0] A)
144144
(hφ : Continuous φ) : Continuous φ.realContinuousMapOfNNReal := by
145-
simp [realContinuousMapOfNNReal]
145+
dsimp [realContinuousMapOfNNReal]
146146
fun_prop
147147

148148
end IsTopologicalRing
@@ -328,7 +328,7 @@ noncomputable def realContinuousMapZeroOfNNReal (φ : C(X, ℝ≥0)₀ →⋆ₙ
328328
@[fun_prop]
329329
lemma continuous_realContinuousMapZeroOfNNReal (φ : C(X, ℝ≥0)₀ →⋆ₙₐ[ℝ≥0] A)
330330
(hφ : Continuous φ) : Continuous φ.realContinuousMapZeroOfNNReal := by
331-
simp [realContinuousMapZeroOfNNReal]
331+
dsimp [realContinuousMapZeroOfNNReal]
332332
fun_prop
333333

334334
end IsTopologicalRing
@@ -444,7 +444,7 @@ lemma NonUnitalStarAlgHomClass.map_cfcₙ (φ : F) (f : R → R) (a : A)
444444
· simp [cfcₙHom_id]
445445
· congr
446446
all_goals
447-
simp [ContinuousMapZero.nonUnitalStarAlgHom_precomp]
447+
dsimp [ContinuousMapZero.nonUnitalStarAlgHom_precomp]
448448
fun_prop
449449

450450
/-- Non-unital star algebra homomorphisms commute with the non-unital continuous functional
@@ -492,7 +492,7 @@ lemma StarAlgHomClass.map_cfc (φ : F) (f : R → R) (a : A)
492492
· simp [cfcHom_id]
493493
· congr
494494
all_goals
495-
simp [ContinuousMap.compStarAlgHom']
495+
dsimp [ContinuousMap.compStarAlgHom']
496496
fun_prop
497497

498498
/-- Star algebra homomorphisms commute with the continuous functional calculus.

Mathlib/MeasureTheory/Function/StronglyMeasurable/Basic.lean

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -625,7 +625,7 @@ variable {n : MeasurableSpace β} in
625625
lemma Finset.stronglyMeasurable_prod_apply {ι : Type*} {f : ι → α → β → M} {g : α → β}
626626
{s : Finset ι} (hf : ∀ i ∈ s, StronglyMeasurable ↿(f i)) (hg : Measurable g) :
627627
StronglyMeasurable fun a ↦ (∏ i ∈ s, f i a) (g a) := by
628-
simp; fun_prop (discharger := assumption)
628+
simp only [Finset.prod_apply]; fun_prop (discharger := assumption)
629629

630630
end CommMonoid
631631

Mathlib/MeasureTheory/Group/Arithmetic.lean

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@ theorem Measurable.mul [MeasurableMul₂ M] (hf : Measurable f) (hg : Measurable
120120
/-- Compositional version of `Measurable.add` for use by `fun_prop`. -/]
121121
lemma Measurable.mul' [MeasurableMul₂ M] {f g : α → β → M} {h : α → β} (hf : Measurable ↿f)
122122
(hg : Measurable ↿g) (hh : Measurable h) : Measurable fun a ↦ (f a * g a) (h a) := by
123-
simp; fun_prop
123+
dsimp; fun_prop
124124

125125
@[to_additive (attr := fun_prop, aesop safe 20 apply (rule_sets := [Measurable]))]
126126
theorem AEMeasurable.mul' [MeasurableMul₂ M] (hf : AEMeasurable f μ) (hg : AEMeasurable g μ) :
@@ -272,7 +272,7 @@ theorem Measurable.div [MeasurableDiv₂ G] (hf : Measurable f) (hg : Measurable
272272
@[to_additive (attr := fun_prop, aesop safe 20 apply (rule_sets := [Measurable]))]
273273
lemma Measurable.div' [MeasurableDiv₂ G] {f g : α → β → G} {h : α → β} (hf : Measurable ↿f)
274274
(hg : Measurable ↿g) (hh : Measurable h) : Measurable fun a ↦ (f a / g a) (h a) := by
275-
simp; fun_prop
275+
dsimp; fun_prop
276276

277277
@[to_additive (attr := fun_prop, aesop safe 20 apply (rule_sets := [Measurable]))]
278278
theorem AEMeasurable.div' [MeasurableDiv₂ G] (hf : AEMeasurable f μ) (hg : AEMeasurable g μ) :
@@ -600,7 +600,7 @@ theorem Measurable.smul [MeasurableSMul₂ M X] (hf : Measurable f) (hg : Measur
600600
/-- Compositional version of `Measurable.vadd` for use by `fun_prop`. -/]
601601
lemma Measurable.smul' [MeasurableSMul₂ M X] {f : α → β → M} {g : α → β → X} {h : α → β}
602602
(hf : Measurable ↿f) (hg : Measurable ↿g) (hh : Measurable h) :
603-
Measurable fun a ↦ (f a • g a) (h a) := by simp; fun_prop
603+
Measurable fun a ↦ (f a • g a) (h a) := by dsimp; fun_prop
604604

605605
@[to_additive (attr := fun_prop, aesop safe 20 apply (rule_sets := [Measurable]))]
606606
theorem AEMeasurable.smul [MeasurableSMul₂ M X] {μ : Measure α} (hf : AEMeasurable f μ)
@@ -918,7 +918,8 @@ theorem Finset.measurable_prod (s : Finset ι) (hf : ∀ i ∈ s, Measurable (f
918918
/-- Compositional version of `Finset.measurable_sum` for use by `fun_prop`. -/]
919919
lemma Finset.measurable_prod_apply {f : ι → α → β → M} {g : α → β} {s : Finset ι}
920920
(hf : ∀ i ∈ s, Measurable ↿(f i)) (hg : Measurable g) :
921-
Measurable fun a ↦ (∏ i ∈ s, f i a) (g a) := by simp; fun_prop (discharger := assumption)
921+
Measurable fun a ↦ (∏ i ∈ s, f i a) (g a) := by
922+
simp only [prod_apply]; fun_prop (discharger := assumption)
922923

923924
@[deprecated (since := "2025-05-30")]
924925
alias Finset.measurable_sum' := Finset.measurable_sum_apply

Mathlib/MeasureTheory/Measure/WithDensity.lean

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -740,7 +740,7 @@ theorem mconv_withDensity_eq_mlconvolution₀ {f g : G → ℝ≥0∞}
740740
lintegral_lintegral_swap]
741741
· simp only [Pi.mul_apply, mul_inv_cancel_left, mlconvolution_def]
742742
conv in (∫⁻ _ , _ ∂μ) * φ _ => rw [(lintegral_mul_const'' _ (by fun_prop)).symm]
743-
all_goals first | fun_prop | simp; fun_prop
743+
all_goals first | fun_prop | dsimp; fun_prop
744744

745745
@[to_additive]
746746
theorem mconv_withDensity_eq_mlconvolution {f g : G → ℝ≥0∞}

Mathlib/Topology/Homeomorph/Lemmas.lean

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,7 @@ def sumPiEquivProdPi (S T : Type*) (A : S ⊕ T → Type*)
204204
(Π (st : S ⊕ T), A st) ≃ₜ (Π (s : S), A (.inl s)) × (Π (t : T), A (.inr t)) where
205205
__ := Equiv.sumPiEquivProdPi _
206206
continuous_toFun := .prodMk (by fun_prop) (by fun_prop)
207-
continuous_invFun := continuous_pi <| by rintro (s | t) <;> simp <;> fun_prop
207+
continuous_invFun := continuous_pi <| by rintro (s | t) <;> dsimp <;> fun_prop
208208

209209
/-- The product `Π t : α, f t` of a family of topological spaces is homeomorphic to the
210210
space `f ⬝` when `α` only contains `⬝`.

0 commit comments

Comments
 (0)