Skip to content
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

Add the example project from the README to quickly test out JAnsi #50

Merged
merged 2 commits into from
Jun 15, 2016
Merged

Add the example project from the README to quickly test out JAnsi #50

merged 2 commits into from
Jun 15, 2016

Conversation

sschuberth
Copy link
Contributor

No description provided.

@chirino
Copy link
Member

chirino commented Jun 14, 2016

Hey. Could you update this or to resolve the conflicts?

@sschuberth
Copy link
Contributor Author

@chirino Done.

@@ -129,7 +129,7 @@ public void close() throws IOException {
};
}

private static boolean isCygwin() {
private static boolean isXterm() {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this one is not about example
but it seems a good change: yes, given the implementation, this method should be renamed!
+1

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it's unrelated to the global topic of the PR, but then again that's why it's a separate commit. As it's such a small change I did not bother to explicitly mention it in the PR's description.

Strictly speaking this method does not check for Cygwin (as it e.g. does
not check to be on Windows) but only for XTerm (which in conjunction
with being on Windows is a sign of running under Cygwin).

Simplify the code a bit to get rid of the explicit null check long the
way.
This provides a minimal test executable for quickly trying out JAnsi. Run
it like

$ mvn --projects example exec:java
@chirino chirino merged commit 55c3817 into fusesource:master Jun 15, 2016
@chirino
Copy link
Member

chirino commented Jun 15, 2016

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants