Power options might not work when settings set to never turn off screen about mamesaver

[mika76]

Watch out @nullpainter, that's how they get us! 😉

I'll merge, sort out the releases and the screenshots some time today. Also time to update the website...

[mika76]

No stress - I'll be sleeping from now till then 🤣

[mika76]

I will try debug this when I'm at my VS2017 machine...

[mika76]

I can confirm if I change the turn off display setting to 1 Min, then Mamesaver works. Putting back to Never crashes again.

[nullpainter]

Oops, I also fixed this by making the TimeSpan nullable. Will create PR for you to look over. Having it nullable and only conditionally create the timer is a bit less hacky than using min/max value.

[nullpainter]

Actually, it'll end up with a merge conflict if I just create a PR. Here's my version of the fix:
nullpainter@c9c42a4

There's logging and no timer creation if there's no sleep configured.

[nullpainter]

Unrelated, but I also added some fixes suggested by VS code analysis in nullpainter@ca5deae

[mika76]

Hey @nullpainter I'm happy to accept your fixes - is there some way I can add these manually? Can't you just merge master into your branch and overwrite my changes?

[nullpainter]

Yep, I can do that. Probably be this evening though (so, in around six hours).

[nullpainter]

All done. Sorry about that - I fixed it before I saw that you already had 😁

btw, do you only have two games currently or are you worried about implied copyright issues? Also - and hardly an issue (just me being a little OCD) - but the MAME icon looks a bit gross against your brown. Wondering whether the bog-standard Windows 10 colour scheme may look better for sample screenshots?

[mika76]

All done. Sorry about that - I fixed it before I saw that you already had 😁

No prob at all - you were a little quiet so I thought you were busy and I would help you out a bit 😊

btw, do you only have two games currently or are you worried about implied copyright issues? Also - and hardly an issue (just me being a little OCD) - but the MAME icon looks a bit gross against your brown. Wondering whether the bog-standard Windows 10 colour scheme may look better for sample screenshots?

Hah it's my theme on my parallels VM - looks much better with the brownish background I guess. Yeah it's not the best. I wanted to include all the settings as screenshots and some demo ones too. I'll update when I get back to my other laptop - just been a bit busy to sit with it.

I only had 2 roms because I don't really use mame on my mac.

[nullpainter]

No prob at all - you were a little quiet so I thought you were busy and I would help you out a bit 😊

Thanks; appreciated! Defects are shameful so I felt a moral responsibility to fix 😉