Useless `main` method in subclass of Action
aloyszhang opened this issue · comments
AloysZhang commented
I found some main
method just sleep 100000000ms in a new thread in some subClass of Action
,
public static void main(String[] args) {
new Thread() {
@Override
public void run() {
try {
Thread.sleep(100000000);
} catch (InterruptedException e) {
e.printStackTrace();
}
}
}.start();
}
It seems useless. and I think wo can remove these code. What do think about it ?
@gosonzhang
tison commented
Make sense to me. Also I'm curious the history why we introduce such code snippets.
Goson Zhang commented
This should be used for unit testing a long time ago, not removed.
Can you fix it?
AloysZhang commented
@gosonzhang ok, I'll contribute a PR.
Goson Zhang commented
thanks