Ignored Exceptions about dbus-java HOT 3 CLOSED

As you might know, I'm not the original author of the code. I just took it and re-organized, cleaned various java warnings, added generics here and there and created a maven project out of it.

There are only a few things added either by myself or by some other committers (like PropertiesChangedListener).

So because of that, I expect that there are lots of ugly parts and code smells within this library.
I also really dislike the if (EXCEPTION_DEBUG) stuff, but I'm just to busy to cleanup all this mess.

But hey, if you don't like it, fix it and I will happily merge your changes.

from dbus-java.

I just merged your changes and I also removed the 'EXCEPTION_DEBUG' flag.

It might be a good idea to review the whole code and clean this exception mess. Exceptions are thrown everywhere, logged and then being wrapped in another exception to be thrown, which in turn will be logged and maybe wrapped in the next method.. This is really bad style.
Also catching RuntimeExceptions does not seem to be a good idea to me.

from dbus-java.

Yeah a lot of waste lays around when code is untouched for 10 years. I don't know what's up with freedesktop.org, but if they are still active we could try to push this officially to get some traction and hints.

from dbus-java.