fmt show metadata

4 files changed, 300 insertions, 198 deletions

diff --git a/src/clients/cave/cmd_show-fmt.hh b/src/clients/cave/cmd_show-fmt.hh
index cf7835e42..2b71fa7b6 100644
--- a/src/clients/cave/cmd_show-fmt.hh
+++ b/src/clients/cave/cmd_show-fmt.hh
@@ -55,3 +55,24 @@ const auto fs_choice_forced_disabled = make_format_string_fetcher("show/choice_f
 const auto fs_choice_disabled = make_format_string_fetcher("show/choice_disabled", 1)
     << c::red() << "-" << param<'s'>() << c::normal() << param<'r'>();
 
+const auto fs_metadata_value_raw = make_format_string_fetcher("show/metadata_value_raw", 1)
+    << "    " << param<'i'>() << param<'i'>() << param<'i'>() << param<'i'>()
+    << param_if<'b'>() << c::bold_normal() << param_endif<'b'>() << param<'s'>() << c::normal()
+    << "%{column 30}" << param<'v'>() << "\\n";
+
+const auto fs_metadata_value_human = make_format_string_fetcher("show/metadata_value_human", 1)
+    << "    " << param<'i'>() << param<'i'>() << param<'i'>() << param<'i'>()
+    << param_if<'b'>() << c::bold_normal() << param_endif<'b'>() << param<'s'>() << c::normal()
+    << "%{column 30}" << param<'v'>() << "\\n";
+
+const auto fs_metadata_continued_value = make_format_string_fetcher("show/metadata_continued_value", 1)
+    << "    " << "%{column 30}" << param<'i'>() << param<'i'>() << param<'v'>() << "\\n";
+
+const auto fs_metadata_subsection_human = make_format_string_fetcher("show/metadata_subsection_human", 1)
+    << "    " << param<'i'>() << param<'i'>() << param<'i'>() << param<'i'>()
+    << c::bold_blue_or_pink() << param<'s'>() << c::normal() << "\\n";
+
+const auto fs_metadata_subsection_raw = make_format_string_fetcher("show/metadata_subsection_raw", 1)
+    << "    " << param<'i'>() << param<'i'>() << param<'i'>() << param<'i'>()
+    << c::bold_blue_or_pink() << param<'s'>() << c::normal() << "\\n";
+
diff --git a/src/clients/cave/cmd_show.cc b/src/clients/cave/cmd_show.cc
index 3c4059ee2..d85293c5a 100644
--- a/src/clients/cave/cmd_show.cc
+++ b/src/clients/cave/cmd_show.cc
@@ -336,10 +336,11 @@ namespace
 
         void visit(const MetadataSectionKey & k)
         {
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_subsection_raw() : f::show_metadata_subsection_human()),
-                    k.raw_name(), k.human_name(), "",
-                    indent, important);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_subsection_raw() : fs_metadata_subsection_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                    fv<'i'>(std::string(indent, ' ')));
+
             std::set<std::shared_ptr<const MetadataKey>, MetadataKeyComparator> keys(k.begin_metadata(), k.end_metadata());
             for (std::set<std::shared_ptr<const MetadataKey>, MetadataKeyComparator>::const_iterator
                     s(keys.begin()), s_end(keys.end()) ; s != s_end ; ++s)
