Re: Patch to hist for changes in bar

[David Bateman]

John W. Eaton wrote:
> On 26-Apr-2007, David Bateman wrote:
> 
> | John W. Eaton wrote:
> | >   ***** assert(hist(1,1),1);
> | > !!!!! test failed
> | > error: invalid vector index = 0
> | >   ***** test
> | >   for n = [10, 30, 100, 1000]
> | >     assert( sum(hist([1:n], n)), n );
> | >     assert( sum(hist([1:n], [2:n-1])), n);
> | >     assert( sum(hist([1:n], [1:n])), n );
> | >     assert( sum(hist([1:n], 29)), n);
> | >     assert( sum(hist([1:n], 30)), n);
> | >   endfor
> | > !!!!! test failed
> | > error: assert (sum (hist ([1:n], n)),n) expected
> | >  10
> | > but got
> | > -2
> | > values do not match
> | >
> | > jwe
> | >
> | >   
> | Grrr, ok.. In fact it seems I was wrong in that matlab doesn't ever
> | return a handle for the hist function. The following patch pretty much
> | undoes the change to hist.m and restores the matlab compatible
> | behavior.. Note though that it keeps the line width to the call to bar
> | from the previous patch..
> 
> OK, please check it in.
> 
> Thanks,
> 
> jwe
> 
Ok, done..

D.