Marko M?kel? wrote:
> On Sat, Aug 06, 2005 at 12:43:53PM +0200, Klaus Schmidinger wrote:
>
>>The way you have done this there will _always_ be a cControl::Shutdown().
>>Could it be that you wanted was to have _no_ cControl::Shutdown() in case
>>there is a recording going on, but the user didn't confirm the
>>"Recording - shut down anyway?" prompt? In that case you should have added
>>braces around the 'if' block, as in
>>
>> if (Interface->Confirm(tr("Recording - shut down anyway?"))) {
>> ForceShutdown = true;
>> cControl::Shutdown();
>> }
>>
>>Please try it that way and let me know.
>
>
> My bad, I was confused by the indentation of the closing brace. I am
> used to indenting the closing brace by one fewer level.
>
> The attached patch avoids the cControl::Shutdown() if the prompt is not
> confirmed.
>
> It would be even better to add a "confirm shutdown" prompt in the case
> that no recording is active, but a recorded stream is being played back
> when hitting the Power key.
>
> It can be confusing that the Power key stops recorded playback but not
> live playback. I'd rather display the "press any key to cancel
> shutdown" prompt and keep playing back the recorded stream. Alas, I
> do not know how to achieve that; maybe it would require some more
> inter-thread communication.
Now you're confusing me.
Isn't that the current behavior?
In version 1.3.25 there was a change
No longer stopping Transfer Mode or replay immediately when the Power button
is pressed (thanks to Rolf Ahrenberg).
Your suggested change would countermand that, wouldn't it?
Since I don't use the power off feature myself, and this is
apparently a rather complex thing, I'd like to hear some other
users' opinion about this before I change anything.
Klaus
