diff options
| author | Timothy Pearson <kb9vqf@pearsoncomputing.net> | 2011-12-30 23:40:14 -0600 |
|---|---|---|
| committer | Timothy Pearson <kb9vqf@pearsoncomputing.net> | 2011-12-30 23:40:14 -0600 |
| commit | 2e95a0338f40a820ea592802357e985cfcbdb388 (patch) | |
| tree | 83f3378b0babf7de568eaa790bcedd55da2e3294 | |
| parent | 40a38a2f9b45bb73460145148bd4e4560e0d9672 (diff) | |
| download | tdelibs-2e95a033.tar.gz tdelibs-2e95a033.zip | |
Update comment
| -rw-r--r-- | kio/kio/kfileitem.cpp | 9 |
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; |