@@ -353,54 +354,72 @@ namespace
         void visit(const MetadataCollectionKey<KeywordNameSet> & k)
         {
             ColourFormatter f(indent);
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    k.raw_name(), k.human_name(),
-                    k.pretty_print_flat(f), indent, important);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                    fv<'v'>(k.pretty_print_flat(f)),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>(important ? "true" : "")
+                    );
         }
 
         void visit(const MetadataCollectionKey<Set<std::string> > & k)
         {
             ColourFormatter f(indent);
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    k.raw_name(), k.human_name(),
-                    k.pretty_print_flat(f), indent, important);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                    fv<'v'>(k.pretty_print_flat(f)),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>(important ? "true" : "")
+                    );
         }
 
         void visit(const MetadataCollectionKey<Sequence<std::string> > & k)
         {
             ColourFormatter f(indent);
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    k.raw_name(), k.human_name(),
-                    k.pretty_print_flat(f), indent, important);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                    fv<'v'>(k.pretty_print_flat(f)),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>(important ? "true" : "")
+                    );
         }
 
         void visit(const MetadataCollectionKey<PackageIDSequence> & k)
         {
             ColourFormatter f(indent);
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    k.raw_name(), k.human_name(),
-                    k.pretty_print_flat(f), indent, important);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                    fv<'v'>(k.pretty_print_flat(f)),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>(important ? "true" : "")
+                    );
         }
 
         void visit(const MetadataCollectionKey<FSEntrySequence> & k)
         {
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    k.raw_name(), k.human_name(),
-                    join(k.value()->begin(), k.value()->end(), ", "), indent, important);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                    fv<'v'>(join(k.value()->begin(), k.value()->end(), ", ")),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>(important ? "true" : "")
+                    );
         }
 
         void visit(const MetadataSpecTreeKey<LicenseSpecTree> & k)
         {
             ColourFormatter f(indent);
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    k.raw_name(), k.human_name(),
-                    k.pretty_print_flat(f), indent, important);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                    fv<'v'>(k.pretty_print_flat(f)),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>(important ? "true" : "")
+                    );
         }
 
         void visit(const MetadataSpecTreeKey<SimpleURISpecTree> & k)
@@ -408,10 +427,13 @@ namespace
             if (cmdline.a_complex_keys.specified() || important)
             {
                 ColourFormatter f(indent);
-                cout << format_general_rhvib(
-                        (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                        k.raw_name(), k.human_name(),
-                        k.pretty_print_flat(f), indent, important);
+                cout << fuc(
+                        (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                        fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                        fv<'v'>(k.pretty_print_flat(f)),
+                        fv<'i'>(std::string(indent, ' ')),
+                        fv<'b'>(important ? "true" : "")
+                        );
             }
         }
 
@@ -421,16 +443,22 @@ namespace
             {
                 ColourFormatter f(indent);
                 if (cmdline.a_flat.specified())
-                    cout << format_general_rhvib(
-                            (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                            k.raw_name(), k.human_name(),
-                            k.pretty_print_flat(f), indent, important);
+                    cout << fuc(
+                            (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                            fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                            fv<'v'>(k.pretty_print_flat(f)),
+                            fv<'i'>(std::string(indent, ' ')),
+                            fv<'b'>(important ? "true" : "")
+                            );
                 else
                 {
-                    cout << format_general_rhvib(
-                            (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                            k.raw_name(), k.human_name(),
-                            "", indent, important);
+                    cout << fuc(
+                            (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                            fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                            fv<'v'>(""),
+                            fv<'i'>(std::string(indent, ' ')),
+                            fv<'b'>(important ? "true" : "")
+                            );
                     cout << k.pretty_print(f);
                 }
             }
@@ -442,16 +470,22 @@ namespace
             {
                 ColourFormatter f(indent);
                 if (cmdline.a_flat.specified())
-                    cout << format_general_rhvib(
-                            (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                            k.raw_name(), k.human_name(),
-                            k.pretty_print_flat(f), indent, important);
+                    cout << fuc(
+                            (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                            fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                            fv<'v'>(k.pretty_print_flat(f)),
+                            fv<'i'>(std::string(indent, ' ')),
+                            fv<'b'>(important ? "true" : "")
+                            );
                 else
                 {
-                    cout << format_general_rhvib(
-                            (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                            k.raw_name(), k.human_name(),
-                            "", indent, important);
+                    cout << fuc(
+                            (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                            fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                            fv<'v'>(""),
+                            fv<'i'>(std::string(indent, ' ')),
+                            fv<'b'>(important ? "true" : "")
+                            );
                     cout << k.pretty_print(f);
                 }
             }
@@ -462,10 +496,13 @@ namespace
             if (cmdline.a_complex_keys.specified() || important)
             {
                 ColourFormatter f(indent);
-                cout << format_general_rhvib(
-                        (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                        k.raw_name(), k.human_name(),
-                        k.pretty_print_flat(f), indent, important);
+                cout << fuc(
+                        (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                        fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                        fv<'v'>(k.pretty_print_flat(f)),
+                        fv<'i'>(std::string(indent, ' ')),
+                        fv<'b'>(important ? "true" : "")
+                        );
             }
         }
 
@@ -475,16 +512,22 @@ namespace
             {
                 ColourFormatter f(indent);
                 if (cmdline.a_flat.specified())
-                    cout << format_general_rhvib(
-                            (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                            k.raw_name(), k.human_name(),
-                            k.pretty_print_flat(f), indent, important);
+                    cout << fuc(
+                            (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                            fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                            fv<'v'>(k.pretty_print_flat(f)),
+                            fv<'i'>(std::string(indent, ' ')),
+                            fv<'b'>(important ? "true" : "")
+                            );
                 else
                 {
-                    cout << format_general_rhvib(
-                            (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                            k.raw_name(), k.human_name(),
-                            "", indent, important);
+                    cout << fuc(
+                            (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                            fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                            fv<'v'>(""),
+                            fv<'i'>(std::string(indent, ' ')),
+                            fv<'b'>(important ? "true" : "")
+                            );
                     cout << k.pretty_print(f);
                 }
             }
@@ -492,50 +535,68 @@ namespace
 
         void visit(const MetadataValueKey<std::string> & k)
         {
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    k.raw_name(), k.human_name(),
-                    stringify(k.value()), indent, important);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                    fv<'v'>(stringify(k.value())),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>(important ? "true" : "")
+                    );
         }
 
         void visit(const MetadataValueKey<SlotName> & k)
         {
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    k.raw_name(), k.human_name(),
-                    stringify(k.value()), indent, important);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                    fv<'v'>(stringify(k.value())),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>(important ? "true" : "")
+                    );
         }
 
         void visit(const MetadataValueKey<long> & k)
         {
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    k.raw_name(), k.human_name(),
-                    stringify(k.value()), indent, important);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                    fv<'v'>(stringify(k.value())),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>(important ? "true" : "")
+                    );
         }
 
         void visit(const MetadataValueKey<bool> & k)
         {
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    k.raw_name(), k.human_name(),
-                    stringify(k.value() ? "true" : "false"), indent, important);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                    fv<'v'>(stringify(k.value())),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>(important ? "true" : "")
+                    );
         }
 
         void visit(const MetadataValueKey<FSEntry> & k)
         {
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    k.raw_name(), k.human_name(),
-                    stringify(k.value()), indent, important);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                    fv<'v'>(stringify(k.value())),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>(important ? "true" : "")
+                    );
         }
 
         void visit(const MetadataValueKey<std::shared_ptr<const PackageID> > & k)
         {
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    k.raw_name(), k.human_name(),
-                    stringify(*k.value()), indent, important);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                    fv<'v'>(stringify(*k.value())),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>(important ? "true" : "")
+                    );
         }
 
         void visit(const MetadataValueKey<std::shared_ptr<const Contents> > & k)
@@ -547,20 +608,26 @@ namespace
                     ContentsDisplayer d(0);
                     std::for_each(indirect_iterator(k.value()->begin()),
                             indirect_iterator(k.value()->end()), accept_visitor(d));
-                    cout << format_general_rhvib(
-                            (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                            k.raw_name(), k.human_name(),
-                            d.s.str(), indent, important);
+                    cout << fuc(
+                            (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                            fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                            fv<'v'>(d.s.str()),
+                            fv<'i'>(std::string(indent, ' ')),
+                            fv<'b'>(important ? "true" : "")
+                            );
                 }
                 else
                 {
                     ContentsDisplayer d(indent);
                     std::for_each(indirect_iterator(k.value()->begin()),
                             indirect_iterator(k.value()->end()), accept_visitor(d));
-                    cout << format_general_rhvib(
-                            (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                            k.raw_name(), k.human_name(),
-                            "", indent, important);
+                    cout << fuc(
+                            (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                            fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                            fv<'v'>(""),
+                            fv<'i'>(std::string(indent, ' ')),
+                            fv<'b'>(important ? "true" : "")
+                            );
                     cout << d.s.str();
                 }
             }
@@ -628,17 +695,25 @@ namespace
                         }
                     }
                 }
-                cout << format_general_rhvib(
-                        (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                        k.raw_name(), k.human_name(),
-                        s.str(), indent, important);
+
+                cout << fuc(
+                        (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                        fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                        fv<'v'>(s.str()),
+                        fv<'i'>(std::string(indent, ' ')),
+                        fv<'b'>(important ? "true" : "")
+                        );
             }
             else
             {
-                cout << format_general_rhvib(
-                        (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                        k.raw_name(), k.human_name(),
-                        "", indent, important);
+                cout << fuc(
+                        (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                        fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                        fv<'v'>(""),
+                        fv<'i'>(std::string(indent, ' ')),
+                        fv<'b'>(important ? "true" : "")
+                        );
+
                 for (Choices::ConstIterator c(k.value()->begin()), c_end(k.value()->end()) ;
                         c != c_end ; ++c)
                 {
@@ -662,10 +737,13 @@ namespace
                             continue;
                     }
 
-                    cout << format_general_rhvib(
-                            (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                            (*c)->raw_name(), (*c)->human_name(),
-                            "", indent + 1, important);
+                    cout << fuc(
+                            (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                            fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                            fv<'v'>(""),
+                            fv<'i'>(std::string(indent + 1, ' ')),
+                            fv<'b'>(important ? "true" : "")
+                            );
 
                     for (Choice::ConstIterator v((*c)->begin()), v_end((*c)->end()) ;
                             v != v_end ; ++v)
@@ -678,46 +756,62 @@ namespace
                         {
                             if ((*v)->locked())
                             {
-                                cout << format_general_rhvib(
-                                        (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                                        fuc(fs_choice_forced_enabled(), fv<'s'>(stringify((*v)->name_with_prefix())),
-                                            fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed))),
-                                        fuc(fs_choice_forced_enabled(), fv<'s'>(stringify((*v)->unprefixed_name())),
-                                            fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed))),
-                                        (*v)->description(), indent + 2, important);
+                                cout << fuc(
+                                        (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                                        fv<'s'>(cmdline.a_raw_names.specified() ?
+                                            fuc(fs_choice_forced_enabled(), fv<'s'>(stringify((*v)->name_with_prefix())),
+                                                fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed))) :
+                                            fuc(fs_choice_forced_enabled(), fv<'s'>(stringify((*v)->unprefixed_name())),
+                                                fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed)))),
+                                        fv<'v'>((*v)->description()),
+                                        fv<'i'>(std::string(indent + 2, ' ')),
+                                        fv<'b'>(important ? "true" : "")
+                                        );
                             }
                             else
                             {
-                                cout << format_general_rhvib(
-                                        (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                                        fuc(fs_choice_enabled(), fv<'s'>(stringify((*v)->name_with_prefix())),
-                                            fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed))),
-                                        fuc(fs_choice_enabled(), fv<'s'>(stringify((*v)->unprefixed_name())),
-                                            fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed))),
-                                        (*v)->description(), indent + 2, important);
+                                cout << fuc(
+                                        (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                                        fv<'s'>(cmdline.a_raw_names.specified() ?
+                                            fuc(fs_choice_enabled(), fv<'s'>(stringify((*v)->name_with_prefix())),
+                                                fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed))) :
+                                            fuc(fs_choice_enabled(), fv<'s'>(stringify((*v)->unprefixed_name())),
+                                                fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed)))),
+                                        fv<'v'>((*v)->description()),
+                                        fv<'i'>(std::string(indent + 2, ' ')),
+                                        fv<'b'>(important ? "true" : "")
+                                        );
                             }
                         }
                         else
                         {
                             if ((*v)->locked())
                             {
-                                cout << format_general_rhvib(
-                                        (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                                        fuc(fs_choice_forced_disabled(), fv<'s'>(stringify((*v)->name_with_prefix())),
-                                            fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed))),
-                                        fuc(fs_choice_forced_disabled(), fv<'s'>(stringify((*v)->unprefixed_name())),
-                                            fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed))),
-                                        (*v)->description(), indent + 2, important);
+                                cout << fuc(
+                                        (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                                        fv<'s'>(cmdline.a_raw_names.specified() ?
+                                            fuc(fs_choice_forced_disabled(), fv<'s'>(stringify((*v)->name_with_prefix())),
+                                                fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed))) :
+                                            fuc(fs_choice_forced_disabled(), fv<'s'>(stringify((*v)->unprefixed_name())),
+                                                fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed)))),
+                                        fv<'v'>((*v)->description()),
+                                        fv<'i'>(std::string(indent + 2, ' ')),
+                                        fv<'b'>(important ? "true" : "")
+                                        );
                             }
                             else
                             {
-                                cout << format_general_rhvib(
-                                        (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                                        fuc(fs_choice_disabled(), fv<'s'>(stringify((*v)->name_with_prefix())),
-                                            fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed))),
-                                        fuc(fs_choice_disabled(), fv<'s'>(stringify((*v)->unprefixed_name())),
-                                            fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed))),
-                                        (*v)->description(), indent + 2, important);
+                                cout << fuc(
+                                        (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                                        fv<'s'>(cmdline.a_raw_names.specified() ?
+                                            fuc(fs_choice_disabled(), fv<'s'>(stringify((*v)->name_with_prefix())),
+                                                fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed))) :
+                                            fuc(fs_choice_disabled(), fv<'s'>(stringify((*v)->unprefixed_name())),
+                                                fv<'r'>(added_or_changed_string(*c, *v, maybe_old_id, old_id_is_installed)))),
+                                        fv<'v'>((*v)->description()),
+                                        fv<'i'>(std::string(indent + 2, ' ')),
+                                        fv<'b'>(important ? "true" : "")
+                                        );
                             }
                         }
                     }
