From 16bb80bd16e765020d1c6bb5cb7eaecf8d91d940 Mon Sep 17 00:00:00 2001
From: Anton Keks <anton@codeborne.com>
Date: Tue, 11 Jun 2024 22:43:15 +0300
Subject: [PATCH] trim() the entered/pasted IPs without triggering modify
 listeners

---
 .../ipscan/gui/feeders/RandomFeederGUI.java   |  4 ++--
 .../ipscan/gui/feeders/RangeFeederGUI.java    | 23 ++++++++-----------
 2 files changed, 11 insertions(+), 16 deletions(-)

diff --git a/src/net/azib/ipscan/gui/feeders/RandomFeederGUI.java b/src/net/azib/ipscan/gui/feeders/RandomFeederGUI.java
index 2c2fcda1e..94e61b91f 100644
--- a/src/net/azib/ipscan/gui/feeders/RandomFeederGUI.java
+++ b/src/net/azib/ipscan/gui/feeders/RandomFeederGUI.java
@@ -97,12 +97,12 @@ public void initialize() {
 	}
 
 	public Feeder createFeeder() {
-		feeder = new RandomFeeder(ipPrototypeText.getText(), ipMaskCombo.getText(), countSpinner.getSelection());
+		feeder = new RandomFeeder(ipPrototypeText.getText().trim(), ipMaskCombo.getText().trim(), countSpinner.getSelection());
 		return feeder;
 	}
 	
 	public String[] serialize() {
-		return new String[] {ipPrototypeText.getText(), ipMaskCombo.getText(), String.valueOf(countSpinner.getSelection())};
+		return new String[] {ipPrototypeText.getText().trim(), ipMaskCombo.getText().trim(), String.valueOf(countSpinner.getSelection())};
 	}
 
 	public void unserialize(String[] parts) {
diff --git a/src/net/azib/ipscan/gui/feeders/RangeFeederGUI.java b/src/net/azib/ipscan/gui/feeders/RangeFeederGUI.java
index 5c83fb15f..fce5b65c3 100644
--- a/src/net/azib/ipscan/gui/feeders/RangeFeederGUI.java
+++ b/src/net/azib/ipscan/gui/feeders/RangeFeederGUI.java
@@ -11,7 +11,9 @@
 import net.azib.ipscan.feeders.RangeFeeder;
 import net.azib.ipscan.gui.actions.FeederActions;
 import org.eclipse.swt.SWT;
-import org.eclipse.swt.events.*;
+import org.eclipse.swt.events.ModifyEvent;
+import org.eclipse.swt.events.ModifyListener;
+import org.eclipse.swt.events.SelectionEvent;
 import org.eclipse.swt.graphics.Rectangle;
 import org.eclipse.swt.layout.GridData;
 import org.eclipse.swt.layout.GridLayout;
@@ -76,7 +78,7 @@ public void initialize() {
         toLabel.setText(getLabel("feeder.range.to"));
 
 		startIPText.addModifyListener(new StartIPModifyListener());
-        endIPText.addKeyListener(new EndIPKeyListener());
+        endIPText.addModifyListener(new EndIPModifyListener());
 
 		FeederActions.HostnameButton hostnameListener = new FeederActions.HostnameButton(hostnameText, startIPText, netmaskCombo) {
 			public void widgetSelected(SelectionEvent event) {
@@ -130,13 +132,11 @@ protected void setInterfaceAddress(InterfaceAddress ifAddr) {
 	}
 
 	public Feeder createFeeder() {
-		startIPText.setText(startIPText.getText().trim());
-		endIPText.setText(endIPText.getText().trim());
-		return feeder = new RangeFeeder(startIPText.getText(), endIPText.getText());
+		return feeder = new RangeFeeder(startIPText.getText().trim(), endIPText.getText().trim());
 	}
 	
 	public String[] serialize() {
-		return new String[] {startIPText.getText(), endIPText.getText()};
+		return new String[] {startIPText.getText().trim(), endIPText.getText().trim()};
 	}
 
 	public void unserialize(String[] parts) {
@@ -150,13 +150,10 @@ public String[] serializePartsLabels() {
 		return new String[] {"feeder.range.startIP", "feeder.range.endIP"};
 	}
 
-	final class EndIPKeyListener implements KeyListener {
-		public void keyPressed(KeyEvent e) {
+	final class EndIPModifyListener implements ModifyListener {
+		@Override public void modifyText(ModifyEvent modifyEvent) {
 			isEndIPUnedited = false;
 		}
-
-		public void keyReleased(KeyEvent e) {
-		}
 	}
 	
 	final class NetmaskResetListener implements Listener {
@@ -169,9 +166,7 @@ public void handleEvent(Event event) {
 
 	final class StartIPModifyListener implements ModifyListener {
 		public void modifyText(ModifyEvent e) {
-			if (isEndIPUnedited) {
-				endIPText.setText(startIPText.getText());
-			}
+			if (isEndIPUnedited) endIPText.setText(startIPText.getText());
 		}
 	}