[Octave-bug-tracker] [bug #31536] demos for patch and isosurface fail

[David Bateman]

Follow-up Comment #2, bug #31536 (project octave):

Opps, I was trying the address the code like

patch(..., "cdata", c1, ...., "cdata", c2, ....)

where cdata should be taken to be c2. I didn't treat the case where its not
at all defined. I like Rik proposal, but it (and Ben's proposal for tht
matter) abuses cs-lists to get the right functionally. Consider a code snippet
like

c = {[1,2,3],[4,5,6]}
c{[2,1]}
cdata = c{[2,1]}

The line c{[2,1]} returns a two element cs-list, whereas  cdata=c{[1,2]} only
has a single argument on the LHS and so the second element of the cs-list is
ignored. Its a bit of a nasty trick in my opinion to get the right behavior,
but without making the code much more complex I don't see a way of getting the
right behavior and avoiding indexing into null arrays. I'd say push a
changeset based on Rik's suggestion

D.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?31536>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/