Revision 625536

Go back to digest for 21st January 2007

Bug Fixes in KDE Base

David Faure committed changes in /trunk/KDE/kdebase/libkonq:

Added unit tests for undo after moving files and for copying a directory.
The latter crashed; found a nice porting bug in konq_undo:
QVector::iterator end = v.end(); while (it!=end) { if (...) { it = v.erase(it); } else ++it; }
This is invalid because erasing elements changing the end, so in such a case we can't cache end().

Also fixed 61442: after renaming a file, undo says "Undo Rename" now instead of "Undo Move".

File Changes

Modified 6 files
  • /trunk/KDE/kdebase/libkonq
  •   /konq_operations.cc
  •   /konq_undo.cc
  •   /konq_undo.h
  •   /konq_undo_p.h
  •   /tests/konqundomanagertest.cpp
  •   /tests/konqundomanagertest.h
6 files changed in total