Update comment

pull/16/head
Timothy Pearson 13 years ago
parent 40a38a2f9b
commit 2e95a0338f

@ -742,10 +742,11 @@ TQPixmap KFileItem::pixmap( int _size, int _state ) const
kdWarning() << "Pixmap not found for mimetype " << m_pMimeType->name() << endl; kdWarning() << "Pixmap not found for mimetype " << m_pMimeType->name() << endl;
if ( mime->name() == "application/x-executable" ) { if ( mime->name() == "application/x-executable" ) {
// FIXME // At first glance it might seem to be a good idea to
// Look for .desktop files for this executable // look for .desktop files for this executable before resorting to the embedded icon
// before resorting to the embedded icon // in the same fashion as the minicli, but on close examination this is NOT A GOOD IDEA.
// (look at how the minicli does it) // 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 #ifdef HAVE_ELFICON
// Check for an embedded icon // Check for an embedded icon
unsigned int icon_size; unsigned int icon_size;

Loading…
Cancel
Save