[Octave-bug-tracker] [bug #52989] classdef: missing error messages on multiply defined properties

[A.R. Burgers]

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

refresh of this patch for octave-6.x.x:


diff -r 62c1443babd8 libinterp/octave-value/cdef-class.cc
--- a/libinterp/octave-value/cdef-class.cc      Sun Mar 10 17:02:27 2019
-0700
+++ b/libinterp/octave-value/cdef-class.cc      Mon Mar 11 19:57:31 2019
+0100
@@ -1062,6 +1062,8 @@
                 for (auto& prop_p : *pb_p->element_list ())
                   {
                     std::string prop_name = prop_p->ident ()->name ();
+                    if (retval.find_property(prop_name).ok())
+                       error ("property %s already defined",
prop_name.c_str());

                     cdef_property prop = cdm.make_property (retval,
prop_name);



    _______________________________________________________

Reply to this item at:

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

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