[Rosegarden-bugs] [Rosegarden Wiki] page changed: dev:layout_code

[<no...@ro...>]

A page in your DokuWiki was added or changed. Here are the details:

Date        : 2013/07/21 15:00
Browser     : Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Ubuntu Chromium/28.0.1500.52 Chrome/28.0.1500.52 Safari/537.36
IP-Address  : 76.100.236.26
Hostname    : c-76-100-236-26.hsd1.md.comcast.net
Old Revision: http://www.rosegardenmusic.com/wiki/dev:layout_code?rev=1374411599
New Revision: http://www.rosegardenmusic.com/wiki/dev:layout_code
Edit Summary: [Let Qt 4 manage your parents] this?  addWidget()
User        : tedfelix

@@ -50,9 +50,9 @@
      m_button = new QPushButton(tr("Button"), vBox);
      layout->addWidget(m_button);
  </code>
  
- All of this vBox begat frame begat QPushButton code can be removed for simplicity.  (QLayout::addItem() which is eventually called by this will take ownership of the widget.  So there is no danger of memory leaks.)  It is rarely necessary or beneficial to supply all these optional parent parameters.  This has many practical advantages, and has been greatly helpful to me in resolving broken layouts during this restructuring work.  If an existing layout ain't broke, there is little point in fixing it, but please do not write code like this in new layouts.  The example above will work perfectly fine when reduced to:
+ All of this vBox begat frame begat QPushButton code can be removed for simplicity.  (QLayout::addItem() which is eventually called by addWidget() will take ownership of the widget.  So there is no danger of memory leaks.)  It is rarely necessary or beneficial to supply all these optional parent parameters.  This has many practical advantages, and has been greatly helpful to me in resolving broken layouts during this restructuring work.  If an existing layout ain't broke, there is little point in fixing it, but please do not write code like this in new layouts.  The example above will work perfectly fine when reduced to:
  
  <code c++>
      QFrame *frame = new QFrame;
      vBoxLayout->addWidget(frame);



-- 
This mail was generated by DokuWiki at
http://www.rosegardenmusic.com/wiki/