[bug #68108] classdef objects do not support creation of initialized arrays
Markus Mützel <[email protected]>
| Newsgroups | gmane.comp.gnu.octave.bugs |
|---|---|
| Message-ID | <[email protected]> |
Please use the bug tracker to post updates to a bug report. The mailing list is intended as a read-only notification stream. Info posted to this mailing list address won't appear in the tracker database where it is most useful.
Update of bug #68108 (group octave):
Status: None => Need Info
_______________________________________________________
Follow-up Comment #3:
Thank you for your contribution.
I haven't tried to build with your proposed changes yet. Just a few of remarks
and questions for now:
* Why are you copying `rhs` to `t_rhs`? Wouldn't it be possible to directly
use `rhs`?
* If `rhs can be used directly, would it make sense to check `is_cs_list()`
already in the `else if`-condition of the new branch that you added?
* I'm not sure I understand what "nonbraced" means in the error message. Could
you please elaborate?
* The comments should describe what the code does - not what you changed in
the diff. The latter should be part of a commit message.
* I don't think that you need to create a new directory for `mre.m` (maybe
also rename that to something a bit more specific). Putting that file into
`test/classdef` should be fine. That would also allow to simplify the test
that you wrote a bit (no need to add or remove paths).
* You'll need to add the new files to `test/classdef/module.mk` (or they would
be missing in a release tarball).
* We don't have an official AI policy yet. So, just out of interest: Did you
use some AI tools (LLMs) to help generate the changes? If yes, to what
extend?
Would the change also fix the following example?
m(1,3) = mre ();
[m.A] = deal (1, 2, 3);
_______________________________________________________
Reply to this item at:
<https://savannah.gnu.org/bugs/?68108>
_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/
signature.asc
(application/pgp-signature, 228 B)
-----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQQk97aszIMMAvLLwm6qLAuaBUf3TgUCaaWOtAAKCRCqLAuaBUf3 TqDCAQDNN1iULw2kvgQSUfsCA3Alc2GSWwg1050LKfO9zI+rkwEA8l/eEPSSfH7G BzYlUfNvo8sd8tytuWmVV3rtf8gKiwM= =8ahp -----END PGP SIGNATURE-----