octave-patch-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Octave-patch-tracker] [patch #8824] [octave forge] (image) added affine


From: Hartmut
Subject: [Octave-patch-tracker] [patch #8824] [octave forge] (image) added affine2d class
Date: Tue, 28 Dec 2021 09:55:57 -0500 (EST)
User-agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:95.0) Gecko/20100101 Firefox/95.0

Follow-up Comment #17, patch #8824 (project octave):

Thanks for preparing this new version of the three files, Avinoam. I looked
over them, and this looks very good.

Some minor points that came up are:
* In the help string of affine2d and affine3d there is one occasion each of
"isRigit". This is misspelled and should probably read "isRigid" with a D
instead of the T.
* in affine3d it would be nice to also have one blank line right before the
3-line comment of the function outputLimits.

After improving these tiny points, I would be happy to see those three files
pushed to the image repository, finally.



    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/patch/?8824>

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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]