1 files changed, 1 insertions, 3 deletions

diff --git a/noncore/graphics/opie-eye/slave/bmp_slave.cpp b/noncore/graphics/opie-eye/slave/bmp_slave.cpp
index 2fa825f..0efadac 100644
--- a/noncore/graphics/opie-eye/slave/bmp_slave.cpp
+++ b/noncore/graphics/opie-eye/slave/bmp_slave.cpp
@@ -37,100 +37,98 @@ namespace {
         QFile _inputfile;
         pBmpHeader m_Header;
 
         static const int OLD = 12;
         static const int WIN = 40;
         static const int OS2 = 64;
         static const int RGB = 0;
         static const int RLE8 = 1;
         static const int RLE4 = 2;
 
     public:
         BmpHeader(const QString&fname);
         virtual ~BmpHeader();
         bool isBmp()const{return qstrncmp(m_Header.type,"BM",2)==0;}
         bool isCompressed()const{return m_Header.Compression != 0;}
         QSize imageSize(){return QSize(m_Header.Width,m_Header.Height);}
         QString imageCompression()const;
         int bitsPixel()const{return m_Header.BitCount;}
         int Size()const{return m_Header.hSize;}
         int compressedSize()const{return m_Header.SizeImage;}
         int ColorsUsed()const{return m_Header.ClrUsed;}
         int XPix()const{return m_Header.XPerMeter;}
         int YPix()const{return m_Header.YPerMeter;}
     };
 
     QString BmpHeader::imageCompression()const
     {
         switch (m_Header.Compression) {
             case RLE8:
                 return "8Bit RLE Encoding";
                 break;
             case RLE4:
                 return "4Bit RLE Encoding";
                 break;
             case RGB:
             default:
                 return "No encoding";
         }
     }
 
     BmpHeader::BmpHeader(const QString&fname)
         : _name(fname),_inputfile(_name)
     {
         read_data();
     }
 
     void BmpHeader::read_data() {
         memset(&m_Header,0,sizeof(pBmpHeader));
-        _inputfile.open(IO_Raw|IO_ReadOnly);
-        if (!_inputfile.isOpen()) {
+        if (!_inputfile.open(IO_Raw|IO_ReadOnly))
             return;
-        }
         QDataStream s(&_inputfile);
         s.setByteOrder( QDataStream::LittleEndian );
         s.readRawBytes(m_Header.type,2);
         if (!isBmp()) {
             _inputfile.close();
             return;
         }
         s >> m_Header.hSize;
         s >> m_Header.reserved1 >> m_Header.reserved2;
         s >> m_Header.Size;
         if ( m_Header.Size == BmpHeader::WIN || m_Header.Size == BmpHeader::OS2 ) {
             s >> m_Header.Width >> m_Header.Height >> m_Header.Planes >> m_Header.BitCount;
             s >> m_Header.Compression >> m_Header.SizeImage;
             s >> m_Header.XPerMeter >> m_Header.YPerMeter;
             s >> m_Header.ClrUsed >> m_Header.ClrImportant;
         } else {
             Q_INT16 w, h;
             s >> w >> h >> m_Header.Planes >> m_Header.BitCount;
             m_Header.Width  = w;
             m_Header.Height = h;
             m_Header.Compression = BmpHeader::RGB;
             m_Header.SizeImage = 0;
             m_Header.XPerMeter = m_Header.YPerMeter = 0;
             m_Header.ClrUsed = m_Header.ClrImportant = 0;
         }
         _inputfile.close();
     }
 
     BmpHeader::~BmpHeader() {
     }
 }
 
 
 BmpSlave::BmpSlave()
     : SlaveInterface(QStringList("bmp"))
 {}
 
 BmpSlave::~BmpSlave() {
 
 }
 
 QString BmpSlave::iconViewName(const QString& str) {
     QString st;
     BmpHeader bh(str);
     if (!bh.isBmp()) {
         st.append("No bmp file");
         return st;
     }