diff --git a/ChangeLog b/ChangeLog index 4dd7e079db..e98a60b406 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,4 +1,7 @@ -Mon Oct 10 12:37:06 2016 Nobuyoshi Nakada +Mon Oct 10 12:40:04 2016 Nobuyoshi Nakada + + * ruby.c (open_load_file): open in binary mode if available, as + parser deals with EOLs. * io.c (prep_io): reduce isatty call (and its system call) on Cygwin. diff --git a/ruby.c b/ruby.c index 0e7dcc41b8..5f4ee53c92 100644 --- a/ruby.c +++ b/ruby.c @@ -1877,18 +1877,21 @@ open_load_file(VALUE fname_v, int *xflag) use O_NONBLOCK. */ #if defined O_NONBLOCK && HAVE_FCNTL && !(O_NONBLOCK & O_ACCMODE) /* TODO: fix conflicting O_NONBLOCK in ruby/win32.h */ -# define MODE_TO_LOAD (O_RDONLY | O_NONBLOCK) +# define MODE_TO_LOAD (O_NONBLOCK) #elif defined O_NDELAY && HAVE_FCNTL && !(O_NDELAY & O_ACCMODE) -# define MODE_TO_LOAD (O_RDONLY | O_NDELAY) +# define MODE_TO_LOAD (O_NDELAY) #else -# define MODE_TO_LOAD (O_RDONLY) +# define MODE_TO_LOAD (0) #endif - int mode = MODE_TO_LOAD; + int mode = O_RDONLY | +#ifdef O_BINARY + O_BINARY | +#endif + MODE_TO_LOAD; #if defined DOSISH || defined __CYGWIN__ { const char *ext = strrchr(fname, '.'); if (ext && STRCASECMP(ext, ".exe") == 0) { - mode |= O_BINARY; *xflag = 1; } } @@ -1899,7 +1902,7 @@ open_load_file(VALUE fname_v, int *xflag) } rb_update_max_fd(fd); -#if defined HAVE_FCNTL && MODE_TO_LOAD != O_RDONLY +#if defined HAVE_FCNTL && MODE_TO_LOAD /* disabling O_NONBLOCK */ if (fcntl(fd, F_SETFL, 0) < 0) { e = errno;