Modified shoot_full disable patch that also prevents leaving <ALT> mode while a script is running.
It might be worth submitting that as a seperate patch - do we really want to allow pressing the <ALT> key to let us leave <ALT> mode while a script is running?
Update : I suppose we could make the "Script Terminate" button a user selectable option - like the choice if <ALT> key is for most cameras. A lot of work though to do on a per camera basis. Maybe not worth it?
Thanks for working on this. I just looked at the patch file, and it looks like you're adding a menu item option. That's a good idea, and should be very useful. But it might be more useful to do it as a script function, after we get it working. The script has to be aware that it needs to exit by itself. A lot of my test scripts seem to include this:
repeat ...something... until false
Personally, I NEVER want to exit <alt> mode while a script is running. I only do it by accident, followed by some choice expletives every time, since it messes up the script.
It looks like CHDK will just ignore the <alt> key and not pass the key press to the script? That's better, but I would like to see the <alt> key be visible in the script. For example, there's no <display> key on the D20, so I modified the script to look for <display> or <video>. But the menu allows <video> to be set as the <alt> key, so the script will never be able to see it, as it stands.
Also, will is_key("shoot_full") work in a script now? It looks like it will to me. I'll check out your patch soon.
[edit]
I'm wary of having scripts that don't allow you to terminate them via the keyboard, at least without some notification to the user.
Yes, I agree. That's why I think it would work better as a Lua function, and turn off automatically when the script terminates. The script would have to tell you how to exit. My script says "Press <menu> to exit". I use the shoot_half key as a reset button when I'm setting my metering areas, but I frequently press shoot_full by mistake and have to start over. There aren't a lot of buttons that are on all cameras, so having the shoot_full button available would be very useful.
