QT4.2 corrupted double-linked list and QRadioButton existance

[pdolbey]

Lets see if I undertstand this code.

iconv_t cd = createIconv_t(UTF16, 0);

iconv_t cd = createIconv_t(UTF16, 0);

To copy to clipboard, switch view to plain text mode 

In reality your traceback implies you mean

iconv_t cd = createIconv_t("UTF-16", NULL);

iconv_t cd = createIconv_t("UTF-16", NULL);

To copy to clipboard, switch view to plain text mode 

yet you get

0xb7696094 in QIconvCodec::createIconv_t (to=0xb76c0ced "UTF-16",    
from=0x6 <Address 0x6 out of bounds>) at codecs/qiconvcodec.cpp:227

0xb7696094 in QIconvCodec::createIconv_t (to=0xb76c0ced "UTF-16",    
from=0x6 <Address 0x6 out of bounds>) at codecs/qiconvcodec.cpp:227

To copy to clipboard, switch view to plain text mode 

Why do you get 0x6 then? I suggest you re-try using the NULL macro instead of 0.

Pete

[mtrpoland]

So yo advise me to change the line:

iconv_t cd = createIconv_t(UTF16, 0);

iconv_t cd = createIconv_t(UTF16, 0);

To copy to clipboard, switch view to plain text mode 

that is placed in some qt file
to

iconv_t cd = createIconv_t(UTF16, NULL);

iconv_t cd = createIconv_t(UTF16, NULL);

To copy to clipboard, switch view to plain text mode 

??
Just don't get the idea why my actions that are actually a commonplace (initialization of two QRadioButton's and its usage) entail a problem I presented above.
It's quite bizarre.

[jacek]

Just don't get the idea why my actions that are actually a commonplace (initialization of two QRadioButton's and its usage) entail a problem I presented above.
It's quite bizarre.

Yes, it is. Can you reproduce that stack trace?

[mtrpoland]

What do you mean by "reproducing the stack trace" ?

[mtrpoland]

Guys, I strove to circumvent the issue by placing one checkbox instead of two radios. It's like an ufo... Now the program works but I get 'core dumped' after having exited the application. So I assume there is a problem with destruction...

(gdb) backtrace
#0  0xffffe410 in __kernel_vsyscall ()
#1  0xb720a770 in raise () from /lib/tls/i686/cmov/libc.so.6
#2  0xb720bef3 in abort () from /lib/tls/i686/cmov/libc.so.6
#3  0xb723fd0b in __fsetlocking () from /lib/tls/i686/cmov/libc.so.6
#4  0xb72478bd in mallopt () from /lib/tls/i686/cmov/libc.so.6
#5  0xb7247a44 in free () from /lib/tls/i686/cmov/libc.so.6
#6  0xb7536e5d in QHashData::freeNode (this=0x8112498, node=0x8124af8)
    at tools/qhash.cpp:132
#7  0xb7991abd in QHash<unsigned long, QWidget*>::remove (this=0x8108d30,
    akey=@0x812498c)
    at ../../include/QtCore/../../src/corelib/tools/qhash.h:450
#8  0xb7982eff in QWidgetPrivate::setWinId (this=0x8124950, id=0)
    at kernel/qwidget.cpp:1180
#9  0xb79b8dcc in QWidget::destroy (this=0x81248b8, destroyWindow=true,
    destroySubWindows=true) at kernel/qwidget_x11.cpp:782
#10 0xb798866c in ~QWidget (this=0x81248b8) at kernel/qwidget.cpp:1162
#11 0xb7bd7ef1 in ~QAbstractButton (this=0x81248b8)
    at widgets/qabstractbutton.cpp:572
#12 0xb7c846e5 in ~QToolButton (this=0x81248b8) at widgets/qtoolbutton.cpp:347
#13 0xb75d4ea1 in QObjectPrivate::deleteChildren (this=0x8123978)
    at kernel/qobject.cpp:1823
#14 0xb7988649 in ~QWidget (this=0x8123938) at kernel/qwidget.cpp:1158
#15 0xb7c7e844 in ~QToolBar (this=0x8123938) at widgets/qtoolbar.cpp:472
---Type <return> to continue, or q <return> to quit---
#16 0xb75d4ea1 in QObjectPrivate::deleteChildren (this=0x810b300)
    at kernel/qobject.cpp:1823
