Debug window Clear ALL Packets patch

Smack Smack Dev
1

I noticed that clear all packets button in the debug window wasn’t working in Spark (only cleared half of them) so I made this patch:

Eclipse Workspace Patch 1.0

#P Smack

Index: source/org/jivesoftware/smackx/debugger/EnhancedDebugger.java

===================================================================

— source/org/jivesoftware/smackx/debugger/EnhancedDebugger.java (revision 12410)

+++ source/org/jivesoftware/smackx/debugger/EnhancedDebugger.java (working copy)

@@ -326,11 +326,7 @@

@Override

public void actionPerformed(ActionEvent e) {

  •    for(int i=0; i< messagesTable.getRowCount();i++)

  •    {

  •    messagesTable.removeRow(i);

  •    }

  •        messagesTable.setRowCount(0);

}

});

2

Can you post it as a file (*.patch)? You can upload files in Advanced editor mode.

3

thx for the patch, submitted into smack_3_2_0 branch

4

Here it is as a .patch file
smackDebugWindowClearAll.patch.zip (464 Bytes)

5

Looks like Wolf has already applied it. It is always better to attach a patch file, because it is easier to upload this to the bug tracker and i won’t introduce any invisible wrong chars by copying code from the page