Make io_data_t pure virtual, its constructor protected

1 files changed, 11 insertions, 12 deletions

diff --git a/io.h b/io.h
index 5f3c1a67..a50c54e5 100644
--- a/io.h
+++ b/io.h
@@ -10,7 +10,7 @@ using std::tr1::shared_ptr;
 */
 enum io_mode_t
 {
-    IO_INVALID, IO_FILE, IO_PIPE, IO_FD, IO_BUFFER, IO_CLOSE
+    IO_FILE, IO_PIPE, IO_FD, IO_BUFFER, IO_CLOSE
 };
 
 /** Represents an FD redirection */
@@ -21,27 +21,26 @@ private:
     io_data_t(const io_data_t &rhs);
     void operator=(const io_data_t &rhs);
 
+protected:
+    io_data_t(io_mode_t m, int f) :
+        io_mode(m),
+        fd(f),
+        is_input(0)
+    {
+    }
+
 public:
     /** Type of redirect */
     io_mode_t io_mode;
     /** FD to redirect */
     int fd;
 
-    virtual void print() const;
+    virtual void print() const = 0;
 
     /** Set to true if this is an input io redirection */
     bool is_input;
 
-    io_data_t(io_mode_t m = IO_INVALID, int f=0) :
-        io_mode(m),
-        fd(f),
-        is_input(0)
-    {
-    }
-
-    virtual ~io_data_t()
-    {
-    }
+    virtual ~io_data_t() = 0;
 };
 
 class io_close_t : public io_data_t