[vdr] [ANNOUNCE] VDR developer version 1.3.49

[Klaus Schmidinger]

Udo Richter wrote:
> Dominique Simon wrote:
>> I have the following problem: If a timer is running and i press power 
>> off on the remote, VDR shows "eine Aufzeichnung läuft, trotzdem 
>> ausschalten?". VDR Shuts down no matter if i press a key or not... Can 
>> you verify this?
> 
> Yes, there's still a logical bug in it...
> 
> The 1.3.49 code shuts down if the recording shut down is confirmed *OR* 
> if the plugin shut down is confirmed. Unfortunately, if no plugin has 
> activity, this counts as confirmed, so VDR will shut down no matter if 
> recording shut down was confirmed or not.
> 
> The attached patch reverses the logic a bit, and shuts down if 
> recordings *AND* plugins confirm. UserShutdown is set up first, so that 
> any non-confirm can leave with 'break'. If all tests pass, ForceShutdown 
> is set too.
> 
> I've tested all variants I can think of, and it seems to work. More 
> testing is of course welcome, so we can mark this one fixed before Sunday.
> 
> Btw: There's a slight change in behavior since 1.3.46: Before, a simple 
> shut down without confirms did not set ForceShutdown=true, since then, 
> ForceShutdown is set true even if no confirmation was necessary. 
> cPluginManager::Active does not differentiate between confirmed, denied 
> and 'no activity'. The new way is slightly more aggressive, but seems 
> more consistent and I don't think that it has bigger negative side effects.
> 
> (1.3.49 can shut down while cutting, 1.3.46 can shut down while cutting 
> only if also a recording is running, and shutdown is confirmed.)

Thanks, looks good to me.

Since I personally don't use the shutdown feature, it's all the more
important that you folks test this extensively.

While adding your patch I realized that the code formatting of the
'case kPower:' was somewhat off, too, so I shifted it to the left.
The final patch now looks as attached.

I guess it's ok if I don't release another new version, it should be
easy enough to apply this patch and test it thoroughly.

So we still have a GO for version 1.4.0 on Sunday :-)

Klaus

-------------- next part --------------
A non-text attachment was scrubbed...
Name: vdr-1.3.49-shutdown-2.diff
Type: text/x-patch
Size: 1783 bytes
Desc: not available
Url : http://www.linuxtv.org/pipermail/vdr/attachments/20060429/d3b5a73e/vdr-1.3.49-shutdown-2.bin