make contourf work (again)

[John W. Eaton]

On  6-Sep-2008, Kai Habel wrote:

| Hello,
| 
| with very recent checkout I see:
| 
| octave:1> contourf(peaks)
| warning: meaning may have changed due to change in precedence for && and 
| || operators
| error: patch: X and Y must be of same size
| error: called from:
| error:   /home/kai/hg-octave/octave/scripts/plot/contourf.m at line 234, 
| column 5
| error:   /home/kai/hg-octave/octave/scripts/plot/contourf.m at line 67, 
| column 32
| octave:1>
| 
| I had to apply the attached fix
| 
| Kai
| # HG changeset patch
| # User address@hidden
| # Date 1220704799 -7200
| # Node ID e6892ff5a9cf40285ed007e4d4cb6b2e4115fab9
| # Parent  46aa040553254660894f13bf131688ca8d384527
| Make contourf work for equal-size matrices (again).
| 
| diff -r 46aa04055325 -r e6892ff5a9cf scripts/ChangeLog
| --- a/scripts/ChangeLog       Wed Sep 03 11:40:55 2008 -0400
| +++ b/scripts/ChangeLog       Sat Sep 06 14:39:59 2008 +0200
| @@ -1,3 +1,8 @@
| +2008-03-14  Kai Habel  <address@hidden>
| +
| +        * plot/contourf.m: Fix case for equal-sized matrices
| +        of X and Y.
| +
|  2008-09-02  David Bateman  <address@hidden>
|  
|       * plot/__add_datasource__.m: Fix indexing of varargin.
| diff -r 46aa04055325 -r e6892ff5a9cf scripts/plot/contourf.m
| --- a/scripts/plot/contourf.m Wed Sep 03 11:40:55 2008 -0400
| +++ b/scripts/plot/contourf.m Sat Sep 06 14:39:59 2008 +0200
| @@ -227,7 +227,7 @@
|      arg(1:4) = [];
|    endif
|  
| -  if (!isvector (X) || !isvector (Y) && any (size (X) != size (Y)))
| +  if ((!isvector (X) || !isvector (Y)) && !size_equal(X,Y))
|      error ("patch: X and Y must be of same size")
|    endif

So many nots!  Maybe it would be clearer to write

  if (! ((isvector (X) && isvector (Y)) || size_equal (X, Y)))

?

jwe