#17 0xb7988649 in ~QWidget (this=0xbf85ebf0) at kernel/qwidget.cpp:1158
#18 0xb7c30c21 in ~QMainWindow (this=0xbf85ebf0) at widgets/qmainwindow.cpp:292
#19 0x080527ca in main ()

(gdb) backtrace
#0  0xffffe410 in __kernel_vsyscall ()
#1  0xb720a770 in raise () from /lib/tls/i686/cmov/libc.so.6
#2  0xb720bef3 in abort () from /lib/tls/i686/cmov/libc.so.6
#3  0xb723fd0b in __fsetlocking () from /lib/tls/i686/cmov/libc.so.6
#4  0xb72478bd in mallopt () from /lib/tls/i686/cmov/libc.so.6
#5  0xb7247a44 in free () from /lib/tls/i686/cmov/libc.so.6
#6  0xb7536e5d in QHashData::freeNode (this=0x8112498, node=0x8124af8)
    at tools/qhash.cpp:132
#7  0xb7991abd in QHash<unsigned long, QWidget*>::remove (this=0x8108d30,
    akey=@0x812498c)
    at ../../include/QtCore/../../src/corelib/tools/qhash.h:450
#8  0xb7982eff in QWidgetPrivate::setWinId (this=0x8124950, id=0)
    at kernel/qwidget.cpp:1180
#9  0xb79b8dcc in QWidget::destroy (this=0x81248b8, destroyWindow=true,
    destroySubWindows=true) at kernel/qwidget_x11.cpp:782
#10 0xb798866c in ~QWidget (this=0x81248b8) at kernel/qwidget.cpp:1162
#11 0xb7bd7ef1 in ~QAbstractButton (this=0x81248b8)
    at widgets/qabstractbutton.cpp:572
#12 0xb7c846e5 in ~QToolButton (this=0x81248b8) at widgets/qtoolbutton.cpp:347
#13 0xb75d4ea1 in QObjectPrivate::deleteChildren (this=0x8123978)
    at kernel/qobject.cpp:1823
#14 0xb7988649 in ~QWidget (this=0x8123938) at kernel/qwidget.cpp:1158
#15 0xb7c7e844 in ~QToolBar (this=0x8123938) at widgets/qtoolbar.cpp:472
---Type <return> to continue, or q <return> to quit---
#16 0xb75d4ea1 in QObjectPrivate::deleteChildren (this=0x810b300)
    at kernel/qobject.cpp:1823
#17 0xb7988649 in ~QWidget (this=0xbf85ebf0) at kernel/qwidget.cpp:1158
#18 0xb7c30c21 in ~QMainWindow (this=0xbf85ebf0) at widgets/qmainwindow.cpp:292
#19 0x080527ca in main ()

To copy to clipboard, switch view to plain text mode 

The code was altered in mymvc.h:

QGridLayout* qvb;
	QLabel* label;
	QCheckBox* checkOne;
	bool caseSensitive;
};
 
#endif

QGridLayout* qvb;
	QLabel* label;
	QCheckBox* checkOne;
	bool caseSensitive;
};

#endif

To copy to clipboard, switch view to plain text mode 

and in mymvc.cpp:

label = new QLabel(tr("Enter a character string below"));
checkOne = new QCheckBox(tr("Case Sensitive"));
checkOne->setChecked(true);
caseSensitive = true;
qvb = new QGridLayout();
	connect(checkOne,SIGNAL(toggled(bool)),this,SLOT(setCaseSensitive(bool)));
connect(qcb,SIGNAL(currentIndexChanged(int)),le,SLOT(clear()));
connect(qcb,SIGNAL(currentIndexChanged(int)),qtv,SLOT(filter()));
connect(le,SIGNAL(textChanged(const QString &)),qtv,SLOT(filter()));
   connect(model,SIGNAL(fromTVtoVM(int)),this,SIGNAL(doubleClickInTreeView(int)));
 
