1 * reconsider use of Thy_Info.get_theory: only works with batch-build, not within PIDE session;
2 + consider in Build_Thydata: Thy_Hierarchy.insert_errpatIDs
3 + consider in Biegelinie.thy: KEStore_Elems.add_thes,
5 * MW: ML antiqutation @{rule_thm NAME} to produce (Rule.Thm ("NAME", ThmC.numerals_to_Free "NAME"));
7 * MW: more concise "setup KEStore_Elems.add_rlss" etc.;
9 * MW: check uses of Unsynchronized.ref vs. Synchronized.var;
11 * MW: proper formal name space for rule set, model patterns, methods;
14 * MW: clarify/eliminate Isabelle/Scala add-ons (presently unused)
16 diff -r /home/makarius/isabelle/repos-Isabelle2021/src/Pure/build-jars ./src/Pure/build-jars
18 > src/Tools/isac/BridgeJEdit/isac.scala
20 diff -r /home/makarius/isabelle/repos-Isabelle2021/src/Pure/Tools/scala_project.scala ./src/Pure/Tools/scala_project.scala
22 > "src/Tools/isac/etc" -> Path.explode("isabelle.isac"),
24 * MW: when Isabelle inner syntax is processed literally in Isabelle/ML, prefer formal Input.source
25 with literal \<open>...\<close> instead of old-fashioned "..."; this will eventually allow automated update of
26 old ASCII notation (e.g. "EX" vs. "\<exists>"), with the help of PIDE markup;
29 * Clarify rule_set "prog_expr": avoid rebinding!?
31 * rule_set: proper name space and declaration order (!?);
33 * Avoid Thm.get_name_hint --- somewhat fragile.
35 * Is it possible to eliminate union_overwrite, in favour of more standard data
38 * What is the purpose of "#numeral" instead of plain numeral?
40 * Check/clarify Context.theory_name vs. Context.theory_long_name.
42 * Eliminate mutable Rewrite_Ord.rew_ord' (!?);
45 * WN: remove always empty arguments in MethodC.prep_input and Problem.prep_input
46 (following the "TODO" comment);
48 * WN: trim-down MethodC.prep_input and Problem.prep_input to what is really needed.
50 * WN: eliminate ThyC.to_ctxt, use Proof_Context.init_global inline to make more
51 clear where the normal Isabelle context-discipline is bypassed;
53 * WN: standardize theory for KEStore_Elems.add_rlss, KEStore_Elems.add_mets, KEStore_Elems.add_pbts
54 - should be always the current @{theory} of the enclosing "setup" command;
55 - avoid old-style "val thy = @{theory}" somewhere in a theory file;
57 * WN: more direct logical foundations wrt. Isabelle/HOL, eliminate many axiomatizations
58 - quite often "axiomatization ..." can be turned into "lemma ... by auto"
60 - sometimes this requires to use more specific types / type classes;
61 - sometimes this requires to use proper definitional mechanisms (e.g. 'primrec', 'fun');
62 - a few "hard" cases will remain, to be reconsidered eventually (e.g. differentiation);
64 * WN: eliminate ThmC.numerals_to_Free, use existing Isabelle/HOL representation
65 - clarify role of type "real" vs. "float" (see theory "HOL-Library.Float");
67 * WN: Part.DONE cleanup remaining ^^^ in comments (but sometimes it is just ASCII art);
68 Left ^^^ in doc-isac (old master-theses, etc: "x^^^#2 + #8" ... # are left, too)
69 Left "ASCII art" in case of indicating comments pointing at facts ABOVE.
71 * WN: "fun pr_ord" is not required if used with @{make_string}, @{print}, @{print tracing};
73 * WN: eliminate global flags like "trace_on", replace Unsynchronized.ref by
76 val rewrite_trace = Attrib.setup_config_bool \<^binding>\<open>rewrite_trace\<close> (K false);