New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ServerManager needs updating #665

Open
csmith opened this Issue Feb 25, 2016 · 1 comment

Comments

Projects
None yet
2 participants
@csmith
Member

csmith commented Feb 25, 2016

I'm pretty sure its handler to remove servers on close no longer works.

Worth taking a look to see if servers can be replaced with connections as well.

@csmith csmith self-assigned this Mar 3, 2016

@greboid

This comment has been minimized.

Show comment
Hide comment
@greboid

greboid Mar 27, 2016

Member

Error during invocation of message handler. Message handler threw exception
java.lang.IllegalArgumentException
at com.google.common.base.Preconditions.checkArgument(Preconditions.java:108)
at com.dmdirc.ui.WindowManager.removeWindow(WindowManager.java:260)
at com.dmdirc.ui.WindowManager.frameClosing(WindowManager.java:414)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at net.engio.mbassy.dispatch.ReflectiveHandlerInvocation.invokeHandler(ReflectiveHandlerInvocation.java:23)
at net.engio.mbassy.dispatch.ReflectiveHandlerInvocation.invoke(ReflectiveHandlerInvocation.java:49)
at net.engio.mbassy.dispatch.MessageDispatcher.dispatch(MessageDispatcher.java:30)
at net.engio.mbassy.dispatch.FilteredMessageDispatcher.dispatch(FilteredMessageDispatcher.java:41)
at net.engio.mbassy.subscription.Subscription.publish(Subscription.java:71)
at net.engio.mbassy.bus.MessagePublication.execute(MessagePublication.java:46)
at net.engio.mbassy.bus.MBassador.publish(MBassador.java:70)
at com.dmdirc.FrameContainer.close(FrameContainer.java:189)
at com.dmdirc.ServerManager.closeAll(ServerManager.java:155)
at com.dmdirc.addons.ui_swing.QuitWorker.doInBackground(QuitWorker.java:75)
at com.dmdirc.addons.ui_swing.QuitWorker.doInBackground(QuitWorker.java:38)
at javax.swing.SwingWorker$1.call(Unknown Source)
at java.util.concurrent.FutureTask.run(Unknown Source)
at javax.swing.SwingWorker.run(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.lang.Thread.run(Unknown Source)

Member

greboid commented Mar 27, 2016

Error during invocation of message handler. Message handler threw exception
java.lang.IllegalArgumentException
at com.google.common.base.Preconditions.checkArgument(Preconditions.java:108)
at com.dmdirc.ui.WindowManager.removeWindow(WindowManager.java:260)
at com.dmdirc.ui.WindowManager.frameClosing(WindowManager.java:414)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at net.engio.mbassy.dispatch.ReflectiveHandlerInvocation.invokeHandler(ReflectiveHandlerInvocation.java:23)
at net.engio.mbassy.dispatch.ReflectiveHandlerInvocation.invoke(ReflectiveHandlerInvocation.java:49)
at net.engio.mbassy.dispatch.MessageDispatcher.dispatch(MessageDispatcher.java:30)
at net.engio.mbassy.dispatch.FilteredMessageDispatcher.dispatch(FilteredMessageDispatcher.java:41)
at net.engio.mbassy.subscription.Subscription.publish(Subscription.java:71)
at net.engio.mbassy.bus.MessagePublication.execute(MessagePublication.java:46)
at net.engio.mbassy.bus.MBassador.publish(MBassador.java:70)
at com.dmdirc.FrameContainer.close(FrameContainer.java:189)
at com.dmdirc.ServerManager.closeAll(ServerManager.java:155)
at com.dmdirc.addons.ui_swing.QuitWorker.doInBackground(QuitWorker.java:75)
at com.dmdirc.addons.ui_swing.QuitWorker.doInBackground(QuitWorker.java:38)
at javax.swing.SwingWorker$1.call(Unknown Source)
at java.util.concurrent.FutureTask.run(Unknown Source)
at javax.swing.SwingWorker.run(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
at java.lang.Thread.run(Unknown Source)

@csmith csmith removed their assignment Jun 2, 2016

@csmith csmith added the Type: Bug label Jul 7, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment