changeset 30081:5ea53078eb8b

maint: use "m_" prefix for member variables in file 'file-stat.h'. * file-stat.cc, file-stat.h: use "m_" prefix for member variables in classes base_file_stat, file_stat, file_fstat.
author Rik <rik@octave.org>
date Mon, 30 Aug 2021 12:13:27 -0700
parents dbbf3d44535d
children 1788b9f6db6a
files liboctave/system/file-stat.cc liboctave/system/file-stat.h
diffstat 2 files changed, 49 insertions(+), 49 deletions(-) [+]
line wrap: on
line diff
--- a/liboctave/system/file-stat.cc	Mon Aug 30 12:03:27 2021 -0700
+++ b/liboctave/system/file-stat.cc	Mon Aug 30 12:13:27 2021 -0700
@@ -176,9 +176,9 @@
     // Private stuff:
 
     file_stat::file_stat (const std::string& n, bool fl)
-      : base_file_stat (), file_name (n), follow_links (fl)
+      : base_file_stat (), m_file_name (n), m_follow_links (fl)
     {
-      if (! file_name.empty ())
+      if (! m_file_name.empty ())
         update_internal ();
     }
 
@@ -187,12 +187,12 @@
     void
     file_stat::update_internal (bool force)
     {
-      if (! initialized || force)
+      if (! m_initialized || force)
         {
-          initialized = false;
-          fail = false;
+          m_initialized = false;
+          m_fail = false;
 
-          std::string full_file_name = sys::file_ops::tilde_expand (file_name);
+          std::string full_file_name = sys::file_ops::tilde_expand (m_file_name);
 
 #if defined (OCTAVE_USE_WINDOWS_API)
           full_file_name = sys::env::make_absolute (full_file_name);
@@ -218,7 +218,7 @@
           time_t sys_atime, sys_mtime, sys_ctime;
 
           int status
-            = (follow_links
+            = (m_follow_links
                ? octave_stat_wrapper (cname, &m_mode, &m_ino, &m_dev,
                                       &m_nlink, &m_uid, &m_gid, &m_size,
                                       &sys_atime, &sys_mtime, &sys_ctime,
@@ -230,8 +230,8 @@
 
           if (status < 0)
             {
-              fail = true;
-              errmsg = std::strerror (errno);
+              m_fail = true;
+              m_errmsg = std::strerror (errno);
             }
           else
             {
@@ -240,30 +240,30 @@
               m_ctime = sys::time (sys_ctime);
             }
 
-          initialized = true;
+          m_initialized = true;
         }
     }
 
     void
     file_fstat::update_internal (bool force)
     {
-      if (! initialized || force)
+      if (! m_initialized || force)
         {
-          initialized = false;
-          fail = false;
+          m_initialized = false;
+          m_fail = false;
 
           time_t sys_atime, sys_mtime, sys_ctime;
 
           int status
-            = octave_fstat_wrapper (fid, &m_mode, &m_ino, &m_dev,
+            = octave_fstat_wrapper (m_fid, &m_mode, &m_ino, &m_dev,
                                     &m_nlink, &m_uid, &m_gid, &m_size,
                                     &sys_atime, &sys_mtime, &sys_ctime,
                                     &m_rdev, &m_blksize, &m_blocks);
 
           if (status < 0)
             {
-              fail = true;
-              errmsg = std::strerror (errno);
+              m_fail = true;
+              m_errmsg = std::strerror (errno);
             }
           else
             {
@@ -272,7 +272,7 @@
               m_ctime = sys::time (sys_ctime);
             }
 
-          initialized = true;
+          m_initialized = true;
         }
     }
   }
--- a/liboctave/system/file-stat.h	Mon Aug 30 12:03:27 2021 -0700
+++ b/liboctave/system/file-stat.h	Mon Aug 30 12:13:27 2021 -0700
@@ -45,26 +45,26 @@
     public:
 
       base_file_stat (void)
-        : initialized (false), fail (false), errmsg (), m_mode (),
+        : m_initialized (false), m_fail (false), m_errmsg (), m_mode (),
           m_ino (), m_dev (), m_nlink (), m_uid (), m_gid (),
           m_size (), m_atime (), m_mtime (), m_ctime (), m_rdev (),
           m_blksize (), m_blocks () { }
 
       base_file_stat (const base_file_stat& fs)
-        : initialized (fs.initialized), fail (fs.fail), errmsg (fs.errmsg),
-          m_mode (fs.m_mode), m_ino (fs.m_ino), m_dev (fs.m_dev),
-          m_nlink (fs.m_nlink), m_uid (fs.m_uid), m_gid (fs.m_gid),
-          m_size (fs.m_size), m_atime (fs.m_atime), m_mtime (fs.m_mtime),
-          m_ctime (fs.m_ctime), m_rdev (fs.m_rdev),
+        : m_initialized (fs.m_initialized), m_fail (fs.m_fail),
+          m_errmsg (fs.m_errmsg), m_mode (fs.m_mode), m_ino (fs.m_ino),
+          m_dev (fs.m_dev), m_nlink (fs.m_nlink), m_uid (fs.m_uid),
+          m_gid (fs.m_gid), m_size (fs.m_size), m_atime (fs.m_atime),
+          m_mtime (fs.m_mtime), m_ctime (fs.m_ctime), m_rdev (fs.m_rdev),
           m_blksize (fs.m_blksize), m_blocks (fs.m_blocks) { }
 
       base_file_stat& operator = (const base_file_stat& fs)
       {
         if (this != &fs)
           {
-            initialized = fs.initialized;
-            fail = fs.fail;
-            errmsg = fs.errmsg;
+            m_initialized = fs.m_initialized;
+            m_fail = fs.m_fail;
+            m_errmsg = fs.m_errmsg;
             m_mode = fs.m_mode;
             m_ino = fs.m_ino;
             m_dev = fs.m_dev;
@@ -140,13 +140,13 @@
 
       std::string mode_as_string (void) const;
 
-      bool ok (void) const { return initialized && ! fail; }
+      bool ok (void) const { return m_initialized && ! m_fail; }
 
       operator bool () const { return ok (); }
 
       bool exists (void) const { return ok (); }
 
-      std::string error (void) const { return ok () ? "" : errmsg; }
+      std::string error (void) const { return ok () ? "" : m_errmsg; }
 
       // Has the file referenced by this object been modified since TIME?
       bool is_newer (const sys::time& time) const { return m_mtime > time; }
@@ -160,13 +160,13 @@
       virtual ~base_file_stat (void) = default;
 
       // TRUE means we have already called stat.
-      bool initialized;
+      bool m_initialized;
 
       // TRUE means the stat for this file failed.
-      bool fail;
+      bool m_fail;
 
       // If a failure occurs, this contains the system error text.
-      std::string errmsg;
+      std::string m_errmsg;
 
       // file type and permissions
       mode_t m_mode;
@@ -219,8 +219,8 @@
       file_stat (const std::string& n = "", bool fl = true);
 
       file_stat (const file_stat& fs)
-        : base_file_stat (fs), file_name (fs.file_name),
-          follow_links (fs.follow_links) { }
+        : base_file_stat (fs), m_file_name (fs.m_file_name),
+          m_follow_links (fs.m_follow_links) { }
 
       file_stat& operator = (const file_stat& fs)
       {
@@ -228,8 +228,8 @@
           {
             base_file_stat::operator = (fs);
 
-            file_name = fs.file_name;
-            follow_links = fs.follow_links;
+            m_file_name = fs.m_file_name;
+            m_follow_links = fs.m_follow_links;
           }
 
         return *this;
@@ -241,17 +241,17 @@
 
       void get_stats (bool force = false)
       {
-        if (! initialized || force)
+        if (! m_initialized || force)
           update_internal (force);
       }
 
       void get_stats (const std::string& n, bool force = false)
       {
-        if (n != file_name || ! initialized || force)
+        if (n != m_file_name || ! m_initialized || force)
           {
-            initialized = false;
+            m_initialized = false;
 
-            file_name = n;
+            m_file_name = n;
 
             update_internal (force);
           }
@@ -260,11 +260,11 @@
     private:
 
       // Name of the file.
-      std::string file_name;
+      std::string m_file_name;
 
       // TRUE means follow symbolic links to the ultimate file (stat).
       // FALSE means get information about the link itself (lstat).
-      bool follow_links;
+      bool m_follow_links;
 
       void update_internal (bool force = false);
     };
@@ -275,13 +275,13 @@
     {
     public:
 
-      file_fstat (int n) : base_file_stat (), fid (n)
+      file_fstat (int n) : base_file_stat (), m_fid (n)
       {
         update_internal ();
       }
 
       file_fstat (const file_fstat& fs)
-        : base_file_stat (fs), fid (fs.fid) { }
+        : base_file_stat (fs), m_fid (fs.m_fid) { }
 
       file_fstat& operator = (const file_fstat& fs)
       {
@@ -289,7 +289,7 @@
           {
             base_file_stat::operator = (fs);
 
-            fid = fs.fid;
+            m_fid = fs.m_fid;
           }
 
         return *this;
@@ -299,17 +299,17 @@
 
       void get_stats (bool force = false)
       {
-        if (! initialized || force)
+        if (! m_initialized || force)
           update_internal (force);
       }
 
       void get_stats (int n, bool force = false)
       {
-        if (n != fid || ! initialized || force)
+        if (n != m_fid || ! m_initialized || force)
           {
-            initialized = false;
+            m_initialized = false;
 
-            fid = n;
+            m_fid = n;
 
             update_internal (force);
           }
@@ -318,7 +318,7 @@
     private:
 
       // Open file descriptor.
-      int fid;
+      int m_fid;
 
       void update_internal (bool force = false);
     };