[Octave-patch-tracker] [patch #9084] uitable implementation

[Noname]

Follow-up Comment #25, patch #9084 (project octave):

Well - if we write m_curData after Table::Table() somewhere at line 444 by
inserting "m_curData = octave_value(tp.get_data ());":

Table::Table (const graphics_object& go, QTableWidget* tableWidget)
    : Object (go, tableWidget), m_tableWidget(tableWidget), m_curData(),
      m_blockUpdates(false)
  {
    uitable::properties& tp = properties<uitable> ();
    m_curData = octave_value(tp.get_data ());
    Matrix bb = tp.get_boundingbox (false);

the problem dissappears.

However i am unsure about the consequences: m_curData is a private. Where is
m_curData() coming from and at which point (file, line) is Table::Table()
instantiated? I personally would expect m_curData should be set as the object
is instantianated.

So for those who want to give it a try i created a patch from my local HG
directory. This patch is identical to  patch_9084_uitable-20170102.diff but
refers to HG ID 9a9b2e0596f5ab013b5180177d4d010934f58c7b. I havent included
the above mentioned line because i cannot say wheather this a "bad-style" fix.
The only contribution i made is the removal of the deprecated warnings by
changing

is_complex_type -> iscomplex
is_bool_type -> islogical
is_numeric_type -> isnumeric
is_integer_type -> isinteger
is_empty -> isempty
is_cell -> iscell

The patch compiles with QT5 and QT4 without any warnings (Fedora 25).

Best regards HG200

(file #42724)
    _______________________________________________________

Additional Item Attachment:

File name: patch_9084_uitable-20171223_nowarnings.diff Size:102 KB


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?9084>

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