Rewrite OFileNotification to use the upcoming inotify (kernel 2.6) interface instead of the deprecated dnotify

4 files changed, 168 insertions, 246 deletions

diff --git a/ChangeLog b/ChangeLog
index 2d5c886..7e81b7e 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,71 +1,72 @@
         2005-??-??Opie 1.2.1
         
         
         New Features
         ------------
         * OpieStumbler: Scans WiFi networks using the wireless extension scanning (skyhusker)
 
         Fixed Bugs
         ----------
+        * #1476 - Wrong order of application entries in the O-menu (skyhusker)
                 * #1535- Missing line break and unnecessary location shown with Today-Calendar plugin (deller)
         * #1565 - crash-fix in odevice.cpp while scanning the distribution table (deller)
         * #1614 - Make Opie-console start in $HOME instead of / (skyhusker)
                 * n.a.- always show volume and wireless applet popups inside visible screen (deller)
         * n.a.  - scale O-Menu-Applets appropriately (mickeyl)
                 * n.a.- libopienet: fix bugs in wireless scanning and setting SSID (skyhusker)
                 * n.a.- Wellenreiter: relax WE version matching test a bit (mickeyl)
-                * n.a.- scale BluezApplet appropriately (mickeyl)
+                * n.a.- scale BluezApplet appropriately and use larger icons (mickeyl)
                 * n.a.- memoryapplet: fix crash in memoryapplet on kernels without swap support (seneca cunningham)
-        * #1476 - Wrong order of application entries in the O-menu (skyhusker)
 
         Internal
         --------
-        * make BluezApplet use OTaskbarApplet (mickeyl)
+        * Make BluezApplet use OTaskbarApplet (mickeyl)
+        * Rewrite OFileNotification to use the upcoming inotify interface instead of the deprecated dnotify (mickeyl)
 
         2005-03-25Opie 1.2.0
 
         Fixed Bugs
         ----------
         * #1613 - AdvancedFM - scale toolbar icons appropriately (drw)
         * #1620 - OFileSelector - show the button on press and not on press on hold (alwin)
         * #1473 - Opie-Eye - Same as #1620 but we lack a common FileSystem Button class (zecke)
         * n.a.  - PackageManager - fix bug where messages show up multiple times in install dialog (drw)
                 * n.a.- make qpeglobal.h include qglobal.h (zecke)
 
         2005-03-20Opie 1.2.0-rc1
 
 
         New Features
         ------------
         * Launcher: Support a static background pixmap (mickeyl)
         * LauncherSettings: Choose whether to have a static background pixmap (mickeyl)
         * PackageManager supports the 'lists_dir' ipkg configuration option (drw)
         * Added hi-res inline images for large resolution devices (drw)
         * Improved launcher icons for consistency (ar)
         * Datebook: Added plugin system to datebook so holidays and birthdays from contacts may displayed in datebook (alwin)
 
         Fixed Bugs
         ----------
         * #1236 - VCards contained empty home and work address entries causing Palms to crash (eilers)
         * #1412 - Launcher Settings - fixed default settings for background in Edit tab dialog (drw)
         * #1472 - Opie textedit now check for unsaved changes on ESC key (ljp)
         * #1474 - Implement GUI for Scrollbars on the Left in Appearance (zecke)
         * #1482 - Fix double '/' in paths (zecke)
         * #1536 - Autosave of custom locations in opie backup (ar)
         * #1539 - Fixed displaying too long path in the delete dialog in opie-eye (zecke)
         * #1540 - Implemented deletion of DocLnks in opie-eye (zecke)
         * #1542 - Fixed Todo crash when priority < 1 or > 5 (drw)
         * #1550 - Browse for alarm sound start in QPEDIR/sounds/ (zecke)
         * #1554 - Fixed Opie-Console name in .desktop file (mickeyl)
         * #1556 - Start to look for sound files in /opt/QtPalmtop/sounds (zecke)
         * #1558 - Fixed opie-login breaking opie startup (mickeyl)
         * #1560 - Fixed Opie-Console picking up the wrong shell (bluelightning)
         * #1570 - Confirmation needed befor Restore a backup and overwrite local data (ar)
         * #1591 - Addressbook sortorder changed by clicking on column 0 (eilers)
         * #1599 - Security - change recommendation for opie-multiauth to opie-securityplugin (gints)
         * #1604 - Checkbook crashed on startup (drw)
         * #1608 - Addressbook crashed after search, which may caused by not initialzed member variable (eilers)
         * #1608 - QueryByExampe-Datediff on SQL database did not worked properly (eilers)
         * n.a.  - Netsystemtime - fixed UI to prevent error dialog from displaying when it shouldn't, get rid of some compiler warnings (drw)
         * n.a.  - PackageManager - fixed where last package in status file was not shown as installed when it should be (drw)
         * n.a.  - PackageManager - fixed bug where lists_dir was not being honored at startup (drw)
