more internal renaming

Author: andiwand

src/odr/file.cpp

@@ -61,24 +61,16 @@ void File::copy(const std::string &path) const {
 std::shared_ptr<internal::abstract::File> File::impl() const { return m_impl; }
 
 std::vector<FileType> DecodedFile::list_file_types(const std::string &path) {
-  return internal::open_strategy::types(
+  return internal::open_strategy::list_file_types(
       std::make_shared<internal::common::DiskFile>(path));
 }
 
 std::vector<DecoderEngine>
 DecodedFile::list_decoder_engines(const std::string &path, FileType as) {
-  return internal::open_strategy::engines(
+  return internal::open_strategy::list_decoder_engines(
       std::make_shared<internal::common::DiskFile>(path), as);
 }
 
-FileType DecodedFile::get_file_type(const std::string &path) {
-  return DecodedFile(path).file_type();
-}
-
-FileMeta DecodedFile::get_file_meta(const std::string &path) {
-  return DecodedFile(path).file_meta();
-}
-
 DecodedFile::DecodedFile(std::shared_ptr<internal::abstract::DecodedFile> impl)
     : m_impl{std::move(impl)} {
   if (m_impl == nullptr) {

src/odr/file.hpp

@@ -177,8 +177,6 @@ class DecodedFile {
   static std::vector<FileType> list_file_types(const std::string &path);
   static std::vector<DecoderEngine>
   list_decoder_engines(const std::string &path, FileType as);
-  static FileType get_file_type(const std::string &path);
-  static FileMeta get_file_meta(const std::string &path);
 
   explicit DecodedFile(std::shared_ptr<internal::abstract::DecodedFile> impl);
   explicit DecodedFile(const File &file);

src/odr/internal/open_strategy.cpp

@@ -48,7 +48,7 @@ template <typename T> auto priority_comparator(const std::vector<T> &priority) {
 } // namespace
 
 std::vector<FileType>
-open_strategy::types(const std::shared_ptr<abstract::File> &file) {
+open_strategy::list_file_types(const std::shared_ptr<abstract::File> &file) {
   std::vector<FileType> result;
 
   auto file_type = magic::file_type(*file);
@@ -127,9 +127,8 @@ open_strategy::types(const std::shared_ptr<abstract::File> &file) {
   return result;
 }
 
-std::vector<DecoderEngine>
-open_strategy::engines(const std::shared_ptr<abstract::File> & /*file*/,
-                       FileType as) {
+std::vector<DecoderEngine> open_strategy::list_decoder_engines(
+    const std::shared_ptr<abstract::File> & /*file*/, FileType as) {
   std::vector<DecoderEngine> result;
 
   result.push_back(DecoderEngine::odr);
@@ -419,7 +418,7 @@ open_strategy::open_file(std::shared_ptr<abstract::File> file,
   if (preference.as_file_type.has_value()) {
     probe_types.push_back(*preference.as_file_type);
   } else {
-    std::vector<FileType> detected_types = types(file);
+    std::vector<FileType> detected_types = list_file_types(file);
     probe_types.insert(probe_types.end(), detected_types.begin(),
                        detected_types.end());
     auto probe_types_end = std::unique(probe_types.begin(), probe_types.end());
@@ -436,7 +435,8 @@ open_strategy::open_file(std::shared_ptr<abstract::File> file,
     if (preference.with_engine.has_value()) {
       probe_engines.push_back(*preference.with_engine);
     } else {
-      std::vector<DecoderEngine> detected_engines = engines(file, as);
+      std::vector<DecoderEngine> detected_engines =
+          list_decoder_engines(file, as);
       probe_engines.insert(probe_engines.end(), detected_engines.begin(),
                            detected_engines.end());
       auto probe_engines_end =

src/odr/internal/open_strategy.hpp

@@ -21,9 +21,10 @@ class Path;
 
 namespace odr::internal::open_strategy {
 std::vector<FileType>
-types(const std::shared_ptr<internal::abstract::File> &file);
+list_file_types(const std::shared_ptr<internal::abstract::File> &file);
 std::vector<DecoderEngine>
-engines(const std::shared_ptr<internal::abstract::File> &file, FileType as);
+list_decoder_engines(const std::shared_ptr<internal::abstract::File> &file,
+                     FileType as);
 
 std::unique_ptr<internal::abstract::DecodedFile>
 open_file(std::shared_ptr<internal::abstract::File> file);