[前][次][番号順一覧][スレッド一覧]

ruby-changes:21721

From: usa <ko1@a...>
Date: Wed, 16 Nov 2011 11:36:18 +0900 (JST)
Subject: [ruby-changes:21721] usa:r33770 (trunk): * io.c (argf_next_argv): wrong timing of setting ecflags.

usa	2011-11-16 11:36:08 +0900 (Wed, 16 Nov 2011)

  New Revision: 33770

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

  Log:
    * io.c (argf_next_argv): wrong timing of setting ecflags.
      fixed the failure of TestArgf#test_textmode introduced at r33662.

  Modified files:
    trunk/ChangeLog
    trunk/io.c

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 33769)
+++ ChangeLog	(revision 33770)
@@ -1,3 +1,8 @@
+Wed Nov 16 11:34:20 2011  NAKAMURA Usaku  <usa@r...>
+
+	* io.c (argf_next_argv): wrong timing of setting ecflags.
+	  fixed the failure of TestArgf#test_textmode introduced at r33662.
+
 Wed Nov 16 10:45:00 2011  NAKAMURA Usaku  <usa@r...>
 
 	* ext/-test-/num2int/num2int.c: remove an unnecessary and wrong decl
Index: io.c
===================================================================
--- io.c	(revision 33769)
+++ io.c	(revision 33770)
@@ -7137,9 +7137,6 @@
 		fmode = FMODE_READABLE;
 		if (!ARGF.binmode) {
 		    fmode |= DEFAULT_TEXTMODE;
-#ifdef TEXTMODE_NEWLINE_DECORATOR_ON_WRITE
-		    fptr->encs.ecflags |= TEXTMODE_NEWLINE_DECORATOR_ON_WRITE;
-#endif
 		}
 		ARGF.current_file = prep_io(fr, fmode, rb_cFile, fn);
 		if (!NIL_P(write_io)) {
@@ -7156,6 +7153,9 @@
 		fptr->encs.ecflags &= ~ECONV_NEWLINE_DECORATOR_MASK;
 		if (!ARGF.binmode) {
 		    fptr->encs.ecflags |= ECONV_DEFAULT_NEWLINE_DECORATOR;
+#ifdef TEXTMODE_NEWLINE_DECORATOR_ON_WRITE
+		    fptr->encs.ecflags |= TEXTMODE_NEWLINE_DECORATOR_ON_WRITE;
+#endif
 		}
 	    }
 	    ARGF.next_p = 0;

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

[前][次][番号順一覧][スレッド一覧]