[maemo-developers] QT4.6 with PR1.2 has introduced issues

[From: Ram Kurvakat]

"In proper place, of course: QDialog *dialog = new QDialog; dialog->show(); int ret = app.exec(); delete dialog; return ret; But in fact it's much easier to allocate dialog statically." 

Hi Daniil, the example only contains a main function, there is no need for that delete, its quite redundant as the heap is cleared anyways after the return and app exits in the example ,is it not ?It would have been necessary had this been allocated in some custom class. cheerskrk969
----- Original Message -----
From: Daniil Ivanov
Sent: 06/01/10 11:34 AM
To: Ram Kurvakat
Subject: Re: QT4.6 with PR1.2 has introduced issues

On Tue, Jun 1, 2010 at 1:16 PM, Ram Kurvakat <rkmaemo at gmx.com> wrote: > > ----- Original Message ----- > > From: Daniil Ivanov > > Sent: 06/01/10 10:44 AM > > To: Ram Kurvakat > > Subject: Re: QT4.6 with PR1.2 has introduced issues > > On Tue, Jun 1, 2010 at 12:00 PM, Ram Kurvakat <rkmaemo at gmx.com> wrote: > > > -- > Sent from my Nokia N900 > > ----- Original message ----- >> Hi Ram! >> >>> Could you please post this questions to FN DiBo? > A > Hi Daniil, yes > of course i can. > I was asked for examples here so i replied. > FN DiBo = > forum nokia discussion board ? Yes. > > FN DiBo post as requested ==> > http://discussion.forum.nokia.com/forum/showthread.php?p=742765 > > Also took the liberty to raise a bug :) > > http://bugreports.qt.nokia.com/browse/QTBUG-11135 > > >> BTW, why do create dialog dynamically and never free it? > > This is > an example only to show the issue. This is basically a bad habit. > > Of course Im aware Daniil :-) thanks anyways , > > but now since y
 ou state that where do you expect to see the delete of the > dialog in this main function ;-) In proper place, of course: QDialog *dialog = new QDialog; dialog->show(); int ret = app.exec(); delete dialog; return ret; But in fact it's much easier to allocate dialog statically. Thanks, Daniil. > > Thanks, Daniil. >> Thanks, Daniil. >> >> On Tue, Jun 1, 2010 at 10:59 AM, > Ram Kurvakat <rkmaemo at gmx.com> wrote: >> > Daniil, Robin or anyone >> > >> > > did you get a chance to look at this one please ? >> > >> > thanks for your > time, >> > >> > cheers >> > >> > krk969 >> > >> > ----- Original Message > ----- >> > >> > From: Ram Kurvakat >> > >> > Sent: 05/29/10 10:32 AM >> > >> >> To: Maemo-developers, viroteck at viroteck.net, Daniil Ivanov >> > >> > > Subject: Re: QT4.6 with PR1.2 has introduced issues >> > >> > Here is the > example requested. >> > >> > Compile the example with qmake from > /opt/qt4-maemo5/bin which is QT4.6 >> > in PR1.1 and run on the device with > QT4.6 libs 
 from PR1.1 installed in >> > /opt/qt4-maemo5/lib >> > >> > ~$: > ldd TEMP2 >> > >> > .... >> > >> > libQtGui.so.4 => > /opt/qt4-maemo5/lib/libQtGui.so.4 (0x40142000) >> > >> > libQtCore.so.4 => > /opt/qt4-maemo5/lib/libQtCore.so.4 (0x40bfd000) >> > >> > .... >> > >> > and > this is the result >> > >> > http://bit.ly/bH2ApK >> > >> > Then >> > >> > > Compile the example with qmake from /usr/bin which is QT4.6 in PR1.2 >> > > and run on the device with QT4.6 libs from PR1.2 installed in /usr/lib >> > >>> > ~$: ldd TEMP2 >> > >> > .... >> > >> > libQtGui.so.4 => > /usr/lib/libQtGui.so.4 (0x40139000) >> > >> > libQtCore.so.4 => > /usr/lib/libQtCore.so.4 (0x40bf4000) >> > >> > .... >> > >> > and you will > see this >> > >> > http://bit.ly/bIpRi0 >> > >> > I think some default > alignments may have been changed in the styles. >> > >> > Thanks again for > taking a look. >> > >> > #include <QtGui> >> > >> > #include <QApplication> >>> > >> > int main(int argc, char* argv[]) >> > >> 
 > { >> > >> > > QApplication app(argc, argv); >> > >> > QDialog *dialog = new QDialog; >>> > >> > QScrollArea *scroll = new QScrollArea; >> > >> > QWidget* > widget = new QWidget(scroll); >> > >> > // Currency code >> > >> > > QLineEdit *currency = new QLineEdit(dialog); >> > >> > // Date format >>> > >> > QStringList dateformatlist; >> > >> > dateformatlist << > "dd-MM-yyyy" << "MM-dd-yyyy" << "dd-MMM-yyyy" << >> > "MMM-dd-yyyy"; >> > >> >> QComboBox *dateformat = new QComboBox; >> > >> > // Graph Period >>> > >> > QStringList periodlist; >> > >> > periodlist << "current > day" << "current month" << "current week" << >> > "current year"; >> > >> > > QComboBox *graphperiod = new QComboBox; >> > >> > > dialog->setStyleSheet("QDialog { border: 2px solid #8f8f91; ; >> > > border-radius: 15px; }"); >> > >> > QFormLayout *layout = new > QFormLayout; >> > >> > layout->addRow("currency code : ", currency); >> >> >> > layout->addRow("display date format : ", dateformat); >> > >> > >
  layout->addRow("default reports period: ", graphperiod); >> > >> > > widget->setLayout(layout); >> > >> > QHBoxLayout *mainlayout = new > QHBoxLayout; >> > >> > scroll->setWidget(widget); >> > >> > > mainlayout->addWidget(scroll, 0, Qt::AlignVCenter); >> > >> > > dialog->setLayout(mainlayout); >> > dialog->show(); >> > return > app.exec(); >> > } >> > >> > ----- Original Message ----- >> > >> > From: > Robin Burchell >> > >> > Sent: 05/28/10 10:36 AM >> > >> > To: Daniil Ivanov >>> > >> > Subject: Re: QT4.6 with PR1.2 has introduced issues >> > >> > On > Fri, May 28, 2010 at 7:25 AM, Daniil Ivanov >> > <daniil.ivanov at gmail.com> > wrote: > Hi Ram! > > Screenshots do not say >> > anything. > Without > seeing the code it's not possible to say if it's >> > because of > problems > with the way how you constructed layouts of the >> > widgets > or with Qt > styles. Seconding what Daniil says - to try >> > pinpoint your issue (and > look for workarounds or fixes), please >> > produ
 ce a minimal testcase. > That is, a *minimal* example (as little >> > code as possible, please! can't > emphasise how important this is) which >> > demonstrates the incorrect > behaviour you are talking about. :) > >> > Thanks, Daniil. Best, Robin > Burchell mob: +447702671419 msn: >> > msn at viroteck.net irc: w00t @ > irc.freenode.net twr: >> > http://twitter.com/w00teh lac: > http://identi.ca/w00t >> > >> > >> > >> > >> > >> > >> > >> > > > > > > >
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.maemo.org/pipermail/maemo-developers/attachments/20100601/f5adb98b/attachment.htm>