lean2/library/theories/number_theory
Leonardo de Moura 42fbc63bb6 fix(library/tc_multigraph): avoid name collisions
@avigad, @fpvandoorn, @rlewis1988, @dselsam

I changed how transitive instances are named.
The motivation is to avoid a naming collision problem found by Daniel.
Before this commit, we were getting an error on the following file
tests/lean/run/collision_bug.lean.

Now, transitive instances contain the prefix "_trans_".
It makes it clear this is an internal definition and it should not be used
by users.

This change also demonstrates (again) how the `rewrite` tactic is
fragile. The problem is that the matching procedure used by it has
very little support for solving matching constraints that involving type
class instances. Eventually, we will need to reimplement `rewrite`
using the new unification procedure used in blast.

In the meantime, the workaround is to use `krewrite` (as usual).
2016-02-04 13:15:42 -08:00
..
bezout.lean feat(library/tactic): make let tactic transparent, introduce new opaque note tactic 2015-12-14 10:14:02 -08:00
irrational_roots.lean refactor(library): cleanup proofs 2016-02-03 19:52:23 -08:00
number_theory.md feat/refactor(library/theories/number_theory/irrational_roots,library/*): show nth roots irrational, and add lots of missing theorems 2015-08-16 23:23:23 -04:00
prime_factorization.lean fix(library/tc_multigraph): avoid name collisions 2016-02-04 13:15:42 -08:00
primes.lean refactor(library/init/subtype.lean): put subtype notation in the namespace 2015-12-22 16:39:13 -05:00