Skip Menu |
 

This queue is for tickets about the Math-BigInt CPAN distribution.

Report information
The Basics
Id: 43694
Status: resolved
Priority: 0/
Queue: Math-BigInt

People
Owner: Nobody in particular
Requestors: emazep [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 1.89
Fixed in: (no value)



Subject: The as_int() method on BigFloat objects loses precision on integers
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Charset: utf8
X-RT-Original-Encoding: utf-8
Content-Type: multipart/mixed; boundary="----------=_1235727198-31384-197"
Content-Length: 0
Content-Type: text/plain
Content-Disposition: inline
Content-Transfer-Encoding: binary
Content-Length: 264
Download (untitled) / with headers
text/plain 264b
The as_int() method on BigFloat objects loses precision even on integer values, while it shouldn't: perl -MMath::BigFloat -e 'print Math::BigFloat->new('1234567890987654321')->as_int, "\n"' # 1234567890987650000 Thanks for your invaluable work, Emanuele Zeppieri
Subject: perl-V.txt
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="----------=_1235727030-31384-196"
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Charset: utf8
Content-Length: 0
Content-Type: text/plain
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: iso-8859-1
Content-Length: 0
Content-Type: text/plain; charset="utf-8"; name="perl-V.txt"
Content-Disposition: inline; filename="perl-V.txt"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: ascii
Content-Length: 2758
Download perl-V.txt
text/plain 2.6k
Summary of my perl5 (revision 5 version 10 subversion 0 patch 34065) configuration: Platform: osname=cygwin, osvers=1.5.25(0.15642), archname=cygwin-thread-multi-64int uname='cygwin_nt-5.1 reini 1.5.25(0.15642) 2008-06-12 19:34 i686 cygwin ' config_args='-de -Dmksymlinks -Dusethreads -Dmad=y -Dusedevel' hint=recommended, useposix=true, d_sigaction=define useithreads=define, usemultiplicity=define useperlio=define, d_sfio=undef, uselargefiles=define, usesocks=undef use64bitint=define, use64bitall=undef, uselongdouble=undef usemymalloc=y, bincompat5005=undef Compiler: cc='gcc', ccflags ='-DPERL_USE_SAFE_PUTENV -U__STRICT_ANSI__ -fno-strict-aliasing -pipe -I/usr/local/include', optimize='-O3', cppflags='-DPERL_USE_SAFE_PUTENV -U__STRICT_ANSI__ -fno-strict-aliasing -pipe -I/usr/local/include' 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', lseeksize=8 alignbytes=8, prototype=define Linker and Libraries: ld='g++', ldflags =' -Wl,--enable-auto-import -Wl,--export-all-symbols -Wl,--stack,8388608 -Wl,--enable-auto-image-base -L/usr/local/lib' libpth=/usr/local/lib /usr/lib /lib libs=-lgdbm -ldb -ldl -lcrypt -lgdbm_compat perllibs=-ldl -lcrypt libc=/usr/lib/libc.a, so=dll, useshrplib=true, libperl=libperl.a gnulibc_version='' Dynamic Linking: dlsrc=dl_dlopen.xs, dlext=dll, d_dlsymun=undef, ccdlflags=' ' cccdlflags=' ', lddlflags=' --shared -Wl,--enable-auto-import -Wl,--export-all-symbols -Wl,--stack,8388608 -Wl,--enable-auto-image-base -L/usr/local/lib' Characteristics of this binary (from libperl): Compile-time options: MULTIPLICITY MYMALLOC PERL_DONT_CREATE_GVSV PERL_IMPLICIT_CONTEXT PERL_MAD PERL_MALLOC_WRAP PERL_USE_SAFE_PUTENV USE_64_BIT_INT USE_ITHREADS USE_LARGE_FILES USE_PERLIO USE_REENTRANT_API Locally applied patches: MAINT34065 CYG11 no-bs CYG12 no archlib in otherlibdirs CYG14 Dynaloader CYG15 static-Win32CORE Bug#55162 File::Spec::case_tolerant performance Built under cygwin Compiled at Jun 30 2008 16:05:15 %ENV: CYGWIN="" @INC: /usr/lib/perl5/5.10/i686-cygwin /usr/lib/perl5/5.10 /usr/lib/perl5/site_perl/5.10/i686-cygwin /usr/lib/perl5/site_perl/5.10 /usr/lib/perl5/vendor_perl/5.10/i686-cygwin /usr/lib/perl5/vendor_perl/5.10 /usr/lib/perl5/vendor_perl/5.10 /usr/lib/perl5/site_perl/5.8 /usr/lib/perl5/vendor_perl/5.8 .
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-2358-1288709167-757.43694-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 1168
Download (untitled) / with headers
text/plain 1.1k
The problem can be traced to the second last line of the as_number() method in Math::BigFloat. The line is $z = Math::BigInt->new( $x->{sign} . $MBI->_num($z)); Before this line $z = [987654321, 234567890, 1] on my computer. It might be different on other computers, depending on $BASE, but it will be something equivalent. So far, so good. Now, $MBI->_num($z) returns the Perl scalar 1.23456789098765e+18, which is then used to construct a Math::BigInt object with the value 1234567890987650000. Clearly, using $MBI->_num($z) here is a mistake. I'm not really sure whether the main problem is in the code or the documentation. The name as_number() suggests that returning a Perl scalar is the right thing, whereas the name as_int() suggests that returning a Math::BigInt (or a Perl scalar integer) is the right thing. This bug is two years(!) old, so it's time to fix it. I don't have a patch yet, but I'll write one. In the meantime, it would be nice if the maintainers did some maintaining. I have written six patches over the last few weeks. So far I have not received any feedback on any of the patches from any of the maintainers. Has maintenance stalled?
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-2358-1288709167-757.43694-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
References: <rt-3.8.HEAD-2358-1288709167-757.43694-0-0 [...] rt.cpan.org>
Content-Type: multipart/mixed; boundary="----------=_1288728519-2357-195"
Message-ID: <rt-3.8.HEAD-2357-1288728519-635.43694-0-0 [...] rt.cpan.org>
X-RT-Original-Encoding: utf-8
Content-Length: 0
Content-Disposition: inline
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 264
Download (untitled) / with headers
text/plain 264b
This bug was introduced in version 1.70. In Math::BigFloat -> as_number() I changed _num() to _str(). The former returns a Perl scalar, but the latter returns a string, preserving all digits. I also added a test to confirms the correct behaviour. Patch included.
MIME-Version: 1.0
Subject: Math-BigInt-1.96-patch-id-43694.txt
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Type: text/plain; charset="utf-8"; name="Math-BigInt-1.96-patch-id-43694.txt"
Content-Disposition: inline; filename="Math-BigInt-1.96-patch-id-43694.txt"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 1354
diff -ur Math-BigInt-1.96-orig/lib/Math/BigFloat.pm Math-BigInt-1.96-patched-id-43694/lib/Math/BigFloat.pm --- Math-BigInt-1.96-orig/lib/Math/BigFloat.pm 2010-09-28 06:34:04.000000000 +0200 +++ Math-BigInt-1.96-patched-id-43694/lib/Math/BigFloat.pm 2010-11-02 21:01:44.171875000 +0100 @@ -3693,7 +3693,7 @@ { $MBI->_lsft( $z, $x->{_e},10); } - $z = Math::BigInt->new( $x->{sign} . $MBI->_num($z)); + $z = Math::BigInt->new( $x->{sign} . $MBI->_str($z)); $z; } diff -ur Math-BigInt-1.96-orig/t/bigfltpm.inc Math-BigInt-1.96-patched-id-43694/t/bigfltpm.inc --- Math-BigInt-1.96-orig/t/bigfltpm.inc 2010-09-13 16:28:42.000000000 +0200 +++ Math-BigInt-1.96-patched-id-43694/t/bigfltpm.inc 2010-11-02 20:26:02.968750000 +0100 @@ -587,6 +587,7 @@ -2:-2 -123.456:-123 -200:-200 +71243225429896467497217836789578596379:71243225429896467497217836789578596379 # test for bug in brsft() not handling cases that return 0 0.000641:0 0.0006412:0 diff -ur Math-BigInt-1.96-orig/t/bigfltpm.t Math-BigInt-1.96-patched-id-43694/t/bigfltpm.t --- Math-BigInt-1.96-orig/t/bigfltpm.t 2010-09-13 16:28:42.000000000 +0200 +++ Math-BigInt-1.96-patched-id-43694/t/bigfltpm.t 2010-11-02 21:02:18.453125000 +0100 @@ -1,7 +1,7 @@ #!/usr/bin/perl -w use strict; -use Test::More tests => 2316 +use Test::More tests => 2317 + 5; # own tests
From jaleto [...] gmail.com Tue Nov 2 16: 28:25 2010
CC: Florian Ragwitz <rafl [...] debian.org>
MIME-Version: 1.0
X-Spam-Status: No, score=-6.28 tagged_above=-99.9 required=10 tests=[AWL=-0.170, BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_HI=-5, SPF_NEUTRAL=0.779, T_TO_NO_BRKTS_FREEMAIL=0.01] autolearn=ham
In-Reply-To: <rt-3.8.HEAD-2358-1288709167-1357.43694-5-0 [...] rt.cpan.org>
X-Spam-Flag: NO
References: <RT-Ticket-43694 [...] rt.cpan.org> <rt-3.8.HEAD-2358-1288709167-1357.43694-5-0 [...] rt.cpan.org>
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Message-ID: <AANLkTik4XFQd-ji1HzWWuFrzDyNiX8DdRq=qz6MqKaug [...] mail.gmail.com>
Content-Type: text/plain; charset="utf-8"
X-RT-Original-Encoding: utf-8
X-Spam-Score: -6.28
Authentication-Results: hipster.bestpractical.com (amavisd-new); dkim=pass header.i= [...] gmail.com
Authentication-Results: hipster.bestpractical.com (amavisd-new); domainkeys=pass header.sender=jaleto [...] gmail.com
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 29456240FCF for <cpan-bug+Math-BigInt [...] hipster.bestpractical.com>; Tue, 2 Nov 2010 16:28:25 -0400 (EDT)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id rN+eVMdG5Rsw for <cpan-bug+Math-BigInt [...] hipster.bestpractical.com>; Tue, 2 Nov 2010 16:28:23 -0400 (EDT)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 0CB04240FC8 for <bug-Math-BigInt [...] rt.cpan.org>; Tue, 2 Nov 2010 16:28:22 -0400 (EDT)
Received: (qmail 27712 invoked by uid 103); 2 Nov 2010 20:28:28 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 2 Nov 2010 20:28:28 -0000
Received: from mail-yw0-f50.google.com (HELO mail-yw0-f50.google.com) (209.85.213.50) by 16.mx.develooper.com (qpsmtpd/0.80) with ESMTP; Tue, 02 Nov 2010 13:28:26 -0700
Received: by ywi4 with SMTP id 4so4644763ywi.9 for <bug-Math-BigInt [...] rt.cpan.org>; Tue, 02 Nov 2010 13:28:24 -0700 (PDT)
Received: by 10.42.228.8 with SMTP id jc8mr7470763icb.86.1288729703753; Tue, 02 Nov 2010 13:28:23 -0700 (PDT)
Received: by 10.220.191.201 with HTTP; Tue, 2 Nov 2010 13:28:08 -0700 (PDT)
Delivered-To: cpan-bug+Math-BigInt [...] hipster.bestpractical.com
Subject: Re: [rt.cpan.org #43694] The as_int() method on BigFloat objects loses precision on integers
Domainkey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:from:date :x-google-sender-auth:message-id:subject:to:cc:content-type :content-transfer-encoding; b=Lq2cZGagoQdik9TG0nX/E/zNmspIp4btjbWqgk7y+2d+xRsAJMkTXzx18+XR7wx/g8 nsyCk47OuFM4cBNSUCPIN7+F9LxsiBhdgXrbFsKvZFBEyTC5vafV9hBWK4sS8OMRftsu IJl6f4a+BZHw5QBOXfzfzd4eG9zYgx/MbwkRk=
Return-Path: <jaleto [...] gmail.com>
Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:mime-version:sender:received :in-reply-to:references:from:date:x-google-sender-auth:message-id :subject:to:cc:content-type:content-transfer-encoding; bh=fgapS40h7KbK12x+d2Omnh6z3DP76gdaqnGnH13iPGY=; b=ScaS6DxPd/ABI7fhnjOMNwOphqgnBSbcXhIQNfP+7m1JVpZLohgvgM0tmYuvY/7L+P pKCYyQ27pWLpl+b9+K200+k3RrnGb7I50U7YWgKPYRBeLmlcMS6V5hrCO6ApWab6ld4q l+9YxemU5b3jpIQJTJzxXSeb6AWsX/0hbQgIc=
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: cpan-bug+Math-BigInt [...] hipster.bestpractical.com
X-RT-Mail-Extension: math-bigint
X-Google-Sender-Auth: Wp8NkYqvI-zviYeDmZm6Ui1JQoA
Sender: jaleto [...] gmail.com
Date: Tue, 2 Nov 2010 13:28:08 -0700
X-Spam-Level:
To: bug-Math-BigInt [...] rt.cpan.org
Content-Transfer-Encoding: quoted-printable
From: Jonathan Leto <jonathan [...] leto.net>
RT-Message-ID: <rt-3.8.HEAD-2358-1288729711-917.43694-0-0 [...] rt.cpan.org>
Content-Length: 1831
Download (untitled) / with headers
text/plain 1.7k
Howdy Peter, Thanks for all of your bug reports recently. Maintenance hasn't stalled, but the current maintainers of these modules, myself included, don't have copious free time to hack on it. Perhaps you would like a commit bit on the relevant repos, so that we are not a bottleneck towards improving them? I definitely give you a +1 for a commit bit. Duke On Tue, Nov 2, 2010 at 7:46 AM, Peter John Acklam via RT <bug-Math-BigInt@rt.cpan.org> wrote: Show quoted text
>       Queue: Math-BigInt >  Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=43694 > > > The problem can be traced to the second last line of the as_number() > method in Math::BigFloat. The line is > >  $z = Math::BigInt->new( $x->{sign} . $MBI->_num($z)); > > Before this line $z = [987654321, 234567890, 1] on my computer. It might > be different on other computers, depending on $BASE, but it will be > something equivalent. So far, so good. > > Now, $MBI->_num($z) returns the Perl scalar 1.23456789098765e+18, which > is then used to construct a Math::BigInt object with the value > 1234567890987650000. Clearly, using $MBI->_num($z) here is a mistake. > > I'm not really sure whether the main problem is in the code or the > documentation. The name as_number() suggests that returning a Perl > scalar is the right thing, whereas the name as_int() suggests that > returning a Math::BigInt (or a Perl scalar integer) is the right thing. > > This bug is two years(!) old, so it's time to fix it. I don't have a > patch yet, but I'll write one. In the meantime, it would be nice if the > maintainers did some maintaining. I have written six patches over the > last few weeks. So far I have not received any feedback on any of the > patches from any of the maintainers. Has maintenance stalled? >
-- Jonathan "Duke" Leto jonathan@leto.net http://leto.net
From emazep [...] gmail.com Tue Nov 2 17: 55:55 2010
MIME-Version: 1.0
X-Spam-Status: No, score=-6.21 tagged_above=-99.9 required=10 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_HI=-5, SPF_NEUTRAL=0.779, T_TO_NO_BRKTS_FREEMAIL=0.01] autolearn=ham
In-Reply-To: <rt-3.8.HEAD-2357-1288728519-1460.43694-6-0 [...] rt.cpan.org>
X-Spam-Flag: NO
References: <RT-Ticket-43694 [...] rt.cpan.org> <rt-3.8.HEAD-2358-1288709167-757.43694-6-0 [...] rt.cpan.org> <rt-3.8.HEAD-2357-1288728519-1460.43694-6-0 [...] rt.cpan.org>
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Message-ID: <4CD088E6.4090804 [...] gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
X-RT-Original-Encoding: utf-8
X-Spam-Score: -6.21
Authentication-Results: hipster.bestpractical.com (amavisd-new); dkim=pass header.i= [...] gmail.com
Authentication-Results: hipster.bestpractical.com (amavisd-new); domainkeys=pass header.from=emazep [...] gmail.com
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 061A2240FD2 for <cpan-bug+Math-BigInt [...] hipster.bestpractical.com>; Tue, 2 Nov 2010 17:55:55 -0400 (EDT)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id yMfMC5zdhNFZ for <cpan-bug+Math-BigInt [...] hipster.bestpractical.com>; Tue, 2 Nov 2010 17:55:53 -0400 (EDT)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 40CE4240FCE for <bug-Math-BigInt [...] rt.cpan.org>; Tue, 2 Nov 2010 17:55:53 -0400 (EDT)
Received: (qmail 5000 invoked by uid 103); 2 Nov 2010 21:55:58 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 2 Nov 2010 21:55:58 -0000
Received: from mail-wy0-f178.google.com (HELO mail-wy0-f178.google.com) (74.125.82.178) by 16.mx.develooper.com (qpsmtpd/0.80) with ESMTP; Tue, 02 Nov 2010 14:55:57 -0700
Received: by wyb29 with SMTP id 29so7002747wyb.9 for <bug-Math-BigInt [...] rt.cpan.org>; Tue, 02 Nov 2010 14:55:53 -0700 (PDT)
Received: by 10.216.64.139 with SMTP id c11mr2466980wed.81.1288734953269; Tue, 02 Nov 2010 14:55:53 -0700 (PDT)
Received: from [192.168.0.20] (adsl-ull-172-47.50-151.net24.it [151.50.47.172]) by mx.google.com with ESMTPS id p4sm5071839wej.4.2010.11.02.14.55.51 (version=SSLv3 cipher=RC4-MD5); Tue, 02 Nov 2010 14:55:52 -0700 (PDT)
Delivered-To: cpan-bug+Math-BigInt [...] hipster.bestpractical.com
User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.12) Gecko/20101027 Thunderbird/3.1.6
Subject: Re: [rt.cpan.org #43694] The as_int() method on BigFloat objects loses precision on integers
Domainkey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:user-agent:mime-version:to:subject:references :in-reply-to:content-type:content-transfer-encoding; b=vbegYFSwkGvhMyWWKQMl+gPf/+XBS/FRCIepTCJmqeYDrkKRU5AJdAl9uN8UZ9R8XM IeG3B8kvSQrUiROIhJSH9Obmi35K7z7LsoxR6dvc7RRbD2ZtZWercjg26SSnlz+wxAs1 Eta7j11Z93B8dGqm2umas9VLo+4Z7LR7SyoWg=
Return-Path: <emazep [...] gmail.com>
Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from :user-agent:mime-version:to:subject:references:in-reply-to :content-type:content-transfer-encoding; bh=g+KBEadAseieyHOBqu4EvqhAlhQ8N6Q+vYI3AnvHxqI=; b=avoSi7Ibku/ZPPdetSuh+UsHGAAlFZa4uv7Fzh38N6u1sYFilCFTl6bJxKwOAM/u7t SOoeABtafVH6e/X+kjrSdQxKW2IffpQk0J0TGx5qZ8RF/IWuDGefe6ihSXQ9ewYGek+H BiQBuWjdHsHPYNrsixEp36oS5UIg64RewavlQ=
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: cpan-bug+Math-BigInt [...] hipster.bestpractical.com
X-RT-Mail-Extension: math-bigint
Date: Tue, 02 Nov 2010 22:55:50 +0100
X-Spam-Level:
To: bug-Math-BigInt [...] rt.cpan.org
Content-Transfer-Encoding: 7bit
From: Emanuele Zeppieri <emazep [...] gmail.com>
RT-Message-ID: <rt-3.8.HEAD-2361-1288734961-72.43694-0-0 [...] rt.cpan.org>
Content-Length: 474
Download (untitled) / with headers
text/plain 474b
On 2010-11-02 21:08, Peter John Acklam via RT wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=43694> > > This bug was introduced in version 1.70. > > In Math::BigFloat -> as_number() I changed _num() to _str(). The former > returns a Perl scalar, but the latter returns a string, preserving all > digits. I also added a test to confirms the correct behaviour. > > Patch included.
It seems the correct solution also to me. Thank you very much Peter! -Emanuele
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-2358-1288729711-917.43694-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <RT-Ticket-43694 [...] rt.cpan.org> <rt-3.8.HEAD-2358-1288709167-1357.43694-5-0 [...] rt.cpan.org> <AANLkTik4XFQd-ji1HzWWuFrzDyNiX8DdRq=qz6MqKaug [...] mail.gmail.com> <rt-3.8.HEAD-2358-1288729711-917.43694-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-2357-1288767691-39.43694-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
RT-Send-CC: jonathan [...] leto.net, rafl [...] debian.org
Content-Length: 624
Download (untitled) / with headers
text/plain 624b
If none of you have time to apply the patch(es), then I'll be happy to do it. When you say "+1 for a commit bit", are you referring to git://perl5.git.perl.org/perl.git or some other repo? Peter On Tue Nov 02 16:28:31 2010, LETO wrote: Show quoted text
> Howdy Peter, > > Thanks for all of your bug reports recently. Maintenance hasn't > stalled, but the current > maintainers of these modules, myself included, don't have copious free > time to hack > on it. Perhaps you would like a commit bit on the relevant repos, so > that we are not a > bottleneck towards improving them? > > I definitely give you a +1 for a commit bit. > > Duke
From rafl [...] debian.org Wed Nov 3 06: 27:27 2010
CC: bug-Math-BigInt [...] rt.cpan.org, jonathan [...] leto.net
MIME-Version: 1.0
X-Spam-Status: No, score=-6.411 tagged_above=-99.9 required=10 tests=[AWL=0.489, BAYES_00=-1.9, RCVD_IN_DNSWL_HI=-5] autolearn=ham
In-Reply-To: <rt-3.8.HEAD-2357-1288767692-1122.43694-7-0 [...] rt.cpan.org> (Peter John Acklam via's message of "Wed, 3 Nov 2010 03:01:32 -0400")
X-Spam_score: -3.9
X-Spam-Flag: NO
References: <RT-Ticket-43694 [...] rt.cpan.org> <rt-3.8.HEAD-2358-1288709167-1357.43694-5-0 [...] rt.cpan.org> <AANLkTik4XFQd-ji1HzWWuFrzDyNiX8DdRq=qz6MqKaug [...] mail.gmail.com> <rt-3.8.HEAD-2358-1288729711-917.43694-7-0 [...] rt.cpan.org> <rt-3.8.HEAD-2357-1288767692-1122.43694-7-0 [...] rt.cpan.org>
X-Virus-Checked: Checked by ClamAV on 16.mx.develooper.com
X-Spam_bar: ---
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Message-ID: <87r5f2ya7i.fsf [...] tardis.home.perldition.org>
Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha1; protocol="application/pgp-signature"
X-Spam-Score: -6.411
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 8F012240FB9 for <cpan-bug+Math-BigInt [...] hipster.bestpractical.com>; Wed, 3 Nov 2010 06:27:27 -0400 (EDT)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id BpV6oMHI3Ffh for <cpan-bug+Math-BigInt [...] hipster.bestpractical.com>; Wed, 3 Nov 2010 06:27:25 -0400 (EDT)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 21001240FB2 for <bug-Math-BigInt [...] rt.cpan.org>; Wed, 3 Nov 2010 06:27:24 -0400 (EDT)
Received: (qmail 14870 invoked by uid 103); 3 Nov 2010 10:27:30 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 3 Nov 2010 10:27:30 -0000
Received: from kief.perldition.org (HELO kief.perldition.org) (78.47.20.161) by 16.mx.develooper.com (qpsmtpd/0.80) with ESMTP; Wed, 03 Nov 2010 03:27:28 -0700
Received: from p4fd73c55.dip.t-dialin.net ([79.215.60.85] helo=tardis.home.perldition.org) by kief.perldition.org with esmtpsa (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.69) (envelope-from <rafl [...] debian.org>) id 1PDaYs-0002kK-Ix; Wed, 03 Nov 2010 11:27:24 +0100
Received: from rafl by tardis.home.perldition.org with local (Exim 4.72) (envelope-from <rafl [...] debian.org>) id 1PDaY0-0007R3-Sb; Wed, 03 Nov 2010 11:26:28 +0100
Delivered-To: cpan-bug+Math-BigInt [...] hipster.bestpractical.com
X-Spam_score_int: -38
Subject: Re: [rt.cpan.org #43694] The as_int() method on BigFloat objects loses precision on integers
User-Agent: Gnus/5.110011 (No Gnus v0.11) Emacs/23.2 (gnu/linux)
Return-Path: <rafl [...] debian.org>
X-Spam-Check-BY: 16.mx.develooper.com
X-Original-To: cpan-bug+Math-BigInt [...] hipster.bestpractical.com
X-RT-Mail-Extension: math-bigint
Date: Wed, 03 Nov 2010 11:26:25 +0100
X-Spam-Level:
X-Spam_report: Spam detection software, running on the system "kief.perldition.org", has identified this incoming email as possible spam. The original message has been attached to this so you can view it (if it isn't spam) or label similar future email. If you have any questions, see the administrator of that system for details. Content preview: "Peter John Acklam via RT" <bug-Math-BigInt [...] rt.cpan.org> writes: > If none of you have time to apply the patch(es), then I'll be happy to > do it. At least for me, it's not so much the applying, but the reviewing. However, reviewing would also be a whole lot easier if there was a git repository to just fetch from. [...] Content analysis details: (-3.9 points, 5.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- -1.8 ALL_TRUSTED Passed through trusted hosts only via SMTP -2.6 BAYES_00 BODY: Bayesian spam probability is 0 to 1% [score: 0.0000] 0.5 AWL AWL: From: address is in the auto white-list
To: pjacklam [...] online.no
From: Florian Ragwitz <rafl [...] debian.org>
RT-Message-ID: <rt-3.8.HEAD-2357-1288780054-1043.43694-0-0 [...] rt.cpan.org>
Content-Length: 0
Content-Type: text/plain; charset="utf-8"
X-RT-Original-Encoding: utf-8
Content-Length: 1579
Download (untitled) / with headers
text/plain 1.5k
"Peter John Acklam via RT" <bug-Math-BigInt@rt.cpan.org> writes: Show quoted text
> If none of you have time to apply the patch(es), then I'll be happy to > do it.
At least for me, it's not so much the applying, but the reviewing. However, reviewing would also be a whole lot easier if there was a git repository to just fetch from. Show quoted text
> When you say "+1 for a commit bit", are you referring to > git://perl5.git.perl.org/perl.git or some other repo?
Handing out commit bits to perl itself unfortunately isn't quite as easy as one would hope. However, There's repositories for all of the bigint stuff on github. They all live at http://github.com/rafl, for example http://github.com/rafl/Math-BigInt and http://github.com/rafl/Math-BigInt-FastCalc. Math-BigRat, the only exception, is at http://github.com/leto/Math-BigRat. While all the bigint modules actually have the perl core as their canonical upstream, I'll be happy to sync changes between the individual repositories for the CPAN versions and the core, like I've been doing in the past. If you want, you can get a commit bit for all of those so you can create branches for the individual issues you've been working on. Alternatively you can just click the Fork button on any of those repositories to get your own repository of them. I'd actually prefer the latter, as that requires the contributor to explicitly tell someone about his changes via private email, or githubs pull request, or whatever, so they can be pulled in - avoiding the problem of forgotten branches. However, I'll be happy to cope with whatever works for you.
Content-Type: application/pgp-signature
Content-Length: 197
Download (untitled)
application/pgp-signature 197b

Message body not shown because it is not plain text.



This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.