spywhere / Javatar

Java Development Plugin for Sublime Text 3
https://javatar.readthedocs.org/
Other
170 stars 20 forks source link

Set Program Arguments Improvements #32

Closed JesseEmond closed 9 years ago

JesseEmond commented 9 years ago
Mause commented 9 years ago

show_delayed_status might be worth throwing into an appropriate utils module, as it can likely be used elsewhere

JesseEmond commented 9 years ago

I did that initially but noticed that it would only be useful in its current state in one other location (javatar_operation (@organize_step_six)) that used the same parameters (i.e. delay to None, require_java to False, sublime timeout of 500ms).

Do you think it is still worth changing?

Mause commented 9 years ago

I guess it depends on whether that pattern is used elsewhere

Mause commented 9 years ago

A quick search for None, False), 500 says no, so i guess it doesn't matter