Remove GFilename::default_extname() and enhance GFilename::extname() method.

The GFilename::default_extname() has been removed and remplaced by an enhanced GFilename::extname() method that either returns the extension name set in the filename, or if not set, the string that has been provided as argument.

Remove GFilename::default_extname() and enhance GFilename::extname() method.
The GFilename::default_extname() has been removed and remplaced by an enhanced GFilename::extname() method that either returns the extension name set in the filename, or if not set, the string that has been provided as argument.
Jürgen Knödlseder
1 parent 537d0e9d
Showing 4 changed files with 18 additions and 28 deletions
include/GFilename.hpp
pyext/GFilename.i
src/support/GFilename.cpp
test/test_GSupport.cpp
@@ -60,13 +60,12 @@ public:
     int                size(void) const;
     int                length(void) const;
     const std::string& filename(void) const;
-    const std::string& extname(void) const;
+    std::string        extname(const std::string& defaultname = "") const;
     const int&         extno(void) const;
     const int&         extver(void) const;
     bool               has_extname(void) const;
     bool               has_extno(void) const;
     bool               has_extver(void) const;
-    void               default_extname(const std::string& extname);
     std::string        print(const GChatter& chatter = NORMAL) const;
 protected:
@@ -147,18 +146,6 @@ const std::string&amp; GFilename::filename(void) const
 /***********************************************************************//**
- * @brief Return extension name
- *
- * @return String containing extension name.
- ***************************************************************************/
-inline
-const std::string& GFilename::extname(void) const
-{
-    return (m_extname);
-}
-
-
-/***********************************************************************//**
  * @brief Return extension number
  *
  * @return Extension number (-1 if not set).
@@ -52,13 +52,12 @@ public:
     int                size(void) const;
     int                length(void) const;
     const std::string& filename(void) const;
-    const std::string& extname(void) const;
+    std::string        extname(const std::string& defaultname = "") const;
     const int&         extno(void) const;
     const int&         extver(void) const;
     bool               has_extname(void) const;
     bool               has_extno(void) const;
     bool               has_extver(void) const;
-    void               default_extname(const std::string& extname);
 };
@@ -200,21 +200,27 @@ GFilename* GFilename::clone(void) const
 /***********************************************************************//**
- * @brief Set default extension name
+ * @brief Return extension name
  *
- * @param[in] extname Extension name.
+ * @param[in] defaultname Default extension name (default: "").
+ * @return String containing extension name.
  *
- * If the object has no extension name then set the extension name.
+ * Returns the extension name. If no extension name is given the name
+ * provided by the @p defaultname argument will be used. By default, the
+ * @p defaultname parameter is an empty string.
  ***************************************************************************/
-void GFilename::default_extname(const std::string& extname)
+std::string GFilename::extname(const std::string& defaultname) const
 {
-    // Set extension name if it has not yet been set
+    // Set the extension name
+    std::string extname = m_extname;
+
+    // If no extension name is set, the use the default extension name
     if (!has_extname()) {
-        m_extname = extname;
+        extname = defaultname;
     }
-    // Return
-    return;
+    // Return extension name
+    return (extname);
 }
@@ -760,13 +760,11 @@ void TestGSupport::test_filename(void)
     // Test default extension name
     filename = "myfile.fits";
-    filename.default_extname("EVENTS");
-    test_assert(filename.extname() == "EVENTS",
+    test_assert(filename.extname("EVENTS") == "EVENTS",
                 "Expected \"EVENTS\" extension name, "
                 "found \""+filename.extname()+"\"");
     filename = "myfile.fits[LUCKY]";
-    filename.default_extname("EVENTS");
-    test_assert(filename.extname() == "LUCKY",
+    test_assert(filename.extname("EVENTS") == "LUCKY",
                 "Expected \"LUCKY\" extension name, "
                 "found \""+filename.extname()+"\"");