what it does:
-adds new listener as requested
-throws exception because it is unable to connect to Skype
what it should do:
add new listener as requested
throw exception because it is unable to connect to Skype
remove added listener because there were an exception
Why it should work in this way:
-usually user expects no state changing in case of exception
-there is no javadoc saying "listener will be added EVEN IN CASE OF ERROR" (what error? any error? list of specific errors?)
what it does: -adds new listener as requested -throws exception because it is unable to connect to Skype
what it should do:
Why it should work in this way: -usually user expects no state changing in case of exception -there is no javadoc saying "listener will be added EVEN IN CASE OF ERROR" (what error? any error? list of specific errors?)