@@ -729,23 +823,31 @@ namespace
         {
             if (k.value())
             {
-                cout << format_general_rhvib(
-                        (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                        k.raw_name(), k.human_name(),
-                        stringify(k.value()->mask_file()), indent, important);
+                cout << fuc(
+                        (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                        fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                        fv<'v'>(stringify(k.value()->mask_file())),
+                        fv<'i'>(std::string(indent, ' ')),
+                        fv<'b'>(important ? "true" : "")
+                        );
                 for (Sequence<std::string>::ConstIterator i(k.value()->comment()->begin()), i_end(k.value()->comment()->end()) ;
                         i != i_end ; ++i)
-                    cout << format_general_rhvib(f::show_metadata_continued_value(), k.raw_name(), k.human_name(),
-                            *i, 0, important);
+                    cout << fuc(fs_metadata_continued_value(),
+                            fv<'v'>(*i),
+                            fv<'i'>("")
+                            );
             }
         }
 
         void visit(const MetadataTimeKey & k)
         {
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    k.raw_name(), k.human_name(),
-                    pretty_print_time(k.value().seconds()), indent, important);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? k.raw_name() : k.human_name()),
+                    fv<'v'>(pretty_print_time(k.value().seconds())),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>(important ? "true" : "")
+                    );
         }
     };
 
