None timeout for Terminal::get_event

cpjreynolds / rustty

A terminal UI library

https://docs.rs/rustty

MIT License

152 stars 14 forks source link

None timeout for Terminal::get_event #42

Closed gridbugs closed 8 years ago

gridbugs commented 8 years ago

This is a breaking change. The timeout argument to get_event is now an Option. Passing it 'None' causes get_event to never timeout. Passing 'Some(duration)' times out after 'duration'.

The examples are updated to use the new interface.

Note that the documentation already specifies this behaviour so I didn't update it.

This is my first rust PR! Feedback is welcome.

cpjreynolds commented 8 years ago

Hiya and thank you very much for the PR! This all looks good to me, and thank you very much for updating the examples as well. I'll get this merged now. Oh and good catch on the ':wq' in the docs!