qvb->addWidget(label,0,0,1,2);
qvb->addWidget(le,1,0);
qvb->addWidget(checkOne,1,1);
qvb->addWidget(qcb,2,0,1,2);
qvb->addWidget(qtv,3,0,1,2);
   setLayout(qvb);

label = new QLabel(tr("Enter a character string below"));
checkOne = new QCheckBox(tr("Case Sensitive"));
checkOne->setChecked(true);
caseSensitive = true;
qvb = new QGridLayout();
	connect(checkOne,SIGNAL(toggled(bool)),this,SLOT(setCaseSensitive(bool)));
connect(qcb,SIGNAL(currentIndexChanged(int)),le,SLOT(clear()));
connect(qcb,SIGNAL(currentIndexChanged(int)),qtv,SLOT(filter()));
connect(le,SIGNAL(textChanged(const QString &)),qtv,SLOT(filter()));
   connect(model,SIGNAL(fromTVtoVM(int)),this,SIGNAL(doubleClickInTreeView(int)));

qvb->addWidget(label,0,0,1,2);
qvb->addWidget(le,1,0);
qvb->addWidget(checkOne,1,1);
qvb->addWidget(qcb,2,0,1,2);
qvb->addWidget(qtv,3,0,1,2);
   setLayout(qvb);

To copy to clipboard, switch view to plain text mode 

[jacek]

What do you mean by "reproducing the stack trace" ?

Try running your application again and see whether the stack trace is the same.

[jacek]

Why do you get 0x6 then? I suggest you re-try using the NULL macro instead of 0.

The problematic code appears to be a part of Qt and it seems perfectly valid. Furthermore there's an assertion in QIconvCodec::createIconv_t() that should detect this situation.


BTW. There is no such thing as NULL macro in C++, it's defined only for compatibility with C.

[pdolbey]

BTW. There is no such thing as NULL macro in C++, it's defined only for compatibility with C.

Apparently (and I've only just googled for this)

The standard says that NULL exists, but is implementation-defined.

Section 18.1, line 4 :

"The macro NULL is an implementation-defined C + + null pointer constant
in this International Standard (4.10)."

It also mentions that it is defined in <cstddef>.

Section 4.10 starts by defining a "null pointer constant" :

"A null pointer constant is an integral constant expression (5.19)
rvalue of integer type that evaluates to zero."

In C++ NULL is typically either 0 or 0L (but not ((void*)0) - I was basically suggesting replacing 0 with NULL as I couldn't see any logical reason why the function call should transform '0' into '0x6' unless it was an compiler "implementation" gotcha. I'd always prefer NULL over 0 but thats because I think its more readable, but there seems to be many opinions on this matter.

I didn't realise how deep this problem was in the Qt4 source code as I don't remember seeing the class QIconvCodec in the public API - just found it in the source tree.

Pete

[jacek]

It also mentions that it is defined in <cstddef>.

Exactly, as I said, it's only for compatibility with C. In the new C++ standard, there will be a nullptr constant, but for now the first paragraph in section 4.10 says: "A null pointer constant is an integral constant expression (5.19) rvalue of integer type that evaluates to zero."

I couldn't see any logical reason why the function call should transform '0' into '0x6' unless it was an compiler "implementation" gotcha.

It could be, but 0 is used as a null pointer in many places in Qt sources, so it shouldn't be a problem.

I'd always prefer NULL over 0 but thats because I think its more readable, but there seems to be many opinions on this matter.

It also causes some problems with templates.

I don't remember seeing the class QIconvCodec in the public API

Neither did I. You can learn new things about Qt every day.

[mtrpoland]

Being desperate I decided to check whether the flawless QLabel evokes the problem and now I dare say I was trounced.
Why with two QLabels I receive an error and with one all is ok????

Here you have backtrace at 21:59 LT:

(gdb) backtrace
#0  0xffffe410 in __kernel_vsyscall ()
#1  0xb72c4770 in raise () from /lib/tls/i686/cmov/libc.so.6
#2  0xb72c5ef3 in abort () from /lib/tls/i686/cmov/libc.so.6
#3  0xb72f9d0b in __fsetlocking () from /lib/tls/i686/cmov/libc.so.6
#4  0xb73018bd in mallopt () from /lib/tls/i686/cmov/libc.so.6
#5  0xb7301a44 in free () from /lib/tls/i686/cmov/libc.so.6
#6  0xb75f0e5d in QHashData::freeNode (this=0x8112498, node=0x8124af8)
    at tools/qhash.cpp:132
#7  0xb7a4babd in QHash<unsigned long, QWidget*>::remove (this=0x8108d30,
    akey=@0x812498c)
    at ../../include/QtCore/../../src/corelib/tools/qhash.h:450
#8  0xb7a3ceff in QWidgetPrivate::setWinId (this=0x8124950, id=0)
    at kernel/qwidget.cpp:1180
#9  0xb7a72dcc in QWidget::destroy (this=0x81248b8, destroyWindow=true,
    destroySubWindows=true) at kernel/qwidget_x11.cpp:782
