some improvement changes

agentjill commented 4 months ago

@MathNya In tradition with the previous PR's this PR is also a slight refactor and slight improvement.

Some &Option<T> changed to Option<&T> similarly for mutable case. I think these were spared during the previous purge. (As always a disclaimer for breaking change)
In src/structs/string_value.rs, get_value_string() was renamed to get_value_str() to properly indicate its return type.(NOT a breaking change)
Some idiomatic changes
In src/structs/spreadsheet.rs the function get_pivot_caches() changed so that it is shortcircuiting. Noticable speed bump is observed.

agentjill commented 4 months ago

@MathNya I will be taking a break with commits to this repo for some while. Is there a way other than gitter where I can contact you. An email or other instant messager perhaps?

MathNya commented 4 months ago

@agentjill Thank you for your continued support. I will merge them.

My email address is now available. It is listed here. https://github.com/MathNya

MathNya / umya-spreadsheet

some improvement changes #180