Commit 2022-02-02 13:59 d0027691
View on Github →refactor(ring_theory): clean up algebraic_iff_integral (#11773)
The definitions is_algebraic_iff_integral, is_algebraic_iff_integral' and algebra.is_algebraic_of_finite have always been annoying me, so I decided to fix that:
- The name
is_algebraic_iff_integral'doesn't explain how it differs fromis_algebraic_iff_integral(namely that the whole algebra is algebraic, rather than one element), so I renamed it toalgebra.is_algebraic_iff_integral. - The two
is_algebraic_iff_integrallemmas have an unnecessarily explicit parameterK, so I made that implicit is_algebraic_of_finitehas no explicit parameters (so we always have to use type ascriptions), so I made them explicit- Half of the usages of
is_algebraic_of_finiteare of the formis_algebraic_iff_integral.mp is_algebraic_of_finite, even thoughis_algebraic_of_finiteis proved asis_algebraic_iff_integral.mpr (some_proof_that_it_is_integral), so I split it up into a part showing it is integral, that we can use directly. As a result, I was able to golf a few proofs.