octave-patch-tracker
[Top][All Lists]
Advanced

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

[Octave-patch-tracker] [patch #8801] add support for the histogram and h


From: Philip Nienhuis
Subject: [Octave-patch-tracker] [patch #8801] add support for the histogram and histcounts functions
Date: Sun, 20 Nov 2016 20:25:58 +0000 (UTC)
User-agent: Mozilla/5.0 (X11; Linux i686 on x86_64; rv:43.0) Gecko/20100101 Firefox/43.0 SeaMonkey/2.40

Follow-up Comment #2, patch #8801 (project octave):

Q:
In histcounts I see this stanza in L.201-205:

        case "BinWidth"
          binwidth = varargin{++iarg};
          if (! isreal (binmethod))
            error ("histcounts: BinWidth must be followed by a real number");
          endif


Shouldn't the if statement check if "binwidth" rather than "binmethod" is
real?

(looks like a copy-paste from he switch clause above)

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?8801>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/




reply via email to

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