diff --git a/examples/opiecore/onotifydemo/onotifydemo.cpp b/examples/opiecore/onotifydemo/onotifydemo.cpp
index a38ffee..0e84d80 100644
--- a/examples/opiecore/onotifydemo/onotifydemo.cpp
+++ b/examples/opiecore/onotifydemo/onotifydemo.cpp
@@ -1,132 +1,140 @@
 #include "onotifydemo.h"
 
 /* OPIE */
 #include <opie2/odebug.h>
 #include <opie2/oapplication.h>
 #include <opie2/ofiledialog.h>
 #include <opie2/olistview.h>
 #include <opie2/ofilenotify.h>
 using namespace Opie::Core;
 using namespace Opie::Ui;
 
 /* QT */
 #include <qcheckbox.h>
 #include <qvbox.h>
 #include <qhbox.h>
 #include <qhbuttongroup.h>
 #include <qvbuttongroup.h>
 #include <qmessagebox.h>
 #include <qpushbutton.h>
 
 DemoApp::DemoApp( int argc, char** argv ) : OApplication( argc, argv, "libopie2 notify demo" )
   {
 
     QVBox* vbox = new QVBox();
     setMainWidget( vbox );
 
     l = new OListView( vbox );
     l->addColumn( "Notification Path" );
     l->addColumn( "Trigger Type" );
     l->addColumn( "Trigger Mask" );
     l->setColumnAlignment( 1, AlignCenter );
     l->setColumnAlignment( 2, AlignCenter );
 
     QHBox* hbox = new QHBox( vbox );
-
     g2 = new QVButtonGroup( "Specify Trigger Type", hbox );
-    //QCheckBox* c1 = new QCheckBox( "Multi", g2 );
-    QCheckBox* c2 = new QCheckBox( "Access", g2 );
-    QCheckBox* c3 = new QCheckBox( "Modify", g2 );
-    QCheckBox* c4 = new QCheckBox( "Create", g2 );
-    QCheckBox* c5 = new QCheckBox( "Delete", g2 );
-    QCheckBox* c6 = new QCheckBox( "Rename", g2 );
-    QCheckBox* c7 = new QCheckBox( "Attrib", g2 );
-    g2->insert( c2, Access );
-    g2->insert( c3, Modify );
-    g2->insert( c4, Create );
-    g2->insert( c5, Delete );
-    g2->insert( c6, Rename );
-    g2->insert( c7, Attrib );
+    QCheckBox* c1 = new QCheckBox( "Access", g2 );
+    QCheckBox* c2 = new QCheckBox( "Modify", g2 );
+    QCheckBox* c3 = new QCheckBox( "Attrib", g2 );
+    QCheckBox* c4 = new QCheckBox( "CloseWrite", g2 );
+    QCheckBox* c5 = new QCheckBox( "CloseNoWrite", g2 );
+    QCheckBox* c6 = new QCheckBox( "MovedFrom", g2 );
+    QCheckBox* c7 = new QCheckBox( "MovedTo", g2 );
+    QCheckBox* c8 = new QCheckBox( "DeleteSubdir", g2 );
+    QCheckBox* c9 = new QCheckBox( "DeleteFile", g2 );
+    QCheckBox* c10 = new QCheckBox( "CreateSubdir", g2 );
+    QCheckBox* c11 = new QCheckBox( "CreateFile", g2 );
+    QCheckBox* c12 = new QCheckBox( "Unmount", g2 );
+    g2->insert( c1, Access );
+    g2->insert( c2, Modify );
+    g2->insert( c3, Attrib );
+    g2->insert( c4, CloseWrite );
+    g2->insert( c5, CloseNoWrite );
+    g2->insert( c6, MovedFrom );
+    g2->insert( c7, MovedTo );
+    g2->insert( c8, DeleteSubdir );
+    g2->insert( c9, DeleteFile );
+    g2->insert( c10, CreateSubdir );
+    g2->insert( c11, CreateFile );
+    g2->insert( c12, Unmount );
     connect( g2, SIGNAL( pressed(int) ), this, SLOT( modifierClicked(int) ) );
 
     g1 = new QVButtonGroup( "Add/Remove", hbox );
     QPushButton* plus1 = new QPushButton( "Add\n&Single", g1 );
     QPushButton* plus2 = new QPushButton( "Add\n&Multi", g1 );
     QPushButton* minus = new QPushButton( "&Remove\nIt!", g1 );
     g1->insert( plus1, 0 );
     g1->insert( plus2, 1 );
     g1->insert( minus, 2 );
     connect( plus1, SIGNAL( clicked() ), this, SLOT( addSingle() ) );
     connect( plus2, SIGNAL( clicked() ), this, SLOT( addMulti() ) );
     connect( minus, SIGNAL( clicked() ), this, SLOT( delTrigger() ) );
 
     g1->show();
     g2->show();
     l->show();
     hbox->show();
     vbox->show();
     showMainWidget( vbox );
   }
 
     void DemoApp::addTrigger( bool multi )
     {
         if ( !m )
         {
             QMessageBox::warning( 0, "Add Trigger", "<p>Can't add trigger without at least one selected trigger type</p>", "&Sorry", 0 );
             return;
         }
 
         QString filename = OFileDialog::getOpenFileName( OFileSelector::ExtendedAll );
         if ( !filename.isEmpty() )
         {
             odebug << "Filename = " << filename << oendl;
 
             int fntype = m;
-            if ( multi ) fntype |=(int) Multi;
-
             QString modifier = QString().sprintf( " = 0x%08x", fntype );
             new OListViewItem( l, filename, multi ? "MULTI" : "SINGLE", modifier );
             if ( !multi )
                 OFileNotification::singleShot( filename, this, SLOT( trigger() ), (OFileNotificationType) fntype );
             else
-                OFileNotification::singleShot( filename, this, SLOT( trigger() ), (OFileNotificationType) fntype );
+                odebug << "not yet implemented..." << oendl;
         }
         else
         {
             odebug << "cancelled." << oendl;
         }
     }
 
     void DemoApp::modifierClicked( int modifier ) { m = static_cast<OFileNotificationType>( (int)m ^ int(modifier) ); };
     void DemoApp::addSingle() { addTrigger(); };
     void DemoApp::addMulti() { addTrigger( true ); };
 
     void DemoApp::delTrigger()
     {
         QListViewItem* item = l->selectedItem();
         if ( !item )
         {
             QMessageBox::warning( 0, "Del Trigger", "<p>No trigger selected!</p>", "&Sorry", 0 );
             return;
         }
         else
         {
             QString filename( item->text( 0 ) );
             odebug << "Filename = " << filename << oendl;
         }
     }
 
     void DemoApp::trigger()
     {
         owarn << "FIRE!" << oendl;
     }
 
 int main( int argc, char** argv )
 {
     DemoApp* app = new DemoApp( argc, argv );
     app->exec();
 
     return 0;
 
 }
 
