octave-maintainers
[Top][All Lists]
Advanced

[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



reply via email to

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