Adding new cameras, applying patches into trunk (with source code prepared)

rudi

129
A590IS_101B, SX260HS_100B

Re: Adding new cameras, applying patches into trunk (with source code prepared)

« Reply #1660 on: 30 / December / 2021, 14:56:21 »

Advertisements

Here is a patch to enable fmath library for hostlua:

add "divide by zero" check for divisor in fmath_new()
re-enable os_listdir(), os_idir() for hostlua with old code from trunk 3415
add lfmath.o to hostlua Makefile

Successfully tested on windows.
rudi

hostlua_fmath-6027.diff (3.09 kB - downloaded 8 times.)

Logged

waterwingz

12128

Re: Adding new cameras, applying patches into trunk (with source code prepared)

« Reply #1661 on: 30 / December / 2021, 22:25:48 »

Patch to make HDMI output on the G16 work correctly.

see : Re: Display (bitmap overlay)

g16 hdmi fix.patch (5.42 kB - downloaded 6 times.)

Logged

Ported : A1200 SD940 G10 Powershot N G16

reyalp

13831

Re: Adding new cameras, applying patches into trunk (with source code prepared)

« Reply #1662 on: 31 / December / 2021, 00:00:53 »

Quote from: waterwingz on 30 / December / 2021, 22:25:48

Patch to make HDMI output on the G16 work correctly.

see : Re: Display (bitmap overlay)

Added in trunk r6033

Logged

Don't forget what the H stands for.

reyalp

13831

Re: Adding new cameras, applying patches into trunk (with source code prepared)

« Reply #1663 on: 31 / December / 2021, 01:05:11 »

Quote from: rudi on 30 / December / 2021, 14:56:21

Here is a patch to enable fmath library for hostlua:
add "divide by zero" check for divisor in fmath_new()
re-enable os_listdir(), os_idir() for hostlua with old code from trunk 3415
add lfmath.o to hostlua Makefile
Successfully tested on windows.
rudi

Thanks. I've added in trunk r6035 with a small modification to make os.listdir and os.idir compatible with table options added with CHDK long filename support. Of course, hostlua doesn't actually control long filenames being returned or not, but the option table is handled correctly.

Quote

add "divide by zero" check for divisor in fmath_new()

Mostly, CHDK ignores divide by 0 (see around https://chdk.setepontos.com/index.php?topic=11316.msg124312#msg124312) but I think in this case it makes sense to error, since it's new functionality and the code easily allows throwing an error.

Logged

Don't forget what the H stands for.

rudi

129
A590IS_101B, SX260HS_100B

Re: Adding new cameras, applying patches into trunk (with source code prepared)

« Reply #1664 on: 31 / December / 2021, 10:20:26 »

Quote from: reyalp on 31 / December / 2021, 01:05:11

I've added in trunk r6035 with a small modification ...

Perfect, thank you.

Logged

rudi

129
A590IS_101B, SX260HS_100B

Re: Adding new cameras, applying patches into trunk (with source code prepared)

« Reply #1665 on: 08 / August / 2022, 10:31:09 »

Here are two tested patches:

the missing rebuild of stubs_auto.S for trunk 6109
simplified SVN revision extraction including language independence
the goal are:
- independence of different messages from different svn versions
- independence of svnversion languages
- no error message in case of missing svnversion
procedure of patch:
- redirect any subversion error to null device, eg. svnversion is missing
- keep only digits and colon, this change any result stings to an empty value
- get valid revision number if is possible
- the result is revision number or empty value
- if result empty then use default revision 'DEF_SVN_REF' from buildconf.inc

[1]_s110-stubs_auto.diff (1.08 kB - downloaded 3 times.)

[2]_Makefile-svn-ref.diff (1.03 kB - downloaded 4 times.)

Logged

philmoz

3441

Re: Adding new cameras, applying patches into trunk (with source code prepared)

« Reply #1666 on: 08 / August / 2022, 20:21:17 »

Quote from: rudi on 08 / August / 2022, 10:31:09

Here are two tested patches:
the missing rebuild of stubs_auto.S for trunk 6109
simplified SVN revision extraction including language independence
the goal are:
independence of different messages from different svn versions
independence of svnversion languages
no error message in case of missing svnversion
procedure of patch:
redirect any subversion error to null device, eg. svnversion is missing
keep only digits and colon, this change any result stings to an empty value
get valid revision number if is possible
the result is revision number or empty value
if result empty then use default revision 'DEF_SVN_REF' from buildconf.inc

Thanks. Added in revisions 6120 and 6121.

Logged

CHDK ports:
sx30is (1.00c, 1.00h, 1.00l, 1.00n & 1.00p)
g12 (1.00c, 1.00e, 1.00f & 1.00g)
sx130is (1.01d & 1.01f)
ixus310hs (1.00a & 1.01a)
sx40hs (1.00d, 1.00g & 1.00i)
g1x (1.00e, 1.00f & 1.00g)
g5x (1.00c, 1.01a, 1.01b)
g7x2 (1.01a, 1.01b, 1.10b)