develooper Front page | perl.perl5.porters | Postings from June 2008

[perl #55420] No warnings when reading from a file only opened for appending.

Thread Next
Paul Fenwick
June 7, 2008 01:47
[perl #55420] No warnings when reading from a file only opened for appending.
Message ID:
# New Ticket Created by  Paul Fenwick 
# Please include the string:  [perl #55420]
# in the subject line of all future correspondence about this issue. 
# <URL: >

This is a bug report for perl from,
generated with the help of perlbug 1.35 running under perl v5.8.8.

[Please enter your report here]

G'day p5p and readers of perlbugs,

Under all versions of Perl I have on my laptop (5.8.4, 5.8.8, 5.10.0, and
5.11.0) the following produces a warning:

	open(FOO, '>', 'foo.txt');

However writing a file for appending doesn't:

	open(FOO, '>>', 'foo.txt');

I thought this would be easy enough to fix:

diff --git a/pp_hot.c b/pp_hot.c
index 64b5fc5..bc0d5fe 100644
--- a/pp_hot.c
+++ b/pp_hot.c
@@ -1586,7 +1586,8 @@ Perl_do_readline(pTHX)
         else if (type == OP_GLOB)
-       else if (ckWARN(WARN_IO) && IoTYPE(io) == IoTYPE_WRONLY) {
+       else if (ckWARN(WARN_IO) && ( IoTYPE(io) == IoTYPE_WRONLY ||
+                                     IoTYPE(io) == IoTYPE_APPEND)) {
             report_evil_fh(PL_last_in_gv, io, OP_phoney_OUTPUT_ONLY);

However it looks like IoTYPE_APPEND also applies to files opened with mode
'+>>', and we certainly don't want to be warning there.

Am I missing an IoTYPE that signifies a file is open for appending
only, or is the reason this bug is still around because we can't
easily spot the difference between the two?

Many thanks,


Paul Fenwick <> |
Director of Training                   | Ph:  +61 3 9354 6001
Perl Training Australia                | Fax: +61 3 9354 2681

[Please do not change anything below this line]
Site configuration information for perl v5.8.8:

Configured by rurban at Sun Jul  8 19:08:44 GMT 2007.

Summary of my perl5 (revision 5 version 8 subversion 8) configuration:
     osname=cygwin, osvers=1.5.24(0.15642), archname=cygwin-thread-multi-64int
     uname='cygwin_nt-5.1 reini 1.5.24(0.15642) 2007-01-31 10:57 i686 cygwin '
     config_args='-de -Dmksymlinks -Duse64bitint -Dusethreads -Uusemymalloc
-Doptimize=-O3 -Dman3ext=3pm -Dusesitecustomize -Dusedevel'
     hint=recommended, useposix=true, d_sigaction=define
     usethreads=define use5005threads=undef useithreads=define
     useperlio=define d_sfio=undef uselargefiles=define usesocks=undef
     use64bitint=define use64bitall=undef uselongdouble=undef
     usemymalloc=n, bincompat5005=undef
     cc='gcc', ccflags ='-DPERL_USE_SAFE_PUTENV -fno-strict-aliasing -pipe
     cppflags='-DPERL_USE_SAFE_PUTENV -fno-strict-aliasing -pipe
     ccversion='', gccversion='3.4.4 (cygming special, gdc 0.12, using dmd
0.125)', gccosandvers=''
     intsize=4, longsize=4, ptrsize=4, doublesize=8, byteorder=12345678
     d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=12
     ivtype='long long', ivsize=8, nvtype='double', nvsize=8, Off_t='off_t',
     alignbytes=8, prototype=define
   Linker and Libraries:
     ld='ld2', ldflags =' -s -L/usr/local/lib'
     libpth=/usr/local/lib /usr/lib /lib
     libs=-lgdbm -ldb -ldl -lcrypt -lgdbm_compat
     perllibs=-ldl -lcrypt -lgdbm_compat
     libc=/usr/lib/libc.a, so=dll, useshrplib=true, libperl=libperl.a
   Dynamic Linking:
     dlsrc=dl_dlopen.xs, dlext=dll, d_dlsymun=undef, ccdlflags=' -s'
     cccdlflags=' ', lddlflags=' -s -L/usr/local/lib'

Locally applied patches:
     CYG01 - ldflags -s
     CYG02 - lib-ExtUtils-Embed insensitive against leading \\s
     CYG03 - lib-Test-Harness-Straps $ENV{PERL5LIB} = ''
     CYG04 - cygperl-5_8.dll and not cygperl-5_8_x.dll
     CYG05 - add Win32CORE to core
     CYG07 - File-Spec-Cygwin-TMPDIR.patch
     Bug#38628 - allow legacy Cwd->cwd()
     Bug#40103 - File-Spec-case_tolerant.patch from 5.9.5

@INC for perl v5.8.8:

Environment for perl v5.8.8:
     LANG (unset)
     LANGUAGE (unset)
     LD_LIBRARY_PATH (unset)
     LOGDIR (unset)

Files/VDMSound/:/cygdrive/c/Program Files/Common
     PERL_BADLANG (unset)
     SHELL (unset)

Thread Next Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at | Group listing | About