lua get_meminfo in the trunk - General Discussion and Assistance - CHDK Forum
« previous next »
Pages: [1]   Go Down

lua get_meminfo in the trunk

  • 2 Replies
  • 2691 Views
*

Offline reyalp

  • ******
  • 14126
lua get_meminfo in the trunk
« on: 27 / April / 2013, 02:55:25 »
Advertisements
lua get_meminfo got broken somewhere along the line in the trunk

The original implementation (as documented in the comment and still present in the release branch) lets you get meminfo for *either* the canon heap or the exmem heap if exmem is enabled. If neither is specified, you get whichever one CHDK allocates from. If you ask for exmem when exmen is not enabled it returns false.

In the trunk, it just gives you whichever one CHDK allocates from, and never returns false.
Logged
Don't forget what the H stands for.
*

Offline philmoz

  • *****
  • 3450
    • Photos
Re: lua get_meminfo in the trunk
« Reply #1 on: 27 / April / 2013, 03:30:43 »
Quote from: reyalp on 27 / April / 2013, 02:55:25
lua get_meminfo got broken somewhere along the line in the trunk

The original implementation (as documented in the comment and still present in the release branch) lets you get meminfo for *either* the canon heap or the exmem heap if exmem is enabled. If neither is specified, you get whichever one CHDK allocates from. If you ask for exmem when exmen is not enabled it returns false.

In the trunk, it just gives you whichever one CHDK allocates from, and never returns false.

My mistake - misread the code when making it platform independent.
Should be fixed in revision 2732.

Phil.
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)
*

Offline reyalp

  • ******
  • 14126
Re: lua get_meminfo in the trunk
« Reply #2 on: 27 / April / 2013, 17:38:51 »
Quote from: philmoz on 27 / April / 2013, 03:30:43
My mistake - misread the code when making it platform independent.
Should be fixed in revision 2732.
Thanks :)

The original wasn't the clearest thing in the world...
Logged
Don't forget what the H stands for.
Pages: [1]   Go Up
« previous next »
 

Related Topics


SimplePortal © 2008-2014, SimplePortal