@@ -769,35 +871,47 @@ namespace
             }
             else
             {
-                cout << format_general_rhvib(
-                        (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                        "Masked", "Masked",
-                        "by " + m.description(), indent, false);
+                cout << fuc(
+                        (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                        fv<'s'>("Masked"),
+                        fv<'v'>("by " + m.description()),
+                        fv<'i'>(std::string(indent, ' ')),
+                        fv<'b'>("")
+                        );
             }
         }
 
         void visit(const UnsupportedMask & m)
         {
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    stringify(m.key()),
-                    m.description(), m.explanation(), indent, false);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? stringify(m.key()) : m.description()),
+                    fv<'v'>(m.explanation()),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>("")
+                    );
         }
 
         void visit(const AssociationMask & m)
         {
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    stringify(m.key()),
-                    "by " + m.description(), stringify(*m.associated_package()), indent, false);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? stringify(m.key()) : "by " + m.description()),
+                    fv<'v'>(stringify(*m.associated_package())),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>("")
+                    );
         }
 
         void visit(const UserMask & m)
         {
-            cout << format_general_rhvib(
-                    (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                    stringify(m.key()),
-                    "by " + m.description(), "", indent, false);
+            cout << fuc(
+                    (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                    fv<'s'>(cmdline.a_raw_names.specified() ? stringify(m.key()) : "by " + m.description()),
+                    fv<'v'>(""),
+                    fv<'i'>(std::string(indent, ' ')),
+                    fv<'b'>("")
+                    );
         }
 
         void visit(const RepositoryMask & m)
@@ -809,10 +923,13 @@ namespace
             }
             else
             {
-                cout << format_general_rhvib(
-                        (cmdline.a_raw_names.specified() ? f::show_metadata_key_value_raw() : f::show_metadata_key_value_human()),
-                        stringify(m.key()),
-                        "by " + m.description(), "", indent, false);
+                cout << fuc(
+                        (cmdline.a_raw_names.specified() ? fs_metadata_value_raw() : fs_metadata_value_human()),
+                        fv<'s'>(cmdline.a_raw_names.specified() ? stringify(m.key()) : "by " + m.description()),
+                        fv<'v'>(""),
+                        fv<'i'>(std::string(indent, ' ')),
+                        fv<'b'>("")
+                        );
             }
         }
     };