#10 0xb7a4266c in ~QWidget (this=0x81248b8) at kernel/qwidget.cpp:1162
#11 0xb7c91ef1 in ~QAbstractButton (this=0x81248b8)
    at widgets/qabstractbutton.cpp:572
#12 0xb7d3e6e5 in ~QToolButton (this=0x81248b8) at widgets/qtoolbutton.cpp:347
#13 0xb768eea1 in QObjectPrivate::deleteChildren (this=0x8123978)
    at kernel/qobject.cpp:1823
#14 0xb7a42649 in ~QWidget (this=0x8123938) at kernel/qwidget.cpp:1158
#15 0xb7d38844 in ~QToolBar (this=0x8123938) at widgets/qtoolbar.cpp:472
---Type <return> to continue, or q <return> to quit---
#16 0xb768eea1 in QObjectPrivate::deleteChildren (this=0x810b300)
    at kernel/qobject.cpp:1823
#17 0xb7a42649 in ~QWidget (this=0xbfec0a50) at kernel/qwidget.cpp:1158
#18 0xb7ceac21 in ~QMainWindow (this=0xbfec0a50) at widgets/qmainwindow.cpp:292
#19 0x080527ca in main ()

(gdb) backtrace
#0  0xffffe410 in __kernel_vsyscall ()
#1  0xb72c4770 in raise () from /lib/tls/i686/cmov/libc.so.6
#2  0xb72c5ef3 in abort () from /lib/tls/i686/cmov/libc.so.6
#3  0xb72f9d0b in __fsetlocking () from /lib/tls/i686/cmov/libc.so.6
#4  0xb73018bd in mallopt () from /lib/tls/i686/cmov/libc.so.6
#5  0xb7301a44 in free () from /lib/tls/i686/cmov/libc.so.6
#6  0xb75f0e5d in QHashData::freeNode (this=0x8112498, node=0x8124af8)
    at tools/qhash.cpp:132
#7  0xb7a4babd in QHash<unsigned long, QWidget*>::remove (this=0x8108d30,
    akey=@0x812498c)
    at ../../include/QtCore/../../src/corelib/tools/qhash.h:450
#8  0xb7a3ceff in QWidgetPrivate::setWinId (this=0x8124950, id=0)
    at kernel/qwidget.cpp:1180
#9  0xb7a72dcc in QWidget::destroy (this=0x81248b8, destroyWindow=true,
    destroySubWindows=true) at kernel/qwidget_x11.cpp:782
#10 0xb7a4266c in ~QWidget (this=0x81248b8) at kernel/qwidget.cpp:1162
#11 0xb7c91ef1 in ~QAbstractButton (this=0x81248b8)
    at widgets/qabstractbutton.cpp:572
#12 0xb7d3e6e5 in ~QToolButton (this=0x81248b8) at widgets/qtoolbutton.cpp:347
#13 0xb768eea1 in QObjectPrivate::deleteChildren (this=0x8123978)
    at kernel/qobject.cpp:1823
#14 0xb7a42649 in ~QWidget (this=0x8123938) at kernel/qwidget.cpp:1158
#15 0xb7d38844 in ~QToolBar (this=0x8123938) at widgets/qtoolbar.cpp:472
---Type <return> to continue, or q <return> to quit---
#16 0xb768eea1 in QObjectPrivate::deleteChildren (this=0x810b300)
    at kernel/qobject.cpp:1823
