Bug 186828 - next/previous file still go through the files randomly instead of alphabetically/directory order
Summary: next/previous file still go through the files randomly instead of alphabetica...
Status: RESOLVED FIXED
Alias: None
Product: kompare
Classification: Applications
Component: navigationpart (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Kompare developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-03-11 00:57 UTC by Otto Bruggeman
Modified: 2009-03-25 00:59 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Otto Bruggeman 2009-03-11 00:57:16 UTC 
Version:           4.2.1/trunk (using KDE 4.2.1)
Compiler:          Does not matter 
OS:                Linux
Installed from:    Compiled From Sources

Most likely sorting is not fixed for multi-file diffs. Need to investigate why.
Comment 1 Otto Bruggeman 2009-03-11 21:44:11 UTC 
SVN commit 938418 by bruggie:

BUG: 186828 Fixed by copying some code from Qt and adapting it to my use case. I should not store pointers in Q3ValueList, this way it is not calling the operator< i defined but it did pointer comparison, that is why the order was so funky. Thanks David for reporting the bug to me on IRC.

 M  +38 -1     diffmodellist.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=938418
Comment 2 Kevin Kofler 2009-03-25 00:59:02 UTC 
SVN commit 944069 by kkofler:

CCBUG: 186828 Fixed by copying some code from Qt and adapting it to my use case. I should not store pointers in Q3ValueList, this way it is not calling the operator< i defined but it did pointer comparison, that is why the order was so funky. Thanks David for reporting the bug to me on IRC.

Backport revision 938418 by bruggie from trunk.

 M  +38 -1     diffmodellist.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=944069