Remove deep_copy hack (SSHKit no longer needs it)

mattbrictson · May 6, 2015 · 66271b1 · 66271b1
1 parent 658f880
commit 66271b1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,7 @@
 ## Next release
 
 * Your contribution here!
+* Fix `Marshal.dump` warnings by removing `deep_copy` workaround that it is no longer needed for the latest SSHKit ([#10](https://github.com/mattbrictson/airbrussh/issues/10)).
 
 ## 0.4.0 (2015-05-03)
 

diff --git a/lib/airbrussh/formatter.rb b/lib/airbrussh/formatter.rb
@@ -80,8 +80,8 @@ def write_log_file_delimiter
 
     def write(obj)
       # SSHKit's :pretty formatter mutates the stdout and stderr data in the
-      # command obj. So we need to clone it to ensure our copy is unscathed.
-      @log_file_formatter << deep_copy(obj)
+      # command obj. So we need to dup it to ensure our copy is unscathed.
+      @log_file_formatter << obj.dup
 
       case obj
       when SSHKit::Command    then write_command(obj)
@@ -225,9 +225,5 @@ def clock
     def config
       Airbrussh.configuration
     end
-
-    def deep_copy(obj)
-      Marshal.load(Marshal.dump(obj))
-    end
   end
 end