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: check uses of Unsynchronized.ref vs. Synchronized.var;
9 * MW: proper formal name space for rule set, model patterns, methods;
12 * MW: clarify/eliminate Isabelle/Scala add-ons (presently unused)
14 diff -r /home/makarius/isabelle/repos-Isabelle2021/src/Pure/build-jars ./src/Pure/build-jars
16 > src/Tools/isac/BridgeJEdit/isac.scala
18 diff -r /home/makarius/isabelle/repos-Isabelle2021/src/Pure/Tools/scala_project.scala ./src/Pure/Tools/scala_project.scala
20 > "src/Tools/isac/etc" -> Path.explode("isabelle.isac"),
22 * MW: when Isabelle inner syntax is processed literally in Isabelle/ML, prefer formal Input.source
23 with literal \<open>...\<close> instead of old-fashioned "..."; this will eventually allow automated update of
24 old ASCII notation (e.g. "EX" vs. "\<exists>"), with the help of PIDE markup;
27 * Clarify symmetric rule: Thm.apply_attribute Calculation.symmetric thm context (!?);
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: clarify Rule.Thm ("minus_divide_left", ...): Should this be @{rule_thm_sym} antiquotation?
47 * WN: remove always empty arguments in MethodC.prep_input and Problem.prep_input
48 (following the "TODO" comment);
50 * WN: trim-down MethodC.prep_input and Problem.prep_input to what is really needed.
52 * WN: eliminate ThyC.to_ctxt, use Proof_Context.init_global inline to make more
53 clear where the normal Isabelle context-discipline is bypassed;
55 * WN: standardize theory for KEStore_Elems.add_rlss, KEStore_Elems.add_mets, KEStore_Elems.add_pbts
56 - should be always the current @{theory} of the enclosing "setup" command;
57 - avoid old-style "val thy = @{theory}" somewhere in a theory file;
59 * WN: more direct logical foundations wrt. Isabelle/HOL, eliminate many axiomatizations
60 - quite often "axiomatization ..." can be turned into "lemma ... by auto"
62 - sometimes this requires to use more specific types / type classes;
63 - sometimes this requires to use proper definitional mechanisms (e.g. 'primrec', 'fun');
64 - a few "hard" cases will remain, to be reconsidered eventually (e.g. differentiation);
66 * WN: eliminate ThmC.numerals_to_Free, use existing Isabelle/HOL representation
67 - clarify role of type "real" vs. "float" (see theory "HOL-Library.Float");
69 * WN: Part.DONE cleanup remaining ^^^ in comments (but sometimes it is just ASCII art);
70 Left ^^^ in doc-isac (old master-theses, etc: "x^^^#2 + #8" ... # are left, too)
71 Left "ASCII art" in case of indicating comments pointing at facts ABOVE.
73 * WN: "fun pr_ord" is not required if used with @{make_string}, @{print}, @{print tracing};
75 * WN: eliminate global flags like "trace_on", replace Unsynchronized.ref by
78 val rewrite_trace = Attrib.setup_config_bool \<^binding>\<open>rewrite_trace\<close> (K false);