Questionable Constructor implementation in AbstractExchange
Status: Inactive
Brought to you by:
grro
I am raising this as a bug. In the below implementation in the AbstractExchange,
The constructor parameter "workerpool" is never used in this particular invocation. Then what is the point in passing the same. Since, you are setting the "executor" instance variable from somewhere else.
protected AbstractExchange(AbstractExchange parentExchange, Executor workerpool) {
this.parentExchange = parentExchange;
executor = HttpUtils.newMultimodeExecutor();
}
There is no sufficient comments also to understand this miscellaneous kind of implementation.