diff --git a/libopie2/opiecore/ofilenotify.cpp b/libopie2/opiecore/ofilenotify.cpp
index 2a9bb8c..c221e58 100644
--- a/libopie2/opiecore/ofilenotify.cpp
+++ b/libopie2/opiecore/ofilenotify.cpp
@@ -1,322 +1,228 @@
 /*
                             This file is part of the Opie Project
-             =.             Copyright (C) 2004 Michael 'Mickey' Lauer <mickey@Vanille.de>
+             =.             Copyright (C) 2004-2005 Michael 'Mickey' Lauer <mickey@Vanille.de>
            .=l.             Copyright (C) The Opie Team <opie-devel@handhelds.org>
           .>+-=
 _;:,     .>    :=|.         This program is free software; you can
 .> <`_,   >  .   <=         redistribute it and/or  modify it under
 :`=1 )Y*s>-.--   :          the terms of the GNU Library General Public
 .="- .-=="i,     .._        License as published by the Free Software
-- .   .-<_>     .<>         Foundation; either version 2 of the License,
-    ._= =}       :          or (at your option) any later version.
+- .   .-<_>     .<>         Foundation; version 2 of the License.
+    ._= =}       :
    .%`+i>       _;_.
    .i_,=:_.      -<s.       This program is distributed in the hope that
     +  .  -:.       =       it will be useful,  but WITHOUT ANY WARRANTY;
    : ..    .:,     . . .    without even the implied warranty of
    =_        +     =;=|`    MERCHANTABILITY or FITNESS FOR A
  _.=:.       :    :=>`:     PARTICULAR PURPOSE. See the GNU
 ..}^=.=       =       ;     Library General Public License for more
 ++=   -.     .`     .:      details.
 :     =  ...= . :.=-
 -.   .:....=;==+<;          You should have received a copy of the GNU
  -_. . .   )=.  =           Library General Public License along with
    --        :-=`           this library; see the file COPYING.LIB.
                             If not, write to the Free Software Foundation,
                             Inc., 59 Temple Place - Suite 330,
                             Boston, MA 02111-1307, USA.
 */
 
 #include "ofilenotify.h"
 using namespace Opie::Core;
 
 /* OPIE */
 
 /* QT */
 #include <qobject.h>
+#include <qsocketnotifier.h>
 #include <qsignal.h>
 #include <qintdict.h>
 #include <qdir.h>
 
 /* STD */
 #include <sys/types.h>
 #include <sys/stat.h>
-#include <assert.h>
+#include <sys/ioctl.h>
 #include <fcntl.h>
+#include <assert.h>
 #include <string.h>
 #include <errno.h>
 #include <unistd.h>
 
 static QIntDict<OFileNotification> notification_list;
 
+QSocketNotifier* OFileNotification::_sn;
+int OFileNotification::_fd = -1;
+
+#define INOTIFY_DEVICE "/dev/inotify"
+
 namespace Opie {
 namespace Core {
 
 OFileNotification::OFileNotification( QObject* parent, const char* name )
-                  :QObject( parent, name ), _active( false )
+                  :QObject( parent, name ), _active( false ), _multi( true )
 {
     qDebug( "OFileNotification::OFileNotification()" );
 }
 
 
 OFileNotification::~OFileNotification()
 {
+    stop();
     qDebug( "OFileNotification::~OFileNotification()" );
 }
 
 
 bool OFileNotification::isActive() const
 {
     return _active;
 }
 
 
-int OFileNotification::start( const QString& path, bool sshot, OFileNotificationType type )
+int OFileNotification::watch( const QString& path, bool sshot, OFileNotificationType type )
 {
-    _path = QString::null;
-    _fd = 0;
-    if ( _active ) stop();
-    QString dirpath;
-
-    // check if path exists and whether it is a file or a directory, if it exists at all
-    int result = ::stat( (const char*) path, &_stat );
-    if ( !(type & Create) && result == -1 )
-    {
-        qWarning( "OFileNotification::start(): Can't stat '%s': %s.", (const char*) path, strerror( errno ) );
-        return -1;
-    }
-
-    // if it is not a directory, we need to find out in which directory the file is
-    bool isDirectory = S_ISDIR( _stat.st_mode );
-    if ( !isDirectory )
-    {
-        int slashpos;
-        slashpos = path.findRev( '/' );
-        if ( slashpos > 0 )
-        {
-            _path = path;
-            dirpath = path.left( slashpos );
-        }
-    }
-    else /* isDirectory */
-    {
-        qWarning( "FIXME FIXME FIXME = Directory Notification Not Yet Implemented!" );
-        _path = path;
-        dirpath = path;
-        assert( 0 );
-    }
-
-    int fd = ::open( (const char*) dirpath, O_RDONLY );
-    if ( fd != -1 )
+    if ( QFile::exists( path ) )
     {
         if ( notification_list.isEmpty() )
         {
-            OFileNotification::registerSignalHandler();
+            OFileNotification::registerEventHandler();
         }
 
-        result = ::fcntl( fd, F_SETSIG, SIGRTMIN );
-        if ( result == -1 )
-        {
-            qWarning( "OFileNotification::start(): Can't subscribe to '%s': %s.", (const char*) dirpath, strerror( errno ) );
-            return -1;
-        }
-        if ( !sshot ) type = static_cast<OFileNotificationType>( (int) type | (int) Multi );
-        result = ::fcntl( fd, F_NOTIFY, type );
-        if ( result == -1 )
+        struct inotify_watch_request iwr;
+        ::memset( &iwr, 0, sizeof iwr );
+        iwr.name = const_cast<char*>( (const char*) path );
+        iwr.mask = type;
+
+        _wd = ::ioctl( OFileNotification::_fd, INOTIFY_WATCH, &iwr );
+
+        if ( _wd < 0 )
         {
-            qWarning( "OFileNotification::start(): Can't subscribe to '%s': %s.", (const char*) dirpath, strerror( errno ) );
+            qWarning( "OFileNotification::watch(): inotify can't watch '%s': %s.", (const char*) path, strerror( errno ) );
             return -1;
         }
-        qDebug( "OFileNotification::start(): Subscribed for changes to %s (fd = %d, mask = 0x%0x)", (const char*) dirpath, fd, type );
-        notification_list.insert( fd, this );
+
+        notification_list.insert( _wd, this );
+        _multi = !sshot;
         _type = type;
-        _fd = fd;
         _active = true;
-        ::memset( &_stat, 0, sizeof _stat );
-        ::stat( _path, &_stat );
-        return fd;
+        qDebug( "OFileNotification::watch(): watching '%s' [wd=%d].", (const char*) path, _wd );
+        return _wd;
     }
     else
     {
-        qWarning( "OFileNotification::start(): Error with path '%s': %s.", (const char*) dirpath, strerror( errno ) );
+        qWarning( "OFileNotification::watch(): Can't watch '%s': %s.", (const char*) path, strerror( errno ) );
         return -1;
     }
 }
 
 
 void OFileNotification::stop()
 {
-    if ( !_active ) return;
-
-    int result = ::fcntl( _fd, F_NOTIFY, 0 );
-    if ( result == -1 )
-    {
-        qWarning( "OFileNotification::stop(): Can't remove subscription to '%s': %s.", (const char*) _path, strerror( errno ) );
-    }
-    else
+    notification_list.remove( _wd );
+    _path = QString::null;
+    _wd = 0;
+    _active = false;
+    if ( notification_list.isEmpty() )
     {
-        ::close( _fd );
-        _type = Single;
-        _path = QString::null;
-        _fd = 0;
-        _active = false;
+        OFileNotification::unregisterEventHandler();
     }
 }
 
 
 OFileNotificationType OFileNotification::type() const
 {
     return _type;
 }
 
 
 QString OFileNotification::path() const
 {
     return _path;
 }
 
 
-int OFileNotification::fileno() const
+bool OFileNotification::activate()
 {
-    return _fd;
+    emit triggered();
+    _signal.activate();
+    if ( !_multi ) stop();
+    return true;
 }
 
 
-bool OFileNotification::activate()
+void OFileNotification::singleShot( const QString& path, QObject* receiver, const char* member, OFileNotificationType type )
 {
-    if ( hasChanged() )
-    {
-        emit triggered();
-        _signal.activate();
-        return true;
-    }
-    else
-        return false;
+    OFileNotification* ofn = new OFileNotification();
+    ofn->_signal.connect( receiver, member );
+    ofn->watch( path, true, type );
 }
 
 
-bool OFileNotification::hasChanged()
+void OFileNotification::inotifyEventHandler()
 {
-    bool c = false;
+    qWarning( "OFileNotification::__eventHandler(): reached." );
 
-    struct stat newstat;
-    ::memset( &newstat, 0, sizeof newstat );
-    int result = ::stat( _path, &newstat ); // may fail if file has been renamed or deleted. that doesn't matter :)
+    char buffer[16384];
+    size_t buffer_i;
+    struct inotify_event *pevent, *event;
+    ssize_t r;
+    size_t event_size;
+    int count = 0;
 
-    qDebug( "result of newstat call is %d (%s=%d)", result, result == -1 ? strerror( errno ) : "success", errno );
-    qDebug( "stat.atime = %0lx, newstat.atime = %0lx", (long)_stat.st_atime, (long)newstat.st_atime );
-    qDebug( "stat.mtime = %0lx, newstat.mtime = %0lx", (long)_stat.st_mtime, (long)newstat.st_mtime );
-    qDebug( "stat.ctime = %0lx, newstat.ctime = %0lx", (long)_stat.st_ctime, (long)newstat.st_ctime );
+    r = ::read(_fd, buffer, 16384);
 
-    if ( !c && (_type & Create) &&
-        (long)_stat.st_atime == 0 && (long)_stat.st_mtime == 0 && (long)_stat.st_ctime == 0 &&
-        (long)newstat.st_atime > 0 && (long)newstat.st_mtime > 0 && (long)newstat.st_ctime > 0)
-    {
-        qDebug( "OFileNotification::hasChanged(): file has been created" );
-        c = true;
-    }
-    if ( !c && (_type & (Delete|Rename)) && (long)newstat.st_atime == 0 && (long)newstat.st_mtime == 0 && (long)newstat.st_ctime == 0)
-    {
-        qDebug( "OFileNotification::hasChanged(): file has been deleted or renamed" );
-        c = true;
-    }
-    if ( !c && (_type & Access) && (long)_stat.st_atime < (long)newstat.st_atime )
-    {
-        qDebug( "OFileNotification::hasChanged(): atime changed" );
-        c = true;
-    }
-    if ( !c && (_type & Modify) && (long)_stat.st_mtime < (long)newstat.st_mtime )
-    {
-        qDebug( "OFileNotification::hasChanged(): mtime changed" );
-        c = true;
-    }
-    if ( !c && (_type & Attrib) && (long)_stat.st_ctime < (long)newstat.st_ctime )
+    if ( r <= 0 )
+        return;
+
+    buffer_i = 0;
+    while ( buffer_i < r )
     {
-        qDebug( "OFileNotification::hasChanged(): ctime changed" );
-        c = true;
-    }
+            /* Parse events and queue them ! */
+            pevent = (struct inotify_event *)&buffer[buffer_i];
+            event_size = sizeof(struct inotify_event) + pevent->len;
+            qDebug( "pevent->len = %d\n", pevent->len);
 
-    return c;
-}
+            OFileNotification* fn = notification_list[ pevent->wd ];
+            if ( fn )
+                fn->activate();
+            else
+                assert( false );
 
+            //event = malloc(event_size);
+            //memmove(event, pevent, event_size);
+            //queue_enqueue(event, q);
+            buffer_i += event_size;
+            count++;
+    }
 
-void OFileNotification::singleShot( const QString& path, QObject* receiver, const char* member, OFileNotificationType type )
-{
-    OFileNotification* ofn = new OFileNotification();
-    ofn->_signal.connect( receiver, member );
-    ofn->start( path, true, type );
+    qDebug( "received %d events...", count );
+    return;
 }
 
 
-void OFileNotification::__signalHandler( int sig, siginfo_t *si, void *data )
+bool OFileNotification::registerEventHandler()
 {
-    Q_UNUSED( sig )
-    Q_UNUSED( data )
-    qWarning( "OFileNotification::__signalHandler(): reached." );
-    int fd = si->si_fd;
-    OFileNotification* fn = notification_list[fd];
-    if ( fn )
-    {
-        // check if it really was the file (dnotify triggers on directory granularity, not file granularity)
-        if ( !fn->activate() )
-        {
-            qDebug( "OFileNotification::__signalHandler(): false alarm ;) Restarting the trigger (if it was single)..." );
-            if ( !(fn->type() & Multi ) )
-            {
-                int result = ::fcntl( fn->fileno(), F_NOTIFY, fn->type() );
-                if ( result == -1 )
-                {
-                    qWarning( "OFileNotification::__signalHandler(): Can't restart the trigger: %s.", strerror( errno ) );
-                }
-            }
-            return;
-        }
-        #if 1
-        if ( !(fn->type() & Multi) )
-        {
-            qDebug( "OFileNotification::__signalHandler(): '%d' was singleShot. Removing from list.", fd );
-            notification_list.remove( fd );
-            if ( notification_list.isEmpty() )
-            {
-                OFileNotification::unregisterSignalHandler();
-            }
-        }
-        #endif
-    }
-    else
+    OFileNotification::_fd = ::open( INOTIFY_DEVICE, O_RDONLY );
+    if ( OFileNotification::_fd < 0 )
     {
-        qWarning( "OFileNotification::__signalHandler(): D'oh! Called without fd in notification_list. Race condition?" );
+        qWarning( "OFileNotification::registerEventHandler(): couldn't register event handler: %s", strerror( errno ) );
+        return false;
     }
-}
 
+    OFileNotification::_sn = new QSocketNotifier( _fd, QSocketNotifier::Read, this, "inotify event" );
+    connect( OFileNotification::_sn, SIGNAL( activated(int) ), this, SLOT( inotifyEventHandler() ) );
 
-bool OFileNotification::registerSignalHandler()
-{
-    struct sigaction act;
-    act.sa_sigaction = OFileNotification::__signalHandler;
-    ::sigemptyset( &act.sa_mask );
-    act.sa_flags = SA_SIGINFO;
-    if ( ::sigaction( SIGRTMIN, &act, NULL ) == -1 )
-    {
-        qWarning( "OFileNotification::registerSignalHandler(): couldn't register signal handler: %s", strerror( errno ) );
-        return false;
-    }
-    qDebug( "OFileNotification::registerSignalHandler(): done" );
+    qDebug( "OFileNotification::registerEventHandler(): done" );
     return true;
 }
 
 
-void OFileNotification::unregisterSignalHandler()
+void OFileNotification::unregisterEventHandler()
 {
-    struct sigaction act;
-    act.sa_sigaction = ( void (*)(int, siginfo_t*, void*) ) SIG_DFL;
-    ::sigemptyset( &act.sa_mask );
-    if ( ::sigaction( SIGRTMIN, &act, NULL ) == -1 )
-    {
-        qWarning( "OFileNotification::unregisterSignalHandler(): couldn't deregister signal handler: %s", strerror( errno ) );
-    }
-    qDebug( "OFileNotification::unregisterSignalHandler(): done" );
+    if ( OFileNotification::_fd )
+    ::close( OFileNotification::_fd );
+    qDebug( "OFileNotification::unregisterEventHandler(): done" );
 }
 
 
 }
 }