diff --git a/src/clients/cave/formats.cc b/src/clients/cave/formats.cc
index dedb933a8..97328987b 100644
--- a/src/clients/cave/formats.cc
+++ b/src/clients/cave/formats.cc
@@ -137,36 +137,6 @@ paludis::cave::c::bold_normal()
 }
 
 const std::string
-paludis::cave::f::show_metadata_key_value_raw()
-{
-    return "    %i%i%i%i%{if b}" + c::bold_normal() + "%{endif}" + "%r" + c::normal() + "%{column 30}%v\\n";
-}
-
-const std::string
-paludis::cave::f::show_metadata_key_value_human()
-{
-    return "    %i%i%i%i%{if b}" + c::bold_normal() + "%{endif}" + "%h" + c::normal() + "%{column 30}%v\\n";
-}
-
-const std::string
-paludis::cave::f::show_metadata_continued_value()
-{
-    return "    %{column 30}%i%i%v\\n";
-}
-
-const std::string
-paludis::cave::f::show_metadata_subsection_human()
-{
-    return "    %i%i%i%i" + c::bold_blue_or_pink() + "%h" + c::normal() + "\\n";
-}
-
-const std::string
-paludis::cave::f::show_metadata_subsection_raw()
-{
-    return "    %i%i%i%i" + c::bold_blue_or_pink() + "%r" + c::normal() + "\\n";
-}
-
-const std::string
 paludis::cave::f::show_package_heading()
 {
     return "* " + c::bold_blue_or_pink() + "%s" + c::normal() + "\\n";
diff --git a/src/clients/cave/formats.hh b/src/clients/cave/formats.hh
index 0d05fcc6c..1e24feb76 100644
--- a/src/clients/cave/formats.hh
+++ b/src/clients/cave/formats.hh
@@ -62,12 +62,6 @@ namespace paludis
             const std::string show_package_id_heading();
             const std::string show_package_id_masks();
             const std::string show_package_id_masks_overridden();
-
-            const std::string show_metadata_key_value_raw();
-            const std::string show_metadata_key_value_human();
-            const std::string show_metadata_continued_value();
-            const std::string show_metadata_subsection_raw();
-            const std::string show_metadata_subsection_human();
         }
     }
 }