#17 0xb7a42649 in ~QWidget (this=0xbfec0a50) at kernel/qwidget.cpp:1158
#18 0xb7ceac21 in ~QMainWindow (this=0xbfec0a50) at widgets/qmainwindow.cpp:292
#19 0x080527ca in main ()

To copy to clipboard, switch view to plain text mode 

and at 22:00 LT

(gdb) backtrace
#0  0xffffe410 in __kernel_vsyscall ()
#1  0xb71f3770 in raise () from /lib/tls/i686/cmov/libc.so.6
#2  0xb71f4ef3 in abort () from /lib/tls/i686/cmov/libc.so.6
#3  0xb7228d0b in __fsetlocking () from /lib/tls/i686/cmov/libc.so.6
#4  0xb72308bd in mallopt () from /lib/tls/i686/cmov/libc.so.6
#5  0xb7230a44 in free () from /lib/tls/i686/cmov/libc.so.6
#6  0xb751fe5d in QHashData::freeNode (this=0x8112498, node=0x8124af8)
    at tools/qhash.cpp:132
#7  0xb797aabd in QHash<unsigned long, QWidget*>::remove (this=0x8108d30,
    akey=@0x812498c)
    at ../../include/QtCore/../../src/corelib/tools/qhash.h:450
#8  0xb796beff in QWidgetPrivate::setWinId (this=0x8124950, id=0)
    at kernel/qwidget.cpp:1180
#9  0xb79a1dcc in QWidget::destroy (this=0x81248b8, destroyWindow=true,
    destroySubWindows=true) at kernel/qwidget_x11.cpp:782
#10 0xb797166c in ~QWidget (this=0x81248b8) at kernel/qwidget.cpp:1162
#11 0xb7bc0ef1 in ~QAbstractButton (this=0x81248b8)
    at widgets/qabstractbutton.cpp:572
#12 0xb7c6d6e5 in ~QToolButton (this=0x81248b8) at widgets/qtoolbutton.cpp:347
#13 0xb75bdea1 in QObjectPrivate::deleteChildren (this=0x8123978)
    at kernel/qobject.cpp:1823
#14 0xb7971649 in ~QWidget (this=0x8123938) at kernel/qwidget.cpp:1158
#15 0xb7c67844 in ~QToolBar (this=0x8123938) at widgets/qtoolbar.cpp:472
---Type <return> to continue, or q <return> to quit---
#16 0xb75bdea1 in QObjectPrivate::deleteChildren (this=0x810b300)
    at kernel/qobject.cpp:1823
#17 0xb7971649 in ~QWidget (this=0xbf9e0d70) at kernel/qwidget.cpp:1158
#18 0xb7c19c21 in ~QMainWindow (this=0xbf9e0d70) at widgets/qmainwindow.cpp:292
#19 0x080527ca in main ()

(gdb) backtrace
#0  0xffffe410 in __kernel_vsyscall ()
#1  0xb71f3770 in raise () from /lib/tls/i686/cmov/libc.so.6
#2  0xb71f4ef3 in abort () from /lib/tls/i686/cmov/libc.so.6
#3  0xb7228d0b in __fsetlocking () from /lib/tls/i686/cmov/libc.so.6
#4  0xb72308bd in mallopt () from /lib/tls/i686/cmov/libc.so.6
#5  0xb7230a44 in free () from /lib/tls/i686/cmov/libc.so.6
#6  0xb751fe5d in QHashData::freeNode (this=0x8112498, node=0x8124af8)
    at tools/qhash.cpp:132
#7  0xb797aabd in QHash<unsigned long, QWidget*>::remove (this=0x8108d30,
    akey=@0x812498c)
    at ../../include/QtCore/../../src/corelib/tools/qhash.h:450
#8  0xb796beff in QWidgetPrivate::setWinId (this=0x8124950, id=0)
    at kernel/qwidget.cpp:1180
