summaryrefslogtreecommitdiffstats
path: root/kio
diff options
context:
space:
mode:
authorTimothy Pearson <kb9vqf@pearsoncomputing.net>2011-12-30 23:40:14 -0600
committerTimothy Pearson <kb9vqf@pearsoncomputing.net>2011-12-30 23:40:14 -0600
commit2e95a0338f40a820ea592802357e985cfcbdb388 (patch)
tree83f3378b0babf7de568eaa790bcedd55da2e3294 /kio
parent40a38a2f9b45bb73460145148bd4e4560e0d9672 (diff)
downloadtdelibs-2e95a0338f40a820ea592802357e985cfcbdb388.tar.gz
tdelibs-2e95a0338f40a820ea592802357e985cfcbdb388.zip
Update comment
Diffstat (limited to 'kio')
-rw-r--r--kio/kio/kfileitem.cpp9
1 files changed, 5 insertions, 4 deletions
diff --git a/kio/kio/kfileitem.cpp b/kio/kio/kfileitem.cpp
index eaedf0a2d..839c38a68 100644
--- a/kio/kio/kfileitem.cpp
+++ b/kio/kio/kfileitem.cpp
@@ -742,10 +742,11 @@ TQPixmap KFileItem::pixmap( int _size, int _state ) const
kdWarning() << "Pixmap not found for mimetype " << m_pMimeType->name() << endl;
if ( mime->name() == "application/x-executable" ) {
- // FIXME
- // Look for .desktop files for this executable
- // before resorting to the embedded icon
- // (look at how the minicli does it)
+ // At first glance it might seem to be a good idea to
+ // look for .desktop files for this executable before resorting to the embedded icon
+ // in the same fashion as the minicli, but on close examination this is NOT A GOOD IDEA.
+ // Specifically it allows one executable to mimic another purely based on filename,
+ // which could at certain times fool any user regardless of experience level.
#ifdef HAVE_ELFICON
// Check for an embedded icon
unsigned int icon_size;