Allow pushing Options - Githubissues

tomaka / hlua

Rust library to interface with Lua

MIT License

507 stars 48 forks source link

Allow pushing Options #131

Closed jonas-schievink closed 7 years ago

jonas-schievink commented 7 years ago

Some(T) pushes the T
None pushes nil

Closes #124

tomaka commented 7 years ago

I think this should be more generic, no? With your code you can only use Option as the return type of a Rust closure, and I don't see why not allow it in all situations.

jonas-schievink commented 7 years ago

Sure, that makes sense. I think I'm beginning to understand how things work :)

tomaka commented 7 years ago

Please move the code to values.rs instead of functions_write, since this is no longer strictly related to functions.

tomaka commented 7 years ago

Thanks