[Octave-bug-tracker] [bug #65431] crash after hgload certain data

[Nicholas Jankowski]

Follow-up Comment #39, bug #65431 (group octave):

Regarding the numbers of points/faces/vertices, I tried to layout the
compatible behavior in comment #19, but it was a bit of a muddle. Ignoring the
rgb dimension:

>From the published support Markus linked, cdata should be a row vector of
length=numel(faces), or size=size(xdata) for vertices. (Rgb extends in dim3)

Facevertexcdata should be a column vector of length=1, length=numel_(faces),
or numel(vertices).

Strict adherence to published data would error on any other input, and
inputting CData or FaceVertexCData should result in the other being set in a
complementary way. 

It appears MATLAB accepts facevertexcdata formatted input for cdata.  It will
actually assign a column vector cdata to facevertexcdata instead, and set a
compatible cdata. I didn't check it the other way around .  

I'm mostly concerned that setting cdata changes vertices. That shouldn't
happen.  


We could choose to implement strict compatibility to the published behavior,
but we haven't had that to date and it may break 
code people expect to work. 
But as some other bugs have revealed we have little input validation for cdata
or fvc except for constraints defined in the patch class, adding that would be
a plus.


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?65431>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/