changelog of trunk including comments / devtalk

[reyalp]

In changeset 831 I've updated most* of the cameras that use playrec_mode to treat 4 as record mode as well.

Rational: We already have a variable to check for menu mode, and the camera still is in record mode even if the menu is displayed.

See: http://chdk.setepontos.com/index.php/topic,2468.msg42154.html#msg42154

* exceptions: a560 (treats 1 and 2 as record) and s2is (treats 1 as record and everything else as play)

[reyalp]

In changeset 846  I've made ubasic_test.exe work again.  To use it, do make check in the lib\ubasic, then run ubasic_test <your bas file>

[reyalp]

changeset 847 should pretty much complete play/rec and capture mode override support.

NOTE: the meaning of set_capture_mode in lua has changed. It now takes a CHDK mode value, not a propcase value!. If you have been using this in any recent builds, replace it with set_capture_mode_canon.

There are tests scripts included in the CHDK/SCRIPTS/TEST folder. I encourage people to use http://chdk.wikia.com/wiki/LUA/Scripts:Standard/Test/Setmode in batch mode, and report their results (with log files) in http://chdk.setepontos.com/index.php/topic,3228.90.html

This commit involved fairly substantial re-organization of the modemap related code. Not only was this needed to implement the script features sanely, it will also make it easier to clean up mode handling more in the future. I don't think I've broken anything, but it's always possible.

Commit Message:

    Mode override support. See http://chdk.kernreaktor.org/mantis/view.php?id=64
    All cameras except ixus40_sd400 should be supported
    * add capture mode API. NOTE: behavior of set_capture_mode in lua HAS CHANGED!

        see http://chdk.wikia.com/wiki/CHDK_firmware_usage/MoreBest#set_record.28state.29 and following

    * move modemap to platform/<cam>/shooting.c so mode handling can be generic
    * move mode_get to platform/generic/shooting.c
    * add helper functions so mode_get can be generic

        rec_mode_active (checks play_rec_mode, weak, can be overriden on cameras that need it)
        screen_opened, screen_rotated: only defined on swivel screen cams, check phsw_ bits for screen pos

    * move old play/rec checking code to rec_switch_state. #ifdef'd out for now
    * move mode enum into it's own header file: modelist.h to allow easy lua generation.
    * use playrec_mode on tx1
    + add new generated lua file LUALIB/GEN/modelist.lua
    + add lua library LUALIB/capmode.lua see http://chdk.wikia.com/wiki/LUA/Scripts:Standard/Lualib/Capmode

    + add test scripts see
    http://chdk.wikia.com/wiki/UBASIC/Scripts:Standard/Test/Setmode
    http://chdk.wikia.com/wiki/LUA/Scripts:Standard/Test/Setmode

[reyalp]

changeset 864 
* Preliminary support for calling event procedures and arbitrary functions. See http://chdk.wikia.com/wiki/LUA/LUA_Reference/Native_Function_Calls
* add support for byte and word access with peek and poke
* add lib/armutil/libarmutil.a for various arm code that doesn't belong elsewhere.
* move reverse_bytes_order lib/armutil

This should be quite useful for development in conjunction with the PTP interface. No more rebuild every time you want to try a different function, or writing special code to inspect the results

[fe50]

 

[PhyrePhoX]

wow. looks like a great xmas present

[reyalp]

In changeset 865 I've fixed lua handling of hex constants, so you can now use ROM addresses directly in peek, poke (not that that would work!), and call_func_ptr.

This is untested on dryos. I'm pretty sure it will work, but if someone can confirm with the following script, it would be nice:

Code: Lua

1. if 0xFFFFFFFF == -1 then
2.         print("ok")
3. else
4.         print("lol",0xFFFFFFFF,"!=",-1)
5. end
6. sleep(1000)
7.  

PS: why does the script console sometimes disappear quickly ?

[msl]

It's also ok for dryos (a720).

Thx for your work.

Happy xmas 

[fe50]

ok on
  SX10_101a,
  SD870_100c and
  SD400_101b
...as expected... Frohes Fest ! 

[reyalp]

Thanks. No need to test on more cameras, the code appears to be 100% the same for all.

[reyalp]

I've added a script to get the crash log. Developers may find this useful.
docs http://chdk.wikia.com/wiki/LUA/Scripts:Standard/Test/Romlog
script http://tools.assembla.com/chdk/browser/trunk/CHDK/SCRIPTS/TEST/romlog.lua

Some more work is needed to turn the vxworks version into a fully human readable form, should be pretty straightforward.

[reyalp]

Regarding changeset 919 Is the problem actually directory name length, or the fact that examples was lower case in the tree (ISTR there are some circumstances where a lower case name gets treated as a long name)

4 characters shouldn't be the limit, given that SCRIPTS is 7.

Not a big deal, just wondering.

[fe50]

Regarding changeset 919 Is the problem actually directory name length, or the fact that examples was lower case in the tree (ISTR there are some circumstances where a lower case name gets treated as a long name)

4 characters shouldn't be the limit, given that SCRIPTS is 7.

Don't know since i don't own one of the affected cameras; i guess the limit is the length of the folder path...

[ultimA]

Regarding changeset 919 Is the problem actually directory name length, or the fact that examples was lower case in the tree (ISTR there are some circumstances where a lower case name gets treated as a long name)

4 characters shouldn't be the limit, given that SCRIPTS is 7.

Not a big deal, just wondering.

I tried changing the directory to uppercase, but it did not help. So it is the length.

[reyalp]

In changeset 935 I've tried to clean up some of the mess of camera specific #ifdefs for zebra. d10 and (probably) sd990 zebra is also fixed.

Please check the affected cameras
sx200is g11 ixus100_sd780 ixus95_sd1200 CAMERA_s90 sx20
to verify that I haven't broken anything.

Further cleanup or corrections are invited In particular, it seems like ixus100_sd780 and ixus95_sd1200 don't actually need the aspect correction portion. Also, the 1 pixel border restore should be changed to no border restore at all, and probably be controlled by a different ifdef. We might want to have a general "camera doesn't have much memory" option.