octave-maintainers
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Patch for erfinv.m


From: John W. Eaton
Subject: Re: Patch for erfinv.m
Date: Tue, 22 Jan 2008 16:34:31 -0500

On 21-Jan-2008, Alois Schloegl wrote:

| >> Consider the following patch for erfinv.
| >>
| >> It (1) replaces z_old and z_new by a single variable z, and
| >> (2) makes the initial checks simpler.
| >>
| >> The code becomes leaner, and a bit faster.

I applied the patch but used

  ## x < 1 or x > 1 ==> NaN
  y(abs (x) >= 1) = NaN;

instead of

| +  y(~(abs(x) < 1)) = NaN;  %% x<1, x>1, x=NaN

Any reason not to make this change?

Thanks,

jwe


reply via email to

[Prev in Thread] Current Thread [Next in Thread]