Re: Patch to hist for changes in bar

[John W. Eaton]

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