more work on the new packet viewer

3 files changed, 80 insertions, 16 deletions

diff --git a/noncore/net/wellenreiter/gui/packetview.cpp b/noncore/net/wellenreiter/gui/packetview.cpp
index e0e626c..3d3aa18 100644
--- a/noncore/net/wellenreiter/gui/packetview.cpp
+++ b/noncore/net/wellenreiter/gui/packetview.cpp
@@ -8,62 +8,114 @@
 ** Foundation and appearing in the file LICENSE.GPL included in the
 ** packaging of this file.
 **
 ** This file is provided AS IS with NO WARRANTY OF ANY KIND, INCLUDING THE
 ** WARRANTY OF DESIGN, MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE.
 **
 **********************************************************************/
 
 #include "packetview.h"
 
 /* OPIE */
 #include <opie2/opcap.h>
+#include <opie2/odebug.h>
+#include <opie2/olistview.h>
 
 /* QT */
-#include <qtextview.h>
-#include <qspinbox.h>
+#include <qfont.h>
 #include <qlabel.h>
 #include <qlayout.h>
 #include <qlist.h>
+#include <qlistview.h>
+#include <qobjectlist.h>
+#include <qspinbox.h>
+#include <qtextview.h>
 
 using namespace Opie::Net;
+using namespace Opie::Core;
+using namespace Opie::Ui;
+
 PacketView::PacketView( QWidget * parent, const char * name, WFlags f )
            :QFrame( parent, name, f )
 {
     _number = new QSpinBox( this );
     _number->setPrefix( "Packet # " );
     _label = new QLabel( this );
     _label->setText( "eth0 2004/03/08 - 00:00:21" );
-    _list = new QLabel( this );
+    
+    _list = new OListView( this );
+    _list->addColumn( "#" );
+    _list->addColumn( "Packet Type" );
+    _list->setColumnAlignment( 0, Qt::AlignCenter );
+    _list->setColumnAlignment( 1, Qt::AlignLeft );
+    _list->setAllColumnsShowFocus( true );
+    _list->setFont( QFont( "Fixed", 8 ) );
+    
     _hex = new QTextView( this );
+    _hex->setFont( QFont( "Fixed", 8 ) );
 
     QVBoxLayout* vb = new QVBoxLayout( this, 2, 2 );
     QHBoxLayout* hb = new QHBoxLayout( vb, 2 );
     hb->addWidget( _label );
     hb->addWidget( _number );
     vb->addWidget( _list );
     vb->addWidget( _hex );
 
     _packets.setAutoDelete( true );
+    
+    connect( _number, SIGNAL( valueChanged( int ) ), this, SLOT( showPacket( int ) ) );
+}
 
-    _list->setText( "<b>[ 802.11 [ LLC [ IP [ UDP [ DHCP ] ] ] ] ]</b>" );
-};
-
-void PacketView::add( OPacket* p )
+void PacketView::add( const OPacket* p )
 {
     _packets.append( p );
-};
+    // Add Circular Buffer and check for number of elements here
+}
+
+void PacketView::showPacket( int number )
+{
+    _list->clear();
+    _hex->setText("");
+    const OPacket* p = _packets.at( number );
+    
+    if ( p )
+    {
+        _doSubPackets( const_cast<QObjectList*>( p->children() ), 0 );
+        _doHexPacket( p );
+    }
+    else
+    {
+        qDebug( "D'oh! No packet!" );
+    }
+}
+
+void PacketView::_doSubPackets( QObjectList* l, int counter )
+{
+    if (!l) return;
+    QObject* o = l->first();
+    while ( o )
+    {
+        new OListViewItem( _list, QString::number( counter++ ), o->name() );
+        _doSubPackets( const_cast<QObjectList*>( o->children() ), counter );
+        o = l->next();
+    }
+}    
+
+void PacketView::_doHexPacket( const OPacket* p )
+{      
+    _hex->setText( p->dump( 16 ) );
+}
 
 const QString PacketView::getLog() const
 {
 }
 
 void PacketView::clear()
 {
     _packets.clear();
     _number->setMinValue( 0 );
     _number->setMaxValue( 0 );
     _label->setText( "---" );
-    _list->setText( " <b>-- no Packet available --</b> " );
+    _list->clear();
     _hex->setText( " <i>-- no Packet available --</i> " );
 }
 
diff --git a/noncore/net/wellenreiter/gui/packetview.h b/noncore/net/wellenreiter/gui/packetview.h
index 938aa19..affedab 100644
--- a/noncore/net/wellenreiter/gui/packetview.h
+++ b/noncore/net/wellenreiter/gui/packetview.h
@@ -10,40 +10,49 @@
 **
 ** This file is provided AS IS with NO WARRANTY OF ANY KIND, INCLUDING THE
 ** WARRANTY OF DESIGN, MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE.
 **
 **********************************************************************/
 
 #ifndef PACKETVIEW_H
 #define PACKETVIEW_H
 
 #include <qlist.h>
 #include <qframe.h>
 
+class QLabel;
 class QString;
 class QSpinBox;
-class QLabel;
 class QTextView;
+class QObjectList;
 namespace Opie {namespace Net {class OPacket;}}
+namespace Opie {namespace Ui {class OListView;}}
 
 class PacketView: public QFrame
 {
-
+  Q_OBJECT 
+  
   public:
     PacketView( QWidget * parent = 0, const char * name = "PacketView", WFlags f = 0 );
 
-    void add( Opie::Net::OPacket* p );
+    void add( const Opie::Net::OPacket* p );
     const QString getLog() const;
     void clear();
+    
+  public slots:
+    void showPacket( int number );
 
   protected:
-
     QSpinBox* _number;
     QLabel* _label;
-    QLabel* _list;
+    Opie::Ui::OListView* _list;
     QTextView* _hex;
-    QList<Opie::Net::OPacket> _packets;
+    QList<const Opie::Net::OPacket> _packets;
+    
+  protected:
+    void _doSubPackets( QObjectList*, int );
+    void _doHexPacket( const Opie::Net::OPacket* );
 
 };
 
 #endif
 
diff --git a/noncore/net/wellenreiter/gui/wellenreiter.cpp b/noncore/net/wellenreiter/gui/wellenreiter.cpp
index c8d77a7..fff7c35 100644
--- a/noncore/net/wellenreiter/gui/wellenreiter.cpp
+++ b/noncore/net/wellenreiter/gui/wellenreiter.cpp
@@ -80,24 +80,25 @@ Wellenreiter::Wellenreiter( QWidget* parent )
 
     #ifdef QWS
     QString sys;
     sys.sprintf( "(i) Running on '%s'.", (const char*) ODevice::inst()->systemString() );
     _system = ODevice::inst()->system();
     logwindow->log( sys );
     #endif
 
     netview->setColumnWidthMode( 1, QListView::Manual );
     connect( netview, SIGNAL( joinNetwork(const QString&,const QString&,int,const QString&) ),
              this, SLOT( joinNetwork(const QString&,const QString&,int,const QString&) ) );
     pcap = new OPacketCapturer();
+    pcap->setAutoDelete( false );
 
     gps = new GPS( this );
 
     QTimer::singleShot( 1000, this, SLOT( initialTimer() ) );
 
 }
 
 
 Wellenreiter::~Wellenreiter()
 {
     delete pcap;
 }
@@ -651,25 +652,27 @@ void Wellenreiter::startClicked()
 
 void Wellenreiter::timerEvent( QTimerEvent* )
 {
     qDebug( "Wellenreiter::timerEvent()" );
     OPacket* p = pcap->next();
     if ( !p ) // no more packets available
     {
         stopClicked();
     }
     else
     {
         receivePacket( p );
-        delete p;
+        // We no longer delete packets here. Ownership of the packets is
+        // transferred to the PacketView.
+        //delete p;
     }
 }
 
 
 void Wellenreiter::doAction( const QString& action, const QString& protocol, OPacket* p )
 {
     #ifdef QWS
     if ( action == "TouchSound" )
         ODevice::inst()->playTouchSound();
     else if ( action == "AlarmSound" )
         ODevice::inst()->playAlarmSound();
     else if ( action == "KeySound" )