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
@@ -17,15 +17,23 @@
 
 /* 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 )
 {
@@ -33,8 +41,17 @@ PacketView::PacketView( QWidget * parent, const char * name, WFlags f )
     _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 );
@@ -44,14 +61,49 @@ PacketView::PacketView( QWidget * parent, const char * name, WFlags f )
     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
 {
@@ -63,7 +115,7 @@ void PacketView::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
@@ -19,29 +19,38 @@
 #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* );
 
 };
 
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
@@ -89,6 +89,7 @@ Wellenreiter::Wellenreiter( QWidget* parent )
     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 );
 
@@ -660,7 +661,9 @@ void Wellenreiter::timerEvent( QTimerEvent* )
     else
     {
         receivePacket( p );
-        delete p;
+        // We no longer delete packets here. Ownership of the packets is
+        // transferred to the PacketView.
+        //delete p;
     }
 }