Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Perl: porters

[perl #70337] perldiag: localtime(...) too large

 

 

Perl porters RSS feed   Index | Next | Previous | View Threaded


perlbug-followup at perl

Nov 8, 2009, 2:04 PM

Post #1 of 2 (251 views)
Permalink
[perl #70337] perldiag: localtime(...) too large

# New Ticket Created by Father Chrysostomos
# Please include the string: [perl #70337]
# in the subject line of all future correspondence about this issue.
# <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=70337 >


The message ‘localtime(...) too large’ is not documented in perldiag.
The attached patch adds it. I know it’s technically ‘%s(%.0f) too
large’, but it’s easier for users to find it under ‘localtime’ and
‘gmtime’. I also had to modify diagnostics.pm to support %.0f, and I
added a test, but I used a hack with STDERR that would make debugging
very difficult to anyone else adding tests. Should this script perhaps
be ported to test.pl, so we can use fresh_perl_like?

BTW, localtime(nan) produces that message, but inf and -inf do not.
Shouldn’t they? All three values produce the same result, which is a
time in the very distant past. They used to return the same value as
0. Was this supposed to change? I don’t think this really matters, but
shouldn’t the warning extend to (-)inf?

Use of uninitialized value $category in concatenation (.) or string
at /usr/local/bin/perlbug5.11.1 line 645.
Use of uninitialized value $severity in concatenation (.) or string
at /usr/local/bin/perlbug5.11.1 line 645.
---
Flags:
category=
severity=
---
Site configuration information for perl 5.11.1:

Configured by sprout at Mon Nov 2 20:26:38 PST 2009.

Summary of my perl5 (revision 5 version 11 subversion 1 patch
v5.11.1-107-g2b67659) configuration:
Snapshot of: 2b6765935f5ee68d8093e686b8e292ad5de5a898
Platform:
osname=darwin, osvers=10.0.0, archname=darwin-2level
uname='darwin pint.local 10.0.0 darwin kernel version 10.0.0: fri
jul 31 22:47:34 pdt 2009; root:xnu-1456.1.25~1release_i386 i386 '
config_args='-de -Dusedevel'
hint=previous, useposix=true, d_sigaction=define
useithreads=undef, usemultiplicity=undef
useperlio=define, d_sfio=undef, uselargefiles=define,
usesocks=undef
use64bitint=undef, use64bitall=undef, uselongdouble=undef
usemymalloc=n, bincompat5005=undef
Compiler:
cc='cc', ccflags ='-fno-common -DPERL_DARWIN -no-cpp-precomp -fno-
strict-aliasing -pipe -fstack-protector -I/usr/local/include',
optimize='-O3',
cppflags='-no-cpp-precomp -fno-common -DPERL_DARWIN -no-cpp-
precomp -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/
include -fno-common -DPERL_DARWIN -no-cpp-precomp -fno-strict-aliasing
-pipe -fstack-protector -I/usr/local/include'
ccversion='', gccversion='4.2.1 (Apple Inc. build 5646)',
gccosandvers=''
intsize=4, longsize=4, ptrsize=4, doublesize=8, byteorder=1234
d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=16
ivtype='long', ivsize=4, nvtype='double', nvsize=8,
Off_t='off_t', lseeksize=8
alignbytes=8, prototype=define
Linker and Libraries:
ld='env MACOSX_DEPLOYMENT_TARGET=10.3 cc', ldflags =' -fstack-
protector -L/usr/local/lib'
libpth=/usr/local/lib /usr/lib
libs=-ldbm -ldl -lm -lutil -lc
perllibs=-ldl -lm -lutil -lc
libc=/usr/lib/libc.dylib, so=dylib, useshrplib=false,
libperl=libperl.a
gnulibc_version=''
Dynamic Linking:
dlsrc=dl_dlopen.xs, dlext=bundle, d_dlsymun=undef, ccdlflags=' '
cccdlflags=' ', lddlflags=' -bundle -undefined dynamic_lookup -L/
usr/local/lib -fstack-protector'

Locally applied patches:


---
@INC for perl 5.11.1:
/usr/local/lib/perl5/site_perl/5.11.1/darwin-2level
/usr/local/lib/perl5/site_perl/5.11.1
/usr/local/lib/perl5/5.11.1/darwin-2level
/usr/local/lib/perl5/5.11.1
/usr/local/lib/perl5/site_perl
.

---
Environment for perl 5.11.1:
DYLD_LIBRARY_PATH (unset)
HOME=/Users/sprout
LANG=en_US.UTF-8
LANGUAGE (unset)
LD_LIBRARY_PATH (unset)
LOGDIR (unset)
PATH=/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin:/usr/X11/bin:/
usr/local/bin
PERL_BADLANG (unset)
SHELL=/bin/bash
Attachments: open_JCOKxPHS.txt (3.02 KB)


rgs at consttype

Nov 13, 2009, 1:22 AM

Post #2 of 2 (200 views)
Permalink
Re: [perl #70337] perldiag: localtime(...) too large [In reply to]

2009/11/8 Father Chrysostomos <perlbug-followup [at] perl>:
> The message ‘localtime(...) too large’ is not documented in perldiag.
> The attached patch adds it. I know it’s technically ‘%s(%.0f) too
> large’, but it’s easier for users to find it under ‘localtime’ and
> ‘gmtime’. I also had to modify diagnostics.pm to support %.0f, and I
> added a test, but I used a hack with STDERR that would make debugging
> very difficult to anyone else adding tests. Should this script perhaps
> be ported to test.pl, so we can use fresh_perl_like?
>
> BTW, localtime(nan) produces that message, but inf and -inf do not.

For me with a debugging blead, they all produce an assertion failure
(in addition of the warning for NaN) :

$ bleadperl -wMstrict -E 'say for localtime("Inf")'
perl: time64.c:172: S_check_tm: Assertion `tm->tm_sec >= 0' failed.
Aborted

> Shouldn’t they? All three values produce the same result, which is a
> time in the very distant past. They used to return the same value as
> 0. Was this supposed to change? I don’t think this really matters, but
> shouldn’t the warning extend to (-)inf?

Maybe +Inf should produce 2012 ? or something ? maybe just under Mayan locales ?

Thanks, patch applied as 8b56d6ffd0fe641abae5662a8e75424a1d9e4dce to bleadperl.

Perl porters RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.