search

Vinatorul / minesweeper-rs

Simple minesweeper in Rust
MIT License
76 stars 9 forks source link

Implement redrawing when it needs for fixing #22. #38

Closed xilec closed 8 years ago

xilec commented 8 years ago

Piston use double buffering for drawing in 2d mode. RenderState is used for redrawing both buffer when it needs.

On my computer, CPU load dropped from 6-7% percents to 0.3% without any input with default settings.

In general, it works fine, but there are some case that needed addition handling: window resizing, stoping timer after changing field size or mine count and may be something else.

Vinatorul commented 8 years ago

Great work! Thank you so much.

there are some case that needed addition handling: window resizing, stoping timer after changing field size or mine count and may be something else.

Are they works now, or I should look how to implement it? I will look deeper and merge it tonight.

xilec commented 8 years ago

Thanks. I'm glad to help.

Are they works now, or I should look how to implement it?

No, they don't, but I suppose that they are not too difficult for fixing.

Vinatorul commented 8 years ago

Looks perfect to me.

@xilec do you want to become a collaborator for this repo? You've done a lot of cool staff here. :+1: