[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Minor OOP example fix.
From: |
Jaroslav Hajek |
Subject: |
Re: Minor OOP example fix. |
Date: |
Wed, 3 Jun 2009 07:08:43 +0200 |
On Wed, Jun 3, 2009 at 5:59 AM, Robert T. Short
<address@hidden> wrote:
> Jaroslav,
>
> The changes you made are better than mine, but also broke the end method.
> Fix attached.
>
> Thanks.
>
> Bob
>
>
> # HG changeset patch
> # User Robert T. Short <address@hidden>
> # Date 1244001379 25200
> # Node ID 74e09577d4ab537364c2b5ef6b625fc440b7811d
> # Parent 56ce6a69836fca989f7022ffac3fd9a27d91824e
> * examples/@polynomial/end.m: Fixe end for zero based indexing.
>
> diff -r 56ce6a69836f -r 74e09577d4ab ChangeLog
> --- a/ChangeLog Tue Jun 02 16:57:22 2009 -0700
> +++ b/ChangeLog Tue Jun 02 20:56:19 2009 -0700
> @@ -1,3 +1,7 @@
> +2009-06-02 Robert T. Short <address@hidden>
> +
> + * examples/@polynomial: Fix end method for zero based indexing.
> +
> 2009-06-02 Jaroslav Hajek <address@hidden>
>
> * examples/@polynomial/subsref.m: Allow chained subscripts,
> diff -r 56ce6a69836f -r 74e09577d4ab examples/@polynomial/end.m
> --- a/examples/@polynomial/end.m Tue Jun 02 16:57:22 2009 -0700
> +++ b/examples/@polynomial/end.m Tue Jun 02 20:56:19 2009 -0700
> @@ -4,6 +4,6 @@
> error ("polynomial object may only have one index")
> endif
>
> - r = length (obj.poly);
> + r = length (obj.poly) - 1;
>
> endfunction
>
>
Ah, yes, sorry. I shot myself in the foot. Thanks for catching this. Applied.
--
RNDr. Jaroslav Hajek
computing expert & GNU Octave developer
Aeronautical Research and Test Institute (VZLU)
Prague, Czech Republic
url: www.highegg.matfyz.cz