#9  0xb79a1dcc in QWidget::destroy (this=0x81248b8, destroyWindow=true,
    destroySubWindows=true) at kernel/qwidget_x11.cpp:782
#10 0xb797166c in ~QWidget (this=0x81248b8) at kernel/qwidget.cpp:1162
#11 0xb7bc0ef1 in ~QAbstractButton (this=0x81248b8)
    at widgets/qabstractbutton.cpp:572
#12 0xb7c6d6e5 in ~QToolButton (this=0x81248b8) at widgets/qtoolbutton.cpp:347
#13 0xb75bdea1 in QObjectPrivate::deleteChildren (this=0x8123978)
    at kernel/qobject.cpp:1823
#14 0xb7971649 in ~QWidget (this=0x8123938) at kernel/qwidget.cpp:1158
#15 0xb7c67844 in ~QToolBar (this=0x8123938) at widgets/qtoolbar.cpp:472
---Type <return> to continue, or q <return> to quit---
#16 0xb75bdea1 in QObjectPrivate::deleteChildren (this=0x810b300)
    at kernel/qobject.cpp:1823
#17 0xb7971649 in ~QWidget (this=0xbf9e0d70) at kernel/qwidget.cpp:1158
#18 0xb7c19c21 in ~QMainWindow (this=0xbf9e0d70) at widgets/qmainwindow.cpp:292
#19 0x080527ca in main ()

To copy to clipboard, switch view to plain text mode 

And finally here you have two complete listings...
mymvc.h

#ifndef MTR_SAFETY_DECLARATION_FOR_MYMVC_H
#define MTR_SAFETY_DECLARATION_FOR_MYMVC_H
#include <QWidget>
namespace MTR {
class Book;
class SBook;
}
class myView;
class myModel;
class QVBoxLayout;
class QLabel;
class QComboBox;
class QLineEdit;
 
class MyMVC : public QWidget {
	Q_OBJECT
public:
	MyMVC(QList<MTR::Book>* list = 0, QWidget* parent = 0);
	void prodModel(QList<MTR::Book>*);
public slots:
	void setCaseSensitive(bool cs) { caseSensitive = cs; }
signals:
	void doubleClickInTreeView(int);
private:
	QList<MTR::SBook>* slist;
	QLineEdit* le;
	QComboBox* qcb;
	myModel* model;
	myView* qtv;
	QVBoxLayout* qvb;
	QLabel* label;
	QLabel* whatTheHell; //IF COMMENTED, DOES NOT CAUSE ERROR
	bool caseSensitive;
};
 
#endif

#ifndef MTR_SAFETY_DECLARATION_FOR_MYMVC_H
#define MTR_SAFETY_DECLARATION_FOR_MYMVC_H
#include <QWidget>
namespace MTR {
class Book;
class SBook;
}
class myView;
class myModel;
class QVBoxLayout;
class QLabel;
class QComboBox;
class QLineEdit;

class MyMVC : public QWidget {
	Q_OBJECT
public:
	MyMVC(QList<MTR::Book>* list = 0, QWidget* parent = 0);
	void prodModel(QList<MTR::Book>*);
public slots:
	void setCaseSensitive(bool cs) { caseSensitive = cs; }
signals:
	void doubleClickInTreeView(int);
private:
	QList<MTR::SBook>* slist;
	QLineEdit* le;
	QComboBox* qcb;
	myModel* model;
	myView* qtv;
	QVBoxLayout* qvb;
	QLabel* label;
	QLabel* whatTheHell; //IF COMMENTED, DOES NOT CAUSE ERROR
	bool caseSensitive;
};

#endif

To copy to clipboard, switch view to plain text mode 

and mymvc.cpp

#include "mymvc.h"
#include "myView.h"
#include "sbook.h"
#include "myModel.h"
#include "myHeaderView.h"
#include <QList>
#include <QVBoxLayout>
#include <QLabel>
#include <QLineEdit>
#include <QComboBox>
#include <QCheckBox>
 
MyMVC::MyMVC(QList<MTR::Book>* list, QWidget* parent) {
	slist = new QList<MTR::SBook>();
	for(int i = 0;i<list->size();++i) {
		slist->append(MTR::SBook(i,list->value(i)));
	}
   setFixedSize(590,400);
 
   le = new QLineEdit();
   qcb = new QComboBox();
   qcb->addItem(tr("filter by Title"));
   qcb->addItem(tr("filter by Author"));
   qcb->addItem(tr("filter by Publisher"));
   qcb->setCurrentIndex(0);
 
   model = new myModel(*slist,le,qcb);
	model->filter();
 
   qtv = new myView(model);
	qtv->header()->setSortIndicator(0,Qt::DescendingOrder);
 
   label = new QLabel(tr("Enter a character string below"));
   whatTheHell = new QLabel(tr("Damn")); //IF COMMENTED, DOES NOT CAUSE ERROR
	caseSensitive = true;
   qvb = new QVBoxLayout();
   connect(qcb,SIGNAL(currentIndexChanged(int)),le,SLOT(clear()));
   connect(qcb,SIGNAL(currentIndexChanged(int)),qtv,SLOT(filter()));
   connect(le,SIGNAL(textChanged(const QString &)),qtv,SLOT(filter()));
   connect(model,SIGNAL(fromTVtoVM(int)),this,SIGNAL(doubleClickInTreeView(int)));
 
   qvb->addWidget(label);
   qvb->addWidget(le);
   qvb->addWidget(whatTheHell); //IF COMMENTED, DOES NOT CAUSE ERROR
   qvb->addWidget(qcb);
   qvb->addWidget(qtv);
   setLayout(qvb);
}
 
void MyMVC::prodModel(QList<MTR::Book>* list) {
	slist->clear();
	for(int i = 0;i<list->size();++i) {
		slist->append(MTR::SBook(i,list->value(i)));
	}
	model->setInternalList(*slist);
   qcb->setCurrentIndex(0);
	qtv->filter();
 
}

#include "mymvc.h"
#include "myView.h"
#include "sbook.h"
#include "myModel.h"
#include "myHeaderView.h"
#include <QList>
#include <QVBoxLayout>
#include <QLabel>
#include <QLineEdit>
#include <QComboBox>
#include <QCheckBox>

MyMVC::MyMVC(QList<MTR::Book>* list, QWidget* parent) {
	slist = new QList<MTR::SBook>();
	for(int i = 0;i<list->size();++i) {
		slist->append(MTR::SBook(i,list->value(i)));
	}
   setFixedSize(590,400);

   le = new QLineEdit();
   qcb = new QComboBox();
   qcb->addItem(tr("filter by Title"));
   qcb->addItem(tr("filter by Author"));
   qcb->addItem(tr("filter by Publisher"));
   qcb->setCurrentIndex(0);

   model = new myModel(*slist,le,qcb);
	model->filter();
	
   qtv = new myView(model);
	qtv->header()->setSortIndicator(0,Qt::DescendingOrder);

   label = new QLabel(tr("Enter a character string below"));
   whatTheHell = new QLabel(tr("Damn")); //IF COMMENTED, DOES NOT CAUSE ERROR
	caseSensitive = true;
   qvb = new QVBoxLayout();
   connect(qcb,SIGNAL(currentIndexChanged(int)),le,SLOT(clear()));
   connect(qcb,SIGNAL(currentIndexChanged(int)),qtv,SLOT(filter()));
   connect(le,SIGNAL(textChanged(const QString &)),qtv,SLOT(filter()));
   connect(model,SIGNAL(fromTVtoVM(int)),this,SIGNAL(doubleClickInTreeView(int)));

   qvb->addWidget(label);
   qvb->addWidget(le);
   qvb->addWidget(whatTheHell); //IF COMMENTED, DOES NOT CAUSE ERROR
   qvb->addWidget(qcb);
   qvb->addWidget(qtv);
   setLayout(qvb);
}

void MyMVC::prodModel(QList<MTR::Book>* list) {
	slist->clear();
	for(int i = 0;i<list->size();++i) {
		slist->append(MTR::SBook(i,list->value(i)));
	}
	model->setInternalList(*slist);
   qcb->setCurrentIndex(0);
	qtv->filter();

}

To copy to clipboard, switch view to plain text mode 

[jacek]

How did you install Qt?

