Review Request: Do not fetch metaInfo on special files (fifo/chr/blk), on the Properties file dialog

Darío Andrés andresbajotierra at gmail.com
Tue May 19 22:02:38 BST 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/723/
-----------------------------------------------------------

Review request for kdelibs.


Summary
-------

Check if the current file is a special file and in that case do not fetch the metaInfo as that will try to open the file and the application will hang

I wonder if this should be done inside KFileMetaInfo to avoid all the possible cases.., but doing it there will need to create a new KFileItem, fetch the mode and check it. (or may be doing it into: "KFileItem::metaInfo"). To Discuss...


This addresses bug 190596.
    https://bugs.kde.org/show_bug.cgi?id=190596


Diffs
-----

  svn://anonsvn.kde.org/home/kde/trunk/KDE/kdelibs/kio/kfile/kmetaprops.cpp 969788 

Diff: http://reviewboard.kde.org/r/723/diff


Testing
-------

The fix works OK. 


Thanks,

Darío





More information about the kde-core-devel mailing list