# HG changeset patch
# User ballarin
# Date 1274690912 -7200
# Node ID 3877a6c45d57217196002436eef46a61a9969bb4
# Parent  6ea25bb157e17ec5b8b35aaec82ab1dc3c508d0f
Avoid recomputation of registration instance for lookup.

diff -r 6ea25bb157e1 -r 3877a6c45d57 src/Pure/Isar/locale.ML
--- a/src/Pure/Isar/locale.ML	Mon May 24 10:48:32 2010 +0200
+++ b/src/Pure/Isar/locale.ML	Mon May 24 10:48:32 2010 +0200
@@ -330,7 +330,7 @@
 
 structure Registrations = Theory_Data
 (
-  type T = ((string * (morphism * morphism)) * serial) list *
+  type T = (((string * term list) * (morphism * morphism)) * serial) list *
     (* registrations, in reverse order of declaration;
        serial points to mixin list *)
     (serial * ((morphism * bool) * serial) list) list;
@@ -348,8 +348,8 @@
 
 (* Primitive operations *)
 
-fun add_reg export (dep, morph) =
-  Registrations.map (apfst (cons ((dep, (morph, export)), serial ())));
+fun add_reg thy export (name, morph) =
+  Registrations.map (apfst (cons (((name, instance_of thy name (morph $> export)), (morph, export)), serial ())));
 
 fun add_mixin serial' mixin =
   (* registration to be amended identified by its serial id *)
@@ -359,8 +359,8 @@
   let
     val (regs, mixins) = Registrations.get thy;
   in
-    case find_first (fn ((name', (morph', export')), _) => ident_eq
-      ((name', instance_of thy name' (morph' $> export')), (name, instance_of thy name morph))) (rev regs) of
+    case find_first (fn (((name', inst'), (morph', export')), _) => ident_eq
+      ((name', inst'), (name, instance_of thy name morph))) (rev regs) of
       NONE => []
     | SOME (_, serial) => the_default [] (AList.lookup (op =) mixins serial)
   end;
@@ -379,14 +379,14 @@
   in (name, base $> mix $> export) end;
 
 fun these_registrations thy name = Registrations.get thy
-  |>> filter (curry (op =) name o fst o fst)
+  |>> filter (curry (op =) name o fst o fst o fst)
   (* with inherited mixins *)
-  |-> (fn regs => fn _ => map (fn ((name, (base, export)) ,_) =>
+  |-> (fn regs => fn _ => map (fn (((name, _), (base, export)) ,_) =>
     (name, base $> (collect_mixins thy (name, base $> export) |> compose_mixins) $> export)) regs);
 
 fun all_registrations thy = Registrations.get thy (* FIXME clone *)
   (* with inherited mixins *)
-  |-> (fn regs => fn _ => map (fn ((name, (base, export)) ,_) =>
+  |-> (fn regs => fn _ => map (fn (((name, _), (base, export)) ,_) =>
     (name, base $> (collect_mixins thy (name, base $> export) |> compose_mixins) $> export)) regs);
 
 fun activate_notes' activ_elem transfer thy export (name, morph) input =
@@ -446,8 +446,7 @@
   let
     val regs = Registrations.get thy |> fst;
     val base = instance_of thy name (morph $> export);
-    fun match ((name', (morph', export')), _) =
-      ident_eq ((name, base), (name', instance_of thy name' (morph' $> export)));
+    fun match (((name', inst'), _), _) = ident_eq ((name, base), (name', inst'));
   in
     case find_first match (rev regs) of
         NONE => error ("No interpretation of locale " ^
@@ -472,7 +471,7 @@
     else
       (get_idents (Context.Theory thy), thy)
       (* add new registrations with inherited mixins *)
-      |> roundup thy (add_reg export) export (name, morph)
+      |> roundup thy (add_reg thy export) export (name, morph)
       |> snd
       (* add mixin *)
       |> (case mixin of NONE => I