simplified Exn.EXCEPTIONS, flatten nested occurrences;
authorwenzelm
Thu, 02 Oct 2008 14:22:44 +0200
changeset 28459 f6a4d913cfb1
parent 28458 0966ac3f4a40
child 28460 455ef74607d7
simplified Exn.EXCEPTIONS, flatten nested occurrences;
src/Pure/ML-Systems/exn.ML
--- a/src/Pure/ML-Systems/exn.ML	Thu Oct 02 14:22:40 2008 +0200
+++ b/src/Pure/ML-Systems/exn.ML	Thu Oct 02 14:22:44 2008 +0200
@@ -13,8 +13,7 @@
   val capture: ('a -> 'b) -> 'a -> 'b result
   val release: 'a result -> 'a
   exception Interrupt
-  val proper_exn: 'a result -> exn option
-  exception EXCEPTIONS of exn list * string
+  exception EXCEPTIONS of exn list
   val release_all: 'a result list -> 'a list
   val release_first: 'a result list -> 'a list
 end;
@@ -40,28 +39,26 @@
   | release (Exn e) = raise e;
 
 
-(* interrupt *)
+(* interrupt and nested exceptions *)
 
 exception Interrupt = Interrupt;
-
-fun proper_exn (Result _) = NONE
-  | proper_exn (Exn Interrupt) = NONE
-  | proper_exn (Exn exn) = SOME exn;
+exception EXCEPTIONS of exn list;
 
+fun plain_exns (Result _) = []
+  | plain_exns (Exn Interrupt) = []
+  | plain_exns (Exn (EXCEPTIONS exns)) = List.concat (map (plain_exns o Exn) exns)
+  | plain_exns (Exn exn) = [exn];
 
-(* release results -- collating interrupts *)
-
-exception EXCEPTIONS of exn list * string;
 
 fun release_all results =
   if List.all (fn Result _ => true | _ => false) results
   then map (fn Result x => x) results
   else
-    (case List.mapPartial proper_exn results of
+    (case List.concat (map plain_exns results) of
       [] => raise Interrupt
-    | exns => raise EXCEPTIONS (exns, ""));
+    | exns => raise EXCEPTIONS exns);
 
 fun release_first results = release_all results
-  handle EXCEPTIONS (exn :: _, _) => raise exn;
+  handle EXCEPTIONS (exn :: _) => raise exn;
 
 end;