# HG changeset patch
# User blanchet
# Date 1361208895 -3600
# Node ID d0fa18638478e3f88a6c21facdee14290d40034c
# Parent  5cbfc644c2614d5ce36bd9eaba15b1a06ba0117e
implement (more) accurate computation of parents

diff -r 5cbfc644c261 -r d0fa18638478 src/HOL/Tools/Sledgehammer/sledgehammer_mash.ML
--- a/src/HOL/Tools/Sledgehammer/sledgehammer_mash.ML	Mon Feb 18 18:34:54 2013 +0100
+++ b/src/HOL/Tools/Sledgehammer/sledgehammer_mash.ML	Mon Feb 18 18:34:55 2013 +0100
@@ -437,13 +437,15 @@
   elide_string threshold
     (backquote_thm (Proof_Context.init_global (Thm.theory_of_thm th)) th)
 
+val thy_name_of_thm = Context.theory_name o Thm.theory_of_thm
+
 fun nickname_of_thm th =
   if Thm.has_name_hint th then
     let val hint = Thm.get_name_hint th in
       (* There must be a better way to detect local facts. *)
       case try (unprefix local_prefix) hint of
         SOME suf =>
-        Context.theory_name (Thm.theory_of_thm th) ^ Long_Name.separator ^ suf ^
+        thy_name_of_thm th ^ Long_Name.separator ^ suf ^
         Long_Name.separator ^ elided_backquote_thm 50 th
       | NONE => hint
     end
@@ -900,14 +902,49 @@
         (true, "")
       end)
 
+fun chunks_and_parents_for chunks th =
+  let
+    fun insert_parent new parents =
+      let val parents = parents |> filter_out (fn p => thm_less_eq (p, new)) in
+        parents |> forall (fn p => not (thm_less_eq (new, p))) parents
+                   ? cons new
+      end
+    fun rechunk seen (rest as th' :: ths) =
+      if thm_less_eq (th', th) then (rev seen, rest)
+      else rechunk (th' :: seen) ths
+    fun do_chunk [] accum = accum
+      | do_chunk (chunk as hd_chunk :: _) (chunks, parents) =
+        if thm_less_eq (hd_chunk, th) then
+          (chunk :: chunks, insert_parent hd_chunk parents)
+        else if thm_less_eq (List.last chunk, th) then
+          let val (front, back as hd_back :: _) = rechunk [] chunk in
+            (front :: back :: chunks, insert_parent hd_back parents)
+          end
+        else
+          (chunk :: chunks, parents)
+  in
+    fold_rev do_chunk chunks ([], [])
+    |>> cons []
+  end
+
 fun attach_parents_to_facts facts =
   let
-    fun do_facts _ [] = []
-      | do_facts parents ((fact as (_, th)) :: facts) =
-        (parents, fact) :: do_facts [nickname_of_thm th] facts
+    fun do_facts _ _ [] = []
+      | do_facts _ parents [fact] = [(parents, fact)]
+      | do_facts chunks parents ((fact as (_, th)) :: (facts as (_, th') :: _)) =
+        let
+          val chunks = app_hd (cons th) chunks
+          val (chunks', parents') =
+            (if thm_less_eq (th, th') andalso
+                thy_name_of_thm th = thy_name_of_thm th' then
+               (chunks, [th])
+             else
+               chunks_and_parents_for chunks th')
+            ||> map nickname_of_thm
+        in (parents, fact) :: do_facts chunks' parents' facts end
   in
     facts |> sort (crude_thm_ord o pairself snd)
-          |> do_facts []
+          |> do_facts [[]] []
   end
 
 fun sendback sub = Active.sendback_markup (sledgehammerN ^ " " ^ sub)
diff -r 5cbfc644c261 -r d0fa18638478 src/HOL/Tools/Sledgehammer/sledgehammer_provers.ML
--- a/src/HOL/Tools/Sledgehammer/sledgehammer_provers.ML	Mon Feb 18 18:34:54 2013 +0100
+++ b/src/HOL/Tools/Sledgehammer/sledgehammer_provers.ML	Mon Feb 18 18:34:55 2013 +0100
@@ -930,8 +930,6 @@
 
 fun rotate_one (x :: xs) = xs @ [x]
 
-fun app_hd f (x :: xs) = f x :: xs
-
 (* "SMT_Failure.Abnormal_Termination" carries the solver's return code. Until
    these are sorted out properly in the SMT module, we have to interpret these
    ourselves. *)
diff -r 5cbfc644c261 -r d0fa18638478 src/HOL/Tools/Sledgehammer/sledgehammer_util.ML
--- a/src/HOL/Tools/Sledgehammer/sledgehammer_util.ML	Mon Feb 18 18:34:54 2013 +0100
+++ b/src/HOL/Tools/Sledgehammer/sledgehammer_util.ML	Mon Feb 18 18:34:55 2013 +0100
@@ -8,6 +8,7 @@
 sig
   val sledgehammerN : string
   val log2 : real -> real
+  val app_hd : ('a -> 'a) -> 'a list -> 'a list
   val plural_s : int -> string
   val serial_commas : string -> string list -> string list
   val simplify_spaces : string -> string
@@ -38,6 +39,8 @@
 
 fun log2 n = Math.log10 n / log10_2
 
+fun app_hd f (x :: xs) = f x :: xs
+
 fun plural_s n = if n = 1 then "" else "s"
 
 val serial_commas = Try.serial_commas