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

Flush output for each command when using !record option #26

Closed
Sachini opened this issue Dec 8, 2014 · 3 comments
Closed

Flush output for each command when using !record option #26

Sachini opened this issue Dec 8, 2014 · 3 comments

Comments

@Sachini
Copy link
Contributor

Sachini commented Dec 8, 2014

The !record command is really useful when viewing large datasets with many columns as terminal isn't horizontally scroll-able.
But having to give !record to start and again !record to view the output for each command is a bit frustrating.
So I would like to suggest adding out.flush() for addline() and print() functions in OutputFile class.. . so that we can record outputs of multiple commands in the same file and view them after each command.

Ps. I found the tool extremely useful.. keep up the good work :)

@Sachini
Copy link
Contributor Author

Sachini commented Dec 8, 2014

I fixed it and submitted a pull request. Hope you would consider integrating it in the next release.

@Sachini
Copy link
Contributor Author

Sachini commented Dec 8, 2014

Or maybe it's cleaner to add
out = new PrintWriter(new FileWriter(file), true);
and enable auto-flushing.

@julianhyde
Copy link
Owner

Fixed in 0f8cfdb.

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

No branches or pull requests

2 participants