bugs and patches

[Marcin Gryszkalis]

hi there

I happy to see upcoming new release but I can see one terrible problem with
current version:

I'm running SVN Head Krusader on KDE 4.3.2 (gentoo) and it makes me cry -
Krusader crashes 100 times a day :(

I provided stack traces at https://bugs.kde.org/show_bug.cgi?id=207276

I thought the problem is caused by Carai's changes in rev 994762 but I tried
to step back to earlier revision and it still crashes. I can do whatever is
needed to fix the bug :)

By the way, current svn doesn't include patch that fixes one of dark-
background bugs. The patch is ok, I applied it and it it fixed the problem.
http://sourceforge.net/tracker/?func=detail&atid=106488&aid=2858133&group_id=6488

best regards
--
Marcin Gryszkalis, PGP 0x9F183FA3
jabber jid:m...@fork.pl, gg:2532994
http://the.fork.pl

[Karai Csaba]

Hi,

It's not a bug.
The problem is, that your QT was compiled in debug mode.

We can't do much if QT crashes. Please recompile QT removing the
assertions from ~QPersistentModelIndex, or use a non-debug QT version.

Or it would be even better to report this bug to QT

#8 0xb62201c8 in abort () at abort.c:88
#9 0xb64f00d6 in qt_message_output (msgType=QtFatalMsg,
buf=0xac14f88 "ASSERT failure in
QPersistentModelIndex::~QPersistentModelIndex: \"persistent model indexes

Thanks,

Csaba

Marcin Gryszkalis írta:

[Marcin Gryszkalis]

On Tuesday of November 3 2009, Karai Csaba wrote:
> We can't do much if QT crashes. Please recompile QT removing the
> assertions from ~QPersistentModelIndex, or use a non-debug QT version.

It would work I guess but I still think if there's problem then it should be
fixed here or there.

> Or it would be even better to report this bug to QT

I filled up report in QT's bugtracker:
http://bugreports.qt.nokia.com/browse/QTBUG-6765

I got following comment there:

>>> The comment from the Qt code says:
>>> // This assert may happen if the model use changePersistentIndex in a way
>>> that could result on two
>>> // QPersistentModelIndex pointing to the same index.

What do you think about it? Can this be a cause?

[Jan Lepper]

On Saturday 19 December 2009 21:13:45 Marcin Gryszkalis wrote:
> On Tuesday of November 3 2009, Karai Csaba wrote:
> > We can't do much if QT crashes. Please recompile QT removing the
> > assertions from ~QPersistentModelIndex, or use a non-debug QT version.
>
> It would work I guess but I still think if there's problem then it should
> be fixed here or there.
>
> > Or it would be even better to report this bug to QT
>
> I filled up report in QT's bugtracker:
> http://bugreports.qt.nokia.com/browse/QTBUG-6765
>
> I got following comment there:
> >>> The comment from the Qt code says:
> >>> // This assert may happen if the model use changePersistentIndex in a
> >>> way that could result on two
> >>> // QPersistentModelIndex pointing to the same index.
>
> What do you think about it? Can this be a cause?
>
> regards

I will take a look at this.

Jan

[Gábor Lehel]

Thank you!

For your information, the regression was introduced some time between
2009-02-27 (r6234) and the official release of 2.0.

> --
>
> You received this message because you are subscribed to the Google Groups "krusader-devel" group.
> To post to this group, send email to krusade...@googlegroups.com.
> To unsubscribe from this group, send email to krusader-deve...@googlegroups.com.
> For more options, visit this group at http://groups.google.com/group/krusader-devel?hl=en.
>
>
>

--
Work is punishment for failing to procrastinate effectively.

[Marcin Gryszkalis]

On Sunday of December 20 2009, Gábor Lehel wrote:
> For your information, the regression was introduced some time between
> 2009-02-27 (r6234) and the official release of 2.0.

I spent some time recompiling different versions of krusader and it seems that
I found something:

The problem IS visible in r6234 if you manually switch to "experimental"
detailed view.

In the commit r6292 (r975294 after migration) Karai changed View names,
Experimental views become default (and old vews are called "old and obsolete",
later "old" views are removed).

[Frank Schoolmeesters]

Hi,

Krusader-2.0.0 "Mars Pathfinder contains the new code but the old code
is used by default.
We did not want to use the new code for the stable release.

The new code is default in 2.1.0-beta1 "Rusty Clutch" and later
(current SVN) and the old code was removed.
See changelog entry
"CHANGED: the old views were replaced by the interviews"

IIRC this change did remove all the latest KDE3 code in Krusader.

Thanks and bye,

Frank