diff --git a/libopie2/opiecore/ofilenotify.h b/libopie2/opiecore/ofilenotify.h
index d820f7c..e3621cf 100644
--- a/libopie2/opiecore/ofilenotify.h
+++ b/libopie2/opiecore/ofilenotify.h
@@ -1,190 +1,197 @@
 /*
                             This file is part of the Opie Project
-             =.             Copyright (C) 2004 Michael 'Mickey' Lauer <mickey@Vanille.de>
+             =.             Copyright (C) 2004-2005 Michael 'Mickey' Lauer <mickey@Vanille.de>
            .=l.             Copyright (C) The Opie Team <opie-devel@handhelds.org>
           .>+-=
 _;:,     .>    :=|.         This program is free software; you can
 .> <`_,   >  .   <=         redistribute it and/or  modify it under
 :`=1 )Y*s>-.--   :          the terms of the GNU Library General Public
 .="- .-=="i,     .._        License as published by the Free Software
-- .   .-<_>     .<>         Foundation; either version 2 of the License,
-    ._= =}       :          or (at your option) any later version.
+- .   .-<_>     .<>         Foundation; version 2 of the License.
+    ._= =}       :
    .%`+i>       _;_.
    .i_,=:_.      -<s.       This program is distributed in the hope that
     +  .  -:.       =       it will be useful,  but WITHOUT ANY WARRANTY;
    : ..    .:,     . . .    without even the implied warranty of
    =_        +     =;=|`    MERCHANTABILITY or FITNESS FOR A
  _.=:.       :    :=>`:     PARTICULAR PURPOSE. See the GNU
 ..}^=.=       =       ;     Library General Public License for more
 ++=   -.     .`     .:      details.
 :     =  ...= . :.=-
 -.   .:....=;==+<;          You should have received a copy of the GNU
  -_. . .   )=.  =           Library General Public License along with
    --        :-=`           this library; see the file COPYING.LIB.
                             If not, write to the Free Software Foundation,
                             Inc., 59 Temple Place - Suite 330,
                             Boston, MA 02111-1307, USA.
 */
 
 #ifndef OFILENOTIFY_H
 #define OFILENOTIFY_H
 #if defined (__GNUC__) && (__GNUC__ < 3)
 #define _GNU_SOURCE
 #endif
 
 /* QT */
