Do not set resize mode of non-existent header sections

[William Mitchell Jr]

When trying to view a pdf file qpdfview fails with

QMetaObject::indexOfSignal: signal finished() from QThread redefinied in SearchTask
ASSERT: "visual != -1" in file itemviews/qheaderview.cpp, line 1184

Built with Qt 4.8.4 from qpdfview-0.4.2.tar.gz
Debain/Sid
KDE 4.8.4
kernel 3.2.41-2 i686

WM

[Benjamin Eltzner (b-eltzner)]

I installed KDE in my Sid VM and tried to reproduce the problem but could not.
Did you build the package with the debian-folder from mentors or did you install from source?
Does the problem occur with all of your files or only with some?
If only some files are affected, could you attach one such file to this bug report?

Best regards, Benjamin

[Adam Reichold (adamreichold)]

Hello again,

I think the two messages might be unrelated. Redefining the signal from QThread is a mistake that I just fixed in trunk revision 1121, but I don't think this causes the assertion in QHeaderView to fail. I'd rather say this is bug #1171826 resurfacing.

Could you try to grab a current snapshot from trunk? (Either by running "bzr branch lp:qpdfview" or by using the "download tarball" link at [1].)

Best regards, Adam.

[1] https://bazaar.launchpad.net/~adamreichold/qpdfview/trunk/revision/head:

[Adam Reichold (adamreichold)]

By the way, do you use debug builds of Qt?

[Adam Reichold (adamreichold)]

Ok, I guarded against setting the resize mode of non-existent sections in trunk revision 1123. (Possibly, the assertions are disabled in Qt release builds?)

[William Mitchell Jr (wdmjun)]

Adam,
Yes. My one of my Qt 4.8.4 config options was -debug.
I'll try the latest snapshot.
Benjamin,
It happens with all pdfs.

On Fri, Apr 26, 2013 at 11:08 AM, Adam Reichold <email address hidden>wrote:

> Ok, I guarded against setting the resize mode of non-existent sections
> in trunk revision 1123. (Possibly, the assertions are disabled in Qt
> release builds?)
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1173045
>
> Title:
> ASSERT in 0.4.2
>
> Status in qpdfview:
> Incomplete
>
> Bug description:
> When trying to view a pdf file qpdfview fails with
>
> QMetaObject::indexOfSignal: signal finished() from QThread redefinied in
> SearchTask
> ASSERT: "visual != -1" in file itemviews/qheaderview.cpp, line 1184
>
> Built with Qt 4.8.4 from qpdfview-0.4.2.tar.gz
> Debain/Sid
> KDE 4.8.4
> kernel 3.2.41-2 i686
>
> WM
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/qpdfview/+bug/1173045/+subscriptions
>

[Adam Reichold (adamreichold)]

Since you reported on the keyboard shortcuts using a current snapshot, has this problem been resolved?

[Adam Reichold (adamreichold)]

I would be nice to be able to close this bug before continue digging deeper into the keyboard issues: So is the crash because of QHeaderView's assertion gone in the current trunk revision?

[William Mitchell Jr (wdmjun)]

Yes.
No crashes in the current trunk version.
WM

On Thu, May 2, 2013 at 10:45 AM, Adam Reichold <email address hidden>wrote:

> I would be nice to be able to close this bug before continue digging
> deeper into the keyboard issues: So is the crash because of
> QHeaderView's assertion gone in the current trunk revision?
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1173045
>
> Title:
> Do not set resize mode of non-existent header sections
>
> Status in qpdfview:
> Incomplete
>
> Bug description:
> When trying to view a pdf file qpdfview fails with
>
> QMetaObject::indexOfSignal: signal finished() from QThread redefinied in
> SearchTask
> ASSERT: "visual != -1" in file itemviews/qheaderview.cpp, line 1184
>
> Built with Qt 4.8.4 from qpdfview-0.4.2.tar.gz
> Debain/Sid
> KDE 4.8.4
> kernel 3.2.41-2 i686
>
> WM
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/qpdfview/+bug/1173045/+subscriptions
>