wyhaines	2010-06-23 05:10:13 +0900 (Wed, 23 Jun 2010)

  New Revision: 28393

  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=28393

  Log:
    io.c: Backport #2267 [ruby-core:26300]; Fix problem with IO so that the file position is correct after a direct write on BSDish platforms like OS X.
    test/ruby/test_io.rb: Added a test for the fix.

  Modified files:
    branches/ruby_1_8_6/ChangeLog
    branches/ruby_1_8_6/io.c
    branches/ruby_1_8_6/test/ruby/test_io.rb
    branches/ruby_1_8_6/version.h

Index: ruby_1_8_6/ChangeLog
===================================================================
--- ruby_1_8_6/ChangeLog	(revision 28392)
+++ ruby_1_8_6/ChangeLog	(revision 28393)
@@ -1,6 +1,11 @@
+Wed Jun 23 04:26:00 Kirk Haines <khaines@r...>
+
+	* io.c: Backport #2267 [ruby-core:26300]; Fix problem with IO so that the file position is correct after a direct write on BSDish platforms like OS X.
+	* test/ruby/test_io.rb: Added a test for the fix.
+
 Wed Jun 23 02:07:00 Kirk Haines <khaines@r...>
 
-	* object.c: Backport #2364 [ruby-core:26733]; Allow result of to_f to be NaN to permit conversion from BigDecimal('NaN') to Float. 
+	* object.c: Backport #2364 [ruby-core:26733]; Allow result of to_f to be NaN to permit conversion from BigDecimal('NaN') to Float. r28392
 
 Tue Jun 22 04:29:00 Kirk Haines <khaines@r...>
 
Index: ruby_1_8_6/version.h
===================================================================
--- ruby_1_8_6/version.h	(revision 28392)
+++ ruby_1_8_6/version.h	(revision 28393)
@@ -2,7 +2,7 @@
 #define RUBY_RELEASE_DATE "2010-06-23"
 #define RUBY_VERSION_CODE 186
 #define RUBY_RELEASE_CODE 20100623
-#define RUBY_PATCHLEVEL 417
+#define RUBY_PATCHLEVEL 418
 
 #define RUBY_VERSION_MAJOR 1
 #define RUBY_VERSION_MINOR 8
Index: ruby_1_8_6/io.c
===================================================================
--- ruby_1_8_6/io.c	(revision 28392)
+++ ruby_1_8_6/io.c	(revision 28393)
@@ -36,6 +36,14 @@
 # define USE_SETVBUF
 #endif
 
+#ifndef BSD_STDIO
+# if defined(__MACH__) || defined(__DARWIN__) || defined(__FreeBSD__) || defined(__NetBSD__) || defined(__OpenBSD__) || defined(__bsdi__)
+#   define BSD_STDIO 1
+# else
+#   define BSD_STDIO 0
+# endif
+#endif
+
 #ifdef __QNXNTO__
 #include "unix.h"
 #endif
@@ -472,6 +480,9 @@
         TRAP_BEG;
 	r = write(fileno(f), RSTRING(str)->ptr+offset, l);
         TRAP_END;
+#if BSD_STDIO
+	fseeko(f, lseek(fileno(f), (off_t)0, SEEK_CUR), SEEK_SET);
+#endif
         if (r == n) return len;
         if (0 <= r) {
             offset += r;
Index: ruby_1_8_6/test/ruby/test_io.rb
===================================================================
--- ruby_1_8_6/test/ruby/test_io.rb	(revision 28392)
+++ ruby_1_8_6/test/ruby/test_io.rb	(revision 28393)
@@ -1,6 +1,33 @@
 require 'test/unit'
+require 'tempfile'
 
 class TestIO < Test::Unit::TestCase
+
+  def make_tempfile
+    t = Tempfile.new("foo")
+    t.binmode
+    t.puts "foo"
+    t.puts "bar"
+    t.puts "baz"
+    t.close
+    t
+  end
+
+  def test_pos
+    t = make_tempfile
+
+    open(t.path, IO::RDWR|IO::CREAT|IO::TRUNC, 0600) do |f|
+      f.write "Hello"
+      assert_equal(5, f.pos)
+    end
+    open(t.path, IO::RDWR|IO::CREAT|IO::TRUNC, 0600) do |f|
+      f.sync = true
+      f.read
+      f.write "Hello"
+      assert_equal(5, f.pos)
+    end
+  end
+
   def test_gets_rs
     r, w = IO.pipe
     w.print "\377xyz"

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/