removeEventListener result #63

Activity

1. tresf commented on Feb 5, 2020

   

   tresf

   on Feb 5, 2020

   More actions

   the method returns true, otherwise false

   Agreed that appears to be incorrect. Instead it either removes the listener or throws an exception. The documentation is incorrect.

   Sometimes I get "Event listener already added" error, but I don't know why.
   every time after working with port I perform removeEventListener. Can you help me?

   A unit test is required to know for sure, so you should provide a small code snippet which reproduces this. Worth noting, closing the port implicitly removes the listener, so you should not attempt to remove a listener after closure. The exception thrown in this scenario isn't the same as you're mentioning, so the best way to reproduce the issue is to know how to reproduce it.

   jssc/src/java/jssc/SerialPort.java

   Lines 1088 to 1092 in 6d09752

   	public boolean closePort() throws SerialPortException {
   	checkPortOpened("closePort()");
   	if(eventListenerAdded){
   	removeEventListener();
   	}
2. tresf

   added

   response required

   on Feb 5, 2020