On Friday, 17 August 2007 at 01:06, Dave Jones wrote: > On Thu, Aug 16, 2007 at 04:36:37PM -0400, Dave Jones wrote: > > On Thu, Aug 16, 2007 at 04:21:38PM -0400, Dave Jones wrote: > > > On Thu, Aug 16, 2007 at 05:08:15PM +0200, Oliver Falk wrote: > > > > > > > If you compile the whole Fedora tree, how many warnings will you see? > > > > > > So I grepped across a make prep'd tree of devel (all 63 gig of it). > > > Here's the fallout.. > > > > > > openmpi/openmpi-1.2.3/orte/runtime/orte_abort.c fd = open(abort_file, O_CREAT); > > > jython/jython-svn-Release_2_2beta1/CPythonLib/test/test_unicode_file.py:f = os.open(TESTFN_ENCODED, os.O_CREAT) > > > perl/perl-5.8.8/t/op/taint.t: eval { sysopen(my $cr, $evil, &O_CREAT) }; > > > proftpd/proftpd-1.3.0a/contrib/mod_rewrite.c: if ((fifo_lockfd = open(fifo_lockname, O_CREAT)) < 0) > > > pwlib/pwlib-1.10.7/configure:sem_t *s = sem_open("test", O_CREAT) > > > pwlib/pwlib-1.10.7/configure.ac: [sem_t *s = sem_open("test", O_CREAT)], > > > python/Python-2.5.1/Lib/test/test_unicode_file.py: f = os.open(filename, os.O_CREAT) > > > python-docs/Python-2.5.1/Lib/test/test_unicode_file.py: f = os.open(filename, os.O_CREAT) > > > xca/xca-0.6.3/_tmp_root/usr/lib/python2.5/test/test_unicode_file.py: f = os.open(filename, os.O_CREAT) > > > > > > Not too bad considering. > > > > Bah, my regexp was imperfect. I'll fix up and rerun. > > More like it. 267 hits. > > Dave > [...] > Checking ./dx/dx-4.4.4/src/exec/libdx/fileio.c > Found open with O_CREAT and no mode. > fd = open(name, O_WRONLY | O_CREAT); I'll patch this up and send fix upstream. Thanks! Regards, R. -- Fedora contributor http://fedoraproject.org/wiki/DominikMierzejewski Livna contributor http://rpm.livna.org MPlayer developer http://mplayerhq.hu "Faith manages." -- Delenn to Lennier in Babylon 5:"Confessions and Lamentations" -- Fedora-maintainers mailing list Fedora-maintainers@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/fedora-maintainers -- Fedora-maintainers-readonly mailing list Fedora-maintainers-readonly@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/fedora-maintainers-readonly