-#include <qobject.h>
+#include <qsocketnotifier.h>
 #include <qsignal.h>
 #include <qstring.h>
 
 /* STD */
-#include <signal.h>
-#include <fcntl.h>
+#include "inotify.h"
 
 namespace Opie {
 namespace Core {
 
 /*======================================================================================
  * OFileNotificationType
  *======================================================================================*/
 
 /**
  * @brief An enumerate for the different types of file notifications
  *
  * This enumerate provides a means to specify the type of events that you are interest in.
  * Valid values are:
  * <ul>
  *    <li>Access: The file was accessed (read)
  *    <li>Modify The file was modified (write,truncate)
- *    <li>Create = The file was created in the directory
- *    <li>Delete = The file was unlinked from directory
- *    <li>Rename = The file was renamed
  *    <li>Attrib = The file had its attributes changed (chmod,chown,chgrp)
+ *    <li>CloseWrite = Writable file was closed
+ *    <li>CloseNoWrite = Unwritable file was closed
+ *    <li>Open = File was opened
+ *    <li>MovedFrom = File was moved from X
+ *    <li>MovedTo = File was moved to Y
+ *    <li>DeleteSubdir = Subdir was deleted
+ *    <li>DeleteFile = Subfile was deleted
+ *    <li>CreateSubdir = Subdir was created
+ *    <li>CreateFile = Subfile was created
+ *    <li>DeleteSelf = Self was deleted
+ *    <li>Unmount = The backing filesystem was unmounted
  * </ul>
  *
  **/
 
-enum OFileNotificationType { Single = 0x0000000,
-                             Multi  = DN_MULTISHOT,
-                             Access = DN_ACCESS,
-                             Modify = DN_MODIFY,
-                             Create = DN_CREATE,
-                             Delete = DN_DELETE,
-                             Rename = DN_RENAME,
-                             Attrib = DN_ATTRIB };
+enum OFileNotificationType
+{
+    Access          =   IN_ACCESS,
+    Modify          =   IN_MODIFY,
+    Attrib          =   IN_ATTRIB,
+    CloseWrite      =   IN_CLOSE_WRITE,
+    CloseNoWrite    =   IN_CLOSE_NOWRITE,
+    Open            =   IN_OPEN,
+    MovedFrom       =   IN_MOVED_FROM,
+    MovedTo         =   IN_MOVED_TO,
+    DeleteSubdir    =   IN_DELETE_SUBDIR,
+    DeleteFile      =   IN_DELETE_FILE,
+    CreateSubdir    =   IN_CREATE_SUBDIR,
+    CreateFile      =   IN_CREATE_FILE,
+    DeleteSelf      =   IN_DELETE_SELF,
+    Unmount         =   IN_UNMOUNT,
+    _QueueOverflow  =   IN_Q_OVERFLOW,  /* Internal, don't use this in client code */
+    _Ignored        =   IN_IGNORED,     /* Internal, don't use this in client code */
+};
 
 /*======================================================================================
  * OFileNotification
  *======================================================================================*/
 
 /**
  * @brief Represents a file notification
  *
  * This class allows to watch for events happening to files.
- * It uses the dnotify kernel interface which is a very efficient signalling interface.
+ * It uses the inotify kernel interface
  *
- * @see <file:///usr/src/linux/Documentation/dnotify.txt>
+ * @see http://www.kernel.org/pub/linux/kernel/people/rml/inotify/
  *
  * @author Michael 'Mickey' Lauer <mickey@vanille.de>
  *
  **/
 
 class OFileNotification : public QObject
 {
   Q_OBJECT
 
   public:
     OFileNotification( QObject* parent = 0, const char* name = 0 );
     ~OFileNotification();
     /**
      * This static function calls a slot when an event with @a type happens to file @a path.
      *
      *  It is very convenient to use this function because you do not need to
      *  bother with a timerEvent or to create a local QTimer object.
      *
      *  Example:
      *  <pre>
      *
      *     #include <opie2/oapplication.h>
-     *     #include <opie2/onitify.h>
+     *     #include <opie2/ofilenotify.h>
      *     using namespace Opie::Core;
      *
      *     int main( int argc, char **argv )
      *     {
      *         OApplication a( argc, argv, "File Notification Example" );
      *         OFileNotification::singleShot( "/tmp/quit", &a, SLOT(quit()), Create );
      *         ... // create and show your widgets
      *         return a.exec();
      *     }
      *  </pre>
      *
-     * This sample program automatically terminates when the file "/tmp/quite" has been created.
+     * This sample program automatically terminates when the file "/tmp/quit" has been created.
      *
      *
      * The @a receiver is the receiving object and the @a member is the slot.
      **/
     static void singleShot( const QString& path, QObject* receiver, const char* member, OFileNotificationType type = Modify );
     /**
      * Starts to watch for @a type changes to @a path. Set @a sshot to True if you want to be notified only once.
      * Note that in that case it may be more convenient to use @ref OFileNotification::singleShot() then.
      **/
-    int start( const QString& path, bool sshot = false, OFileNotificationType type = Modify );
+    int watch( const QString& path, bool sshot = false, OFileNotificationType type = Modify );
     /**
      * Stop watching for file events.
      **/
     void stop();
     /**
      * @returns the notification type as set by @ref start().
      **/
     OFileNotificationType type() const;
     /**
      * @returns the path to the file being watched by this instance.
      **/
     QString path() const;
     /**
-     * @returns the UNIX file descriptor for the file being watched.
-     **/
-    int fileno() const;
-    /**
      * @returns if a file is currently being watched.
      **/
     bool isActive() const;
 
   signals:
     /**
      * This signal is emitted if an event happens of the specified type happens to the file being watched.
      **/
     void triggered();
 
   protected:
     bool activate();
-    virtual bool hasChanged();
-    static bool registerSignalHandler();
-    static void unregisterSignalHandler();
-    static void __signalHandler( int sig, siginfo_t *si, void *data );
+
+  private slots:
+    void inotifyEventHandler();
 
   private:
+    bool registerEventHandler();
+    void unregisterEventHandler();
+
     QString _path;
     OFileNotificationType _type;
     QSignal _signal;
-    int _fd;
     bool _active;
-    struct stat _stat;
-};
-
-#if 0
-
-class ODirectoryNotification : public OFileNotification
-{
-
-  public:
-      virtual bool hasChanged() { return true; };
+    bool _multi;
+    static QSocketNotifier* _sn;
+    int _wd; // inotify watch descriptor
+    static int _fd; // inotify device descriptor
 };
 
-#endif
-
 }
 }
 
 #endif