[jpn]

What does valgrind have to say about it?

[mtrpoland]

Valgrind?
I do not know whether it is useful as not long time ago I played with FlowLayout example (QT 4.2) and valgrind gave me:

==6237== LEAK SUMMARY:
==6237==    definitely lost: 88 bytes in 3 blocks.
==6237==    indirectly lost: 120 bytes in 10 blocks.
==6237==      possibly lost: 744 bytes in 3 blocks.
==6237==    still reachable: 322,563 bytes in 4,588 blocks.
==6237==         suppressed: 0 bytes in 0 blocks.

==6237== LEAK SUMMARY:
==6237==    definitely lost: 88 bytes in 3 blocks.
==6237==    indirectly lost: 120 bytes in 10 blocks.
==6237==      possibly lost: 744 bytes in 3 blocks.
==6237==    still reachable: 322,563 bytes in 4,588 blocks.
==6237==         suppressed: 0 bytes in 0 blocks.

To copy to clipboard, switch view to plain text mode 

Now my 1600+ lines application gives me:

==6214== LEAK SUMMARY:
==6214==    definitely lost: 449 bytes in 32 blocks.
==6214==    indirectly lost: 1,526 bytes in 36 blocks.
==6214==      possibly lost: 744 bytes in 3 blocks.
==6214==    still reachable: 331,034 bytes in 4,629 blocks.
==6214==         suppressed: 0 bytes in 0 blocks.

==6214== LEAK SUMMARY:
==6214==    definitely lost: 449 bytes in 32 blocks.
==6214==    indirectly lost: 1,526 bytes in 36 blocks.
==6214==      possibly lost: 744 bytes in 3 blocks.
==6214==    still reachable: 331,034 bytes in 4,629 blocks.
==6214==         suppressed: 0 bytes in 0 blocks.

To copy to clipboard, switch view to plain text mode 

So I suppose that some of those are because of qt libraries... Am I right?

Anyway, I may write some parts of valgrind message after checking my application.

[marcel]

I don't think this is getting anywhere.
If the app is only around 1600 lines, then why don't you post it here?
Would that be possible? Then someone could test it.

Regards

[mtrpoland]

Below are links to working application (I have circumvented the problem above) and package of code files and file containing some data to test program (one humble *.dat file).

Anyway, I would be glad if someone experienced checked some of my code for bugs and bad habits. I know it is difficult to ask you for such thing but marcel suggested such a solution himself.

The "funny" thing is when I de-comment code connected to QTranslator in main.cpp I get SIGSEV gift.
In other words I cannot translate my application using .ts file made by Linguist.

So my inference is that there is some crucial memory management bug in MY code that sometimes throws irrational SIGSEV, blocking me of using certain normal ways of programming.

I just cannot find it!

Program and code package in:
http://www.mjakobczyk.pl/pub/code/

[marcel]

I compiled your application( on Windows, Qt 4.3 ) but I cannot get it to crash( I uncommented the translator lines in main).
Could you say what steps are involved?

Regards

[jpn]

Here's at least one issue that prevents it from launching for me:

void MWin::initialization(void) {
	...
	tableViewWidget = new MyMVC(list); // list is yet uninitialized
	...
	list = new QList<MTR::Book>; // it's being initialized somewhere here afterwards
	...
}

void MWin::initialization(void) {
	...
	tableViewWidget = new MyMVC(list); // list is yet uninitialized
	...
	list = new QList<MTR::Book>; // it's being initialized somewhere here afterwards
	...
}

To copy to clipboard, switch view to plain text mode 

I would suggest allocating QList objects on the stack and passing them as references. Allocating them on the heap is more error prone. QList is an implicitly shared class so unmodified copies are cheap.

[marcel]

Yeah, that and correct the following from:

namespace MTR {
class Book;
class SBook;
}

namespace MTR {
class Book;
class SBook;
}

To copy to clipboard, switch view to plain text mode 

to:

namespace MTR {
class Book;
struct SBook;
}

namespace MTR {
class Book;
struct SBook;
}

To copy to clipboard, switch view to plain text mode 

I was getting a few warnings and a linker error because of this.

Regards