integrate cfgperl contents into mainline
[p5sagit/p5-mst-13.2.git] / lib / Math / Complex.pm
index 36ca060..1a47f4a 100644 (file)
@@ -8,13 +8,14 @@
 require Exporter;
 package Math::Complex;
 
+use 5.005_64;
 use strict;
 
-use vars qw($VERSION @ISA @EXPORT %EXPORT_TAGS);
+our($VERSION, @ISA, @EXPORT, %EXPORT_TAGS);
 
 my ( $i, $ip2, %logn );
 
-$VERSION = sprintf("%s", q$Id: Complex.pm,v 1.25 1998/02/05 16:07:37 jhi Exp $ =~ /(\d+\.\d+)/);
+$VERSION = sprintf("%s", q$Id: Complex.pm,v 1.26 1998/11/01 00:00:00 dsl Exp $ =~ /(\d+\.\d+)/);
 
 @ISA = qw(Exporter);
 
@@ -65,9 +66,10 @@ use overload
 # Package "privates"
 #
 
-my $package = 'Math::Complex';         # Package name
-my $display = 'cartesian';             # Default display format
-my $eps     = 1e-14;                   # Epsilon
+my $package        = 'Math::Complex';  # Package name
+my %DISPLAY_FORMAT = ('style' => 'cartesian',
+                     'polar_pretty_print' => 1);
+my $eps            = 1e-14;            # Epsilon
 
 #
 # Object attributes (internal):
@@ -160,7 +162,7 @@ sub new { &make }           # For backward compatibility only.
 #
 sub cplx {
        my ($re, $im) = @_;
-       return $package->make($re, defined $im ? $im : 0);
+       return __PACKAGE__->make($re, defined $im ? $im : 0);
 }
 
 #
@@ -171,7 +173,7 @@ sub cplx {
 #
 sub cplxe {
        my ($rho, $theta) = @_;
-       return $package->emake($rho, defined $theta ? $theta : 0);
+       return __PACKAGE__->emake($rho, defined $theta ? $theta : 0);
 }
 
 #
@@ -179,28 +181,36 @@ sub cplxe {
 #
 # The number defined as pi = 180 degrees
 #
-use constant pi => 4 * atan2(1, 1);
+sub pi () { 4 * CORE::atan2(1, 1) }
 
 #
 # pit2
 #
 # The full circle
 #
-use constant pit2 => 2 * pi;
+sub pit2 () { 2 * pi }
 
 #
 # pip2
 #
 # The quarter circle
 #
-use constant pip2 => pi / 2;
+sub pip2 () { pi / 2 }
+
+#
+# deg1
+#
+# One degree in radians, used in stringify_polar.
+#
+
+sub deg1 () { pi / 180 }
 
 #
 # uplog10
 #
 # Used in log10().
 #
-use constant uplog10 => 1 / log(10);
+sub uplog10 () { 1 / CORE::log(10) }
 
 #
 # i
@@ -238,7 +248,7 @@ sub update_cartesian {
        my $self = shift;
        my ($r, $t) = @{$self->{'polar'}};
        $self->{c_dirty} = 0;
-       return $self->{'cartesian'} = [$r * cos $t, $r * sin $t];
+       return $self->{'cartesian'} = [$r * CORE::cos($t), $r * CORE::sin($t)];
 }
 
 #
@@ -252,7 +262,7 @@ sub update_polar {
        my ($x, $y) = @{$self->{'cartesian'}};
        $self->{p_dirty} = 0;
        return $self->{'polar'} = [0, 0] if $x == 0 && $y == 0;
-       return $self->{'polar'} = [sqrt($x*$x + $y*$y), atan2($y, $x)];
+       return $self->{'polar'} = [CORE::sqrt($x*$x + $y*$y), CORE::atan2($y, $x)];
 }
 
 #
@@ -393,45 +403,32 @@ sub divide {
 }
 
 #
-# _zerotozero
-#
-# Die on zero raised to the zeroth.
-#
-sub _zerotozero {
-    my $mess = "The zero raised to the zeroth power is not defined.\n";
-
-    my @up = caller(1);
-
-    $mess .= "Died at $up[1] line $up[2].\n";
-
-    die $mess;
-}
-
-#
 # (power)
 #
 # Computes z1**z2 = exp(z2 * log z1)).
 #
 sub power {
        my ($z1, $z2, $inverted) = @_;
-       my $z1z = $z1 == 0;
-       my $z2z = $z2 == 0;
-       _zerotozero if ($z1z and $z2z);
        if ($inverted) {
-           return 0 if ($z2z);
-           return 1 if ($z1z or $z2 == 1);
+           return 1 if $z1 == 0 || $z2 == 1;
+           return 0 if $z2 == 0 && Re($z1) > 0;
        } else {
-           return 0 if ($z1z);
-           return 1 if ($z2z or $z1 == 1);
+           return 1 if $z2 == 0 || $z1 == 1;
+           return 0 if $z1 == 0 && Re($z2) > 0;
        }
-       return $inverted ? exp($z1 * log $z2) : exp($z2 * log $z1);
+       my $w = $inverted ? CORE::exp($z1 * CORE::log($z2))
+                         : CORE::exp($z2 * CORE::log($z1));
+       # If both arguments cartesian, return cartesian, else polar.
+       return $z1->{c_dirty} == 0 &&
+              (not ref $z2 or $z2->{c_dirty} == 0) ?
+              cplx(@{$w->cartesian}) : $w;
 }
 
 #
 # (spaceship)
 #
 # Computes z1 <=> z2.
-# Sorts on the real part first, then on the imaginary part. Thus 2-4i > 3+8i.
+# Sorts on the real part first, then on the imaginary part. Thus 2-4i < 3+8i.
 #
 sub spaceship {
        my ($z1, $z2, $inverted) = @_;
@@ -536,9 +533,9 @@ sub arg {
 sub sqrt {
        my ($z) = @_;
        my ($re, $im) = ref $z ? @{$z->cartesian} : ($z, 0);
-       return $re < 0 ? cplx(0, sqrt(-$re)) : sqrt($re) if $im == 0;
+       return $re < 0 ? cplx(0, CORE::sqrt(-$re)) : CORE::sqrt($re) if $im == 0;
        my ($r, $t) = @{$z->polar};
-       return (ref $z)->emake(sqrt($r), $t/2);
+       return (ref $z)->emake(CORE::sqrt($r), $t/2);
 }
 
 #
@@ -550,10 +547,10 @@ sub sqrt {
 #
 sub cbrt {
        my ($z) = @_;
-       return $z < 0 ? -exp(log(-$z)/3) : ($z > 0 ? exp(log($z)/3): 0)
+       return $z < 0 ? -CORE::exp(CORE::log(-$z)/3) : ($z > 0 ? CORE::exp(CORE::log($z)/3): 0)
            unless ref $z;
        my ($r, $t) = @{$z->polar};
-       return (ref $z)->emake(exp(log($r)/3), $t/3);
+       return (ref $z)->emake(CORE::exp(CORE::log($r)/3), $t/3);
 }
 
 #
@@ -584,15 +581,17 @@ sub _rootbad {
 sub root {
        my ($z, $n) = @_;
        _rootbad($n) if ($n < 1 or int($n) != $n);
-       my ($r, $t) = ref $z ? @{$z->polar} : (abs($z), $z >= 0 ? 0 : pi);
+       my ($r, $t) = ref $z ? @{$z->polar} : (CORE::abs($z), $z >= 0 ? 0 : pi);
        my @root;
        my $k;
        my $theta_inc = pit2 / $n;
        my $rho = $r ** (1/$n);
        my $theta;
-       my $complex = ref($z) || $package;
+       my $cartesian = ref $z && $z->{c_dirty} == 0;
        for ($k = 0, $theta = $t / $n; $k < $n; $k++, $theta += $theta_inc) {
-               push(@root, $complex->emake($rho, $theta));
+           my $w = cplxe($rho, $theta);
+           # Yes, $cartesian is loop invariant.
+           push @root, $cartesian ? cplx(@{$w->cartesian}) : $w;
        }
        return @root;
 }
@@ -657,7 +656,7 @@ sub theta {
 sub exp {
        my ($z) = @_;
        my ($x, $y) = @{$z->cartesian};
-       return (ref $z)->emake(exp($x), $y);
+       return (ref $z)->emake(CORE::exp($x), $y);
 }
 
 #
@@ -690,13 +689,13 @@ sub log {
        my ($z) = @_;
        unless (ref $z) {
            _logofzero("log") if $z == 0;
-           return $z > 0 ? log($z) : cplx(log(-$z), pi);
+           return $z > 0 ? CORE::log($z) : cplx(CORE::log(-$z), pi);
        }
        my ($r, $t) = @{$z->polar};
        _logofzero("log") if $r == 0;
        if    ($t >   pi()) { $t -= pit2 }
        elsif ($t <= -pi()) { $t += pit2 }
-       return (ref $z)->make(log($r), $t);
+       return (ref $z)->make(CORE::log($r), $t);
 }
 
 #
@@ -725,8 +724,8 @@ sub logn {
        my ($z, $n) = @_;
        $z = cplx($z, 0) unless ref $z;
        my $logn = $logn{$n};
-       $logn = $logn{$n} = log($n) unless defined $logn;       # Cache log(n)
-       return log($z) / $logn;
+       $logn = $logn{$n} = CORE::log($n) unless defined $logn; # Cache log(n)
+       return CORE::log($z) / $logn;
 }
 
 #
@@ -737,10 +736,10 @@ sub logn {
 sub cos {
        my ($z) = @_;
        my ($x, $y) = @{$z->cartesian};
-       my $ey = exp($y);
+       my $ey = CORE::exp($y);
        my $ey_1 = 1 / $ey;
-       return (ref $z)->make(cos($x) * ($ey + $ey_1)/2,
-                             sin($x) * ($ey_1 - $ey)/2);
+       return (ref $z)->make(CORE::cos($x) * ($ey + $ey_1)/2,
+                             CORE::sin($x) * ($ey_1 - $ey)/2);
 }
 
 #
@@ -751,10 +750,10 @@ sub cos {
 sub sin {
        my ($z) = @_;
        my ($x, $y) = @{$z->cartesian};
-       my $ey = exp($y);
+       my $ey = CORE::exp($y);
        my $ey_1 = 1 / $ey;
-       return (ref $z)->make(sin($x) * ($ey + $ey_1)/2,
-                             cos($x) * ($ey - $ey_1)/2);
+       return (ref $z)->make(CORE::sin($x) * ($ey + $ey_1)/2,
+                             CORE::cos($x) * ($ey - $ey_1)/2);
 }
 
 #
@@ -764,9 +763,9 @@ sub sin {
 #
 sub tan {
        my ($z) = @_;
-       my $cz = cos($z);
-       _divbyzero "tan($z)", "cos($z)" if (abs($cz) < $eps);
-       return sin($z) / $cz;
+       my $cz = CORE::cos($z);
+       _divbyzero "tan($z)", "cos($z)" if (CORE::abs($cz) < $eps);
+       return CORE::sin($z) / $cz;
 }
 
 #
@@ -776,7 +775,7 @@ sub tan {
 #
 sub sec {
        my ($z) = @_;
-       my $cz = cos($z);
+       my $cz = CORE::cos($z);
        _divbyzero "sec($z)", "cos($z)" if ($cz == 0);
        return 1 / $cz;
 }
@@ -788,7 +787,7 @@ sub sec {
 #
 sub csc {
        my ($z) = @_;
-       my $sz = sin($z);
+       my $sz = CORE::sin($z);
        _divbyzero "csc($z)", "sin($z)" if ($sz == 0);
        return 1 / $sz;
 }
@@ -807,9 +806,9 @@ sub cosec { Math::Complex::csc(@_) }
 #
 sub cot {
        my ($z) = @_;
-       my $sz = sin($z);
+       my $sz = CORE::sin($z);
        _divbyzero "cot($z)", "sin($z)" if ($sz == 0);
-       return cos($z) / $sz;
+       return CORE::cos($z) / $sz;
 }
 
 #
@@ -826,19 +825,19 @@ sub cotan { Math::Complex::cot(@_) }
 #
 sub acos {
        my $z = $_[0];
-       return atan2(sqrt(1-$z*$z), $z) if (! ref $z) && abs($z) <= 1;
+       return CORE::atan2(CORE::sqrt(1-$z*$z), $z) if (! ref $z) && CORE::abs($z) <= 1;
        my ($x, $y) = ref $z ? @{$z->cartesian} : ($z, 0);
-       my $t1 = sqrt(($x+1)*($x+1) + $y*$y);
-       my $t2 = sqrt(($x-1)*($x-1) + $y*$y);
+       my $t1 = CORE::sqrt(($x+1)*($x+1) + $y*$y);
+       my $t2 = CORE::sqrt(($x-1)*($x-1) + $y*$y);
        my $alpha = ($t1 + $t2)/2;
        my $beta  = ($t1 - $t2)/2;
        $alpha = 1 if $alpha < 1;
        if    ($beta >  1) { $beta =  1 }
        elsif ($beta < -1) { $beta = -1 }
-       my $u = atan2(sqrt(1-$beta*$beta), $beta);
-       my $v = log($alpha + sqrt($alpha*$alpha-1));
+       my $u = CORE::atan2(CORE::sqrt(1-$beta*$beta), $beta);
+       my $v = CORE::log($alpha + CORE::sqrt($alpha*$alpha-1));
        $v = -$v if $y > 0 || ($y == 0 && $x < -1);
-       return $package->make($u, $v);
+       return __PACKAGE__->make($u, $v);
 }
 
 #
@@ -848,19 +847,19 @@ sub acos {
 #
 sub asin {
        my $z = $_[0];
-       return atan2($z, sqrt(1-$z*$z)) if (! ref $z) && abs($z) <= 1;
+       return CORE::atan2($z, CORE::sqrt(1-$z*$z)) if (! ref $z) && CORE::abs($z) <= 1;
        my ($x, $y) = ref $z ? @{$z->cartesian} : ($z, 0);
-       my $t1 = sqrt(($x+1)*($x+1) + $y*$y);
-       my $t2 = sqrt(($x-1)*($x-1) + $y*$y);
+       my $t1 = CORE::sqrt(($x+1)*($x+1) + $y*$y);
+       my $t2 = CORE::sqrt(($x-1)*($x-1) + $y*$y);
        my $alpha = ($t1 + $t2)/2;
        my $beta  = ($t1 - $t2)/2;
        $alpha = 1 if $alpha < 1;
        if    ($beta >  1) { $beta =  1 }
        elsif ($beta < -1) { $beta = -1 }
-       my $u =  atan2($beta, sqrt(1-$beta*$beta));
-       my $v = -log($alpha + sqrt($alpha*$alpha-1));
+       my $u =  CORE::atan2($beta, CORE::sqrt(1-$beta*$beta));
+       my $v = -CORE::log($alpha + CORE::sqrt($alpha*$alpha-1));
        $v = -$v if $y > 0 || ($y == 0 && $x < -1);
-       return $package->make($u, $v);
+       return __PACKAGE__->make($u, $v);
 }
 
 #
@@ -870,10 +869,10 @@ sub asin {
 #
 sub atan {
        my ($z) = @_;
-       return atan2($z, 1) unless ref $z;
+       return CORE::atan2($z, 1) unless ref $z;
        _divbyzero "atan(i)"  if ( $z == i);
        _divbyzero "atan(-i)" if (-$z == i);
-       my $log = log((i + $z) / (i - $z));
+       my $log = CORE::log((i + $z) / (i - $z));
        $ip2 = 0.5 * i unless defined $ip2;
        return $ip2 * $log;
 }
@@ -914,10 +913,10 @@ sub acosec { Math::Complex::acsc(@_) }
 #
 sub acot {
        my ($z) = @_;
-       _divbyzero "acot(0)"  if (abs($z)     < $eps);
-       return ($z >= 0) ? atan2(1, $z) : atan2(-1, -$z) unless ref $z;
-       _divbyzero "acot(i)"  if (abs($z - i) < $eps);
-       _logofzero "acot(-i)" if (abs($z + i) < $eps);
+       _divbyzero "acot(0)"  if (CORE::abs($z)     < $eps);
+       return ($z >= 0) ? CORE::atan2(1, $z) : CORE::atan2(-1, -$z) unless ref $z;
+       _divbyzero "acot(i)"  if (CORE::abs($z - i) < $eps);
+       _logofzero "acot(-i)" if (CORE::abs($z + i) < $eps);
        return atan(1 / $z);
 }
 
@@ -937,14 +936,14 @@ sub cosh {
        my ($z) = @_;
        my $ex;
        unless (ref $z) {
-           $ex = exp($z);
+           $ex = CORE::exp($z);
            return ($ex + 1/$ex)/2;
        }
        my ($x, $y) = @{$z->cartesian};
-       $ex = exp($x);
+       $ex = CORE::exp($x);
        my $ex_1 = 1 / $ex;
-       return (ref $z)->make(cos($y) * ($ex + $ex_1)/2,
-                             sin($y) * ($ex - $ex_1)/2);
+       return (ref $z)->make(CORE::cos($y) * ($ex + $ex_1)/2,
+                             CORE::sin($y) * ($ex - $ex_1)/2);
 }
 
 #
@@ -956,14 +955,14 @@ sub sinh {
        my ($z) = @_;
        my $ex;
        unless (ref $z) {
-           $ex = exp($z);
+           $ex = CORE::exp($z);
            return ($ex - 1/$ex)/2;
        }
        my ($x, $y) = @{$z->cartesian};
-       $ex = exp($x);
+       $ex = CORE::exp($x);
        my $ex_1 = 1 / $ex;
-       return (ref $z)->make(cos($y) * ($ex - $ex_1)/2,
-                             sin($y) * ($ex + $ex_1)/2);
+       return (ref $z)->make(CORE::cos($y) * ($ex - $ex_1)/2,
+                             CORE::sin($y) * ($ex + $ex_1)/2);
 }
 
 #
@@ -1036,15 +1035,15 @@ sub cotanh { Math::Complex::coth(@_) }
 sub acosh {
        my ($z) = @_;
        unless (ref $z) {
-           return log($z + sqrt($z*$z-1)) if $z >= 1;
+           return CORE::log($z + CORE::sqrt($z*$z-1)) if $z >= 1;
            $z = cplx($z, 0);
        }
        my ($re, $im) = @{$z->cartesian};
        if ($im == 0) {
-           return cplx(log($re + sqrt($re*$re - 1)), 0) if $re >= 1;
-           return cplx(0, atan2(sqrt(1-$re*$re), $re)) if abs($re) <= 1;
+           return cplx(CORE::log($re + CORE::sqrt($re*$re - 1)), 0) if $re >= 1;
+           return cplx(0, CORE::atan2(CORE::sqrt(1-$re*$re), $re)) if CORE::abs($re) <= 1;
        }
-       return log($z + sqrt($z*$z - 1));
+       return CORE::log($z + CORE::sqrt($z*$z - 1));
 }
 
 #
@@ -1054,7 +1053,7 @@ sub acosh {
 #
 sub asinh {
        my ($z) = @_;
-       return log($z + sqrt($z*$z + 1));
+       return CORE::log($z + CORE::sqrt($z*$z + 1));
 }
 
 #
@@ -1065,12 +1064,12 @@ sub asinh {
 sub atanh {
        my ($z) = @_;
        unless (ref $z) {
-           return log((1 + $z)/(1 - $z))/2 if abs($z) < 1;
+           return CORE::log((1 + $z)/(1 - $z))/2 if CORE::abs($z) < 1;
            $z = cplx($z, 0);
        }
        _divbyzero 'atanh(1)',  "1 - $z" if ($z ==  1);
        _logofzero 'atanh(-1)'           if ($z == -1);
-       return 0.5 * log((1 + $z) / (1 - $z));
+       return 0.5 * CORE::log((1 + $z) / (1 - $z));
 }
 
 #
@@ -1109,14 +1108,14 @@ sub acosech { Math::Complex::acsch(@_) }
 #
 sub acoth {
        my ($z) = @_;
-       _divbyzero 'acoth(0)'            if (abs($z)     < $eps);
+       _divbyzero 'acoth(0)'            if (CORE::abs($z)     < $eps);
        unless (ref $z) {
-           return log(($z + 1)/($z - 1))/2 if abs($z) > 1;
+           return CORE::log(($z + 1)/($z - 1))/2 if CORE::abs($z) > 1;
            $z = cplx($z, 0);
        }
-       _divbyzero 'acoth(1)',  "$z - 1" if (abs($z - 1) < $eps);
-       _logofzero 'acoth(-1)', "1 / $z" if (abs($z + 1) < $eps);
-       return log((1 + $z) / ($z - 1)) / 2;
+       _divbyzero 'acoth(1)',  "$z - 1" if (CORE::abs($z - 1) < $eps);
+       _logofzero 'acoth(-1)', "1 / $z" if (CORE::abs($z + 1) < $eps);
+       return CORE::log((1 + $z) / ($z - 1)) / 2;
 }
 
 #
@@ -1142,7 +1141,7 @@ sub atan2 {
            ($re2, $im2) = ref $z2 ? @{$z2->cartesian} : ($z2, 0);
        }
        if ($im2 == 0) {
-           return cplx(atan2($re1, $re2), 0) if $im1 == 0;
+           return cplx(CORE::atan2($re1, $re2), 0) if $im1 == 0;
            return cplx(($im1<=>0) * pip2, 0) if $re2 == 0;
        }
        my $w = atan($z1/$z2);
@@ -1156,34 +1155,53 @@ sub atan2 {
 # display_format
 # ->display_format
 #
-# Set (fetch if no argument) display format for all complex numbers that
+# Set (get if no argument) the display format for all complex numbers that
 # don't happen to have overridden it via ->display_format
 #
-# When called as a method, this actually sets the display format for
+# When called as an object method, this actually sets the display format for
 # the current object.
 #
 # Valid object formats are 'c' and 'p' for cartesian and polar. The first
 # letter is used actually, so the type can be fully spelled out for clarity.
 #
 sub display_format {
-       my $self = shift;
-       my $format = undef;
+       my $self  = shift;
+       my %display_format = %DISPLAY_FORMAT;
 
-       if (ref $self) {                        # Called as a method
-               $format = shift;
-       } else {                                # Regular procedure call
-               $format = $self;
-               undef $self;
+       if (ref $self) {                        # Called as an object method
+           if (exists $self->{display_format}) {
+               my %obj = %{$self->{display_format}};
+               @display_format{keys %obj} = values %obj;
+           }
+           if (@_ == 1) {
+               $display_format{style} = shift;
+           } else {
+               my %new = @_;
+               @display_format{keys %new} = values %new;
+           }
+       } else {                                # Called as a class method
+           if (@_ = 1) {
+               $display_format{style} = $self;
+           } else {
+               my %new = @_;
+               @display_format{keys %new} = values %new;
+           }
+           undef $self;
        }
 
        if (defined $self) {
-               return defined $self->{display} ? $self->{display} : $display
-                       unless defined $format;
-               return $self->{display} = $format;
+           $self->{display_format} = { %display_format };
+           return
+               wantarray ?
+                   %{$self->{display_format}} :
+                   $self->{display_format}->{style};
        }
 
-       return $display unless defined $format;
-       return $display = $format;
+       %DISPLAY_FORMAT = %display_format;
+       return
+           wantarray ?
+               %DISPLAY_FORMAT :
+                   $DISPLAY_FORMAT{style};
 }
 
 #
@@ -1198,12 +1216,12 @@ sub display_format {
 #
 sub stringify {
        my ($z) = shift;
-       my $format;
 
-       $format = $display;
-       $format = $z->{display} if defined $z->{display};
+       my $style = $z->display_format;
 
-       return $z->stringify_polar if $format =~ /^p/i;
+       $style = $DISPLAY_FORMAT{style} unless defined $style;
+
+       return $z->stringify_polar if $style =~ /^p/i;
        return $z->stringify_cartesian;
 }
 
@@ -1218,25 +1236,66 @@ sub stringify_cartesian {
        my ($re, $im);
 
        $x = int($x + ($x < 0 ? -1 : 1) * $eps)
-               if int(abs($x)) != int(abs($x) + $eps);
+               if int(CORE::abs($x)) != int(CORE::abs($x) + $eps);
        $y = int($y + ($y < 0 ? -1 : 1) * $eps)
-               if int(abs($y)) != int(abs($y) + $eps);
+               if int(CORE::abs($y)) != int(CORE::abs($y) + $eps);
+
+       $re = "$x" if CORE::abs($x) >= $eps;
 
-       $re = "$x" if abs($x) >= $eps;
-        if ($y == 1)                           { $im = 'i' }
-        elsif ($y == -1)                       { $im = '-i' }
-        elsif (abs($y) >= $eps)                { $im = $y . "i" }
+       my %format = $z->display_format;
+       my $format = $format{format};
+
+       if ($y == 1)                           { $im = 'i' }
+       elsif ($y == -1)                       { $im = '-i' }
+       elsif (CORE::abs($y) >= $eps) {
+           $im = (defined $format ? sprintf($format, $y) : $y) . "i";
+       }
 
        my $str = '';
-       $str = $re if defined $re;
-       $str .= "+$im" if defined $im;
-       $str =~ s/\+-/-/;
-       $str =~ s/^\+//;
-       $str = '0' unless $str;
+       $str = defined $format ? sprintf($format, $re) : $re
+           if defined $re;
+       if (defined $im) {
+           if ($y < 0) {
+               $str .= $im;
+           } elsif ($y > 0)  {
+               $str .= "+" if defined $re;
+               $str .= $im;
+           }
+       }
 
        return $str;
 }
 
+
+# Helper for stringify_polar, a Greatest Common Divisor with a memory.
+
+sub _gcd {
+    my ($a, $b) = @_;
+
+    use integer;
+
+    # Loops forever if given negative inputs.
+
+    if    ($b and $a > $b) { return gcd($a % $b, $b) }
+    elsif ($a and $b > $a) { return gcd($b % $a, $a) }
+    else                   { return $a ? $a : $b     }
+}
+
+my %gcd;
+
+sub gcd {
+    my ($a, $b) = @_;
+
+    my $id = "$a $b";
+
+    unless (exists $gcd{$id}) {
+       $gcd{$id} = _gcd($a, $b);
+       $gcd{"$b $a"} = $gcd{$id};
+    }
+
+    return $gcd{$id};
+}
+
 #
 # ->stringify_polar
 #
@@ -1249,19 +1308,21 @@ sub stringify_polar {
 
        return '[0,0]' if $r <= $eps;
 
+       my %format = $z->display_format;
+
        my $nt = $t / pit2;
        $nt = ($nt - int($nt)) * pit2;
        $nt += pit2 if $nt < 0;                 # Range [0, 2pi]
 
-       if (abs($nt) <= $eps)           { $theta = 0 }
-       elsif (abs(pi-$nt) <= $eps)     { $theta = 'pi' }
+       if (CORE::abs($nt) <= $eps)             { $theta = 0 }
+       elsif (CORE::abs(pi-$nt) <= $eps)       { $theta = 'pi' }
 
        if (defined $theta) {
                $r = int($r + ($r < 0 ? -1 : 1) * $eps)
-                       if int(abs($r)) != int(abs($r) + $eps);
+                       if int(CORE::abs($r)) != int(CORE::abs($r) + $eps);
                $theta = int($theta + ($theta < 0 ? -1 : 1) * $eps)
                        if ($theta ne 'pi' and
-                           int(abs($theta)) != int(abs($theta) + $eps));
+                           int(CORE::abs($theta)) != int(CORE::abs($theta) + $eps));
                return "\[$r,$theta\]";
        }
 
@@ -1270,24 +1331,41 @@ sub stringify_polar {
        #
 
        $nt -= pit2 if $nt > pi;
-       my ($n, $k, $kpi);
 
-       for ($k = 1, $kpi = pi; $k < 10; $k++, $kpi += pi) {
+       if ($format{polar_pretty_print} && CORE::abs($nt) >= deg1) {
+           my ($n, $k, $kpi);
+
+           for ($k = 1, $kpi = pi; $k < 10; $k++, $kpi += pi) {
                $n = int($kpi / $nt + ($nt > 0 ? 1 : -1) * 0.5);
-               if (abs($kpi/$n - $nt) <= $eps) {
-                       $theta = ($nt < 0 ? '-':'').
-                                ($k == 1 ? 'pi':"${k}pi").'/'.abs($n);
-                       last;
+               if (CORE::abs($kpi/$n - $nt) <= $eps) {
+                   $n = CORE::abs($n);
+                   my $gcd = gcd($k, $n);
+                   if ($gcd > 1) {
+                       $k /= $gcd;
+                       $n /= $gcd;
+                   }
+                   next if $n > 360;
+                   $theta = ($nt < 0 ? '-':'').
+                            ($k == 1 ? 'pi':"${k}pi");
+                   $theta .= '/'.$n if $n > 1;
+                   last;
                }
+           }
        }
 
        $theta = $nt unless defined $theta;
 
        $r = int($r + ($r < 0 ? -1 : 1) * $eps)
-               if int(abs($r)) != int(abs($r) + $eps);
+               if int(CORE::abs($r)) != int(CORE::abs($r) + $eps);
        $theta = int($theta + ($theta < 0 ? -1 : 1) * $eps)
                if ($theta !~ m(^-?\d*pi/\d+$) and
-                   int(abs($theta)) != int(abs($theta) + $eps));
+                   int(CORE::abs($theta)) != int(CORE::abs($theta) + $eps));
+
+       my $format = $format{format};
+        if (defined $format) {
+           $r     = sprintf($format, $r);
+           $theta = sprintf($format, $theta);
+       }
 
        return "\[$r,$theta\]";
 }
@@ -1295,6 +1373,7 @@ sub stringify_polar {
 1;
 __END__
 
+=pod
 =head1 NAME
 
 Math::Complex - complex numbers and associated mathematical functions
@@ -1578,9 +1657,9 @@ It is possible to write:
 
        $x = cplxe(-3, pi/4);
 
-but that will be silently converted into C<[3,-3pi/4]>, since the modulus
-must be non-negative (it represents the distance to the origin in the complex
-plane).
+but that will be silently converted into C<[3,-3pi/4]>, since the
+modulus must be non-negative (it represents the distance to the origin
+in the complex plane).
 
 It is also possible to have a complex number as either argument of
 either the C<make> or C<emake>: the appropriate component of
@@ -1592,31 +1671,67 @@ the argument will be used.
 =head1 STRINGIFICATION
 
 When printed, a complex number is usually shown under its cartesian
-form I<a+bi>, but there are legitimate cases where the polar format
+style I<a+bi>, but there are legitimate cases where the polar style
 I<[r,t]> is more appropriate.
 
-By calling the routine C<Math::Complex::display_format> and supplying either
-C<"polar"> or C<"cartesian">, you override the default display format,
-which is C<"cartesian">. Not supplying any argument returns the current
-setting.
+By calling the class method C<Math::Complex::display_format> and
+supplying either C<"polar"> or C<"cartesian"> as an argument, you
+override the default display style, which is C<"cartesian">. Not
+supplying any argument returns the current settings.
 
 This default can be overridden on a per-number basis by calling the
 C<display_format> method instead. As before, not supplying any argument
-returns the current display format for this number. Otherwise whatever you
-specify will be the new display format for I<this> particular number.
+returns the current display style for this number. Otherwise whatever you
+specify will be the new display style for I<this> particular number.
 
 For instance:
 
        use Math::Complex;
 
        Math::Complex::display_format('polar');
-       $j = ((root(1, 3))[1];
-       print "j = $j\n";               # Prints "j = [1,2pi/3]
+       $j = (root(1, 3))[1];
+       print "j = $j\n";               # Prints "j = [1,2pi/3]"
        $j->display_format('cartesian');
        print "j = $j\n";               # Prints "j = -0.5+0.866025403784439i"
 
-The polar format attempts to emphasize arguments like I<k*pi/n>
-(where I<n> is a positive integer and I<k> an integer within [-9,+9]).
+The polar style attempts to emphasize arguments like I<k*pi/n>
+(where I<n> is a positive integer and I<k> an integer within [-9,+9]),
+this is called I<polar pretty-printing>.
+
+=head2 CHANGED IN PERL 5.6
+
+The C<display_format> class method and the corresponding
+C<display_format> object method can now be called using
+a parameter hash instead of just a one parameter.
+
+The old display format style, which can have values C<"cartesian"> or
+C<"polar">, can be changed using the C<"style"> parameter.  (The one
+parameter calling convention also still works.)
+
+There are two new display parameters.
+
+The first one is C<"format">, which is a sprintf()-style format
+string to be used for both parts of the complex number(s).  The
+default is C<undef>, which corresponds usually (this is somewhat
+system-dependent) to C<"%.15g">.  You can revert to the default by
+setting the format string to C<undef>.
+
+       # the $j from the above example
+
+       $j->display_format('format' => '%.5f');
+       print "j = $j\n";               # Prints "j = -0.50000+0.86603i"
+       $j->display_format('format' => '%.6f');
+       print "j = $j\n";               # Prints "j = -0.5+0.86603i"
+
+Notice that this affects also the return values of the
+C<display_format> methods: in list context the whole parameter hash
+will be returned, as opposed to only the style parameter value.  If
+you want to know the whole truth for a complex number, you must call
+both the class method and the object method:
+
+The second new display parameter is C<"polar_pretty_print">, which can
+be set to true or false, the default being true.  See the previous
+section for what this means.
 
 =head1 USAGE
 
@@ -1646,7 +1761,7 @@ Here are some examples:
 The division (/) and the following functions
 
        log     ln      log10   logn
-       tan     sec     csc     cot
+       tan     sec     csc     cot
        atan    asec    acsc    acot
        tanh    sech    csch    coth
        atanh   asech   acsch   acoth
@@ -1700,9 +1815,14 @@ All routines expect to be given real or complex numbers. Don't attempt to
 use BigFloat, since Perl has currently no rule to disambiguate a '+'
 operation (for instance) between two overloaded entities.
 
+In Cray UNICOS there is some strange numerical instability that results
+in root(), cos(), sin(), cosh(), sinh(), losing accuracy fast.  Beware.
+The bug may be in UNICOS math libs, in UNICOS C compiler, in Math::Complex.
+Whatever it is, it does not manifest itself anywhere else where Perl runs.
+
 =head1 AUTHORS
 
-Raphael Manfredi <F<Raphael_Manfredi@grenoble.hp.com>> and
+Raphael Manfredi <F<Raphael_Manfredi@pobox.com>> and
 Jarkko Hietaniemi <F<jhi@iki.fi>>.
 
 Extensive patches by Daniel S. Lewart <F<d-lewart@uiuc.edu>>.