ESP8266 Community Forum

- Wed May 27, 2015 3:24 pm #18656

The leak is showing even if i remove the "tmr = nil" I just added that to see if i got a different result sorry for getting that in the example. But its good to know that i should not set it up like that.

??. The second example with the leak doesn't include the local tmr =tmr statement.

Please see my Developer FAQ for Q&A on nodeMCU Lua before asking a question here that's already been answered. This and the other official documentation should be your first reference for developing Lua Apps

- Wed May 27, 2015 6:19 pm #18677

Moin,

IMHO lua is mostly a glue or prototype language.
If you identify proper functions that need speed, encode them in c and call them from lua.

My € 0,02 cents,
Cal

Don't want to hijack but can you point me to an example call to C routine?

- Wed May 27, 2015 11:56 pm #18691

Moin,

IMHO lua is mostly a glue or prototype language.
If you identify proper functions that need speed, encode them in c and call them from lua.

My € 0,02 cents,
Cal

Don't want to hijack but can you point me to an example call to C routine?

All lua modules can be taken as an example. "app/modules". Nearly all of luas functionality is written in c.
If you want to get started understanding the interface the following may help.

viewtopic.php?f=24&t=2526
I had fun writing that.
Leave a note if you found it useful.

Cal

Get your modules here:

https://www.banggood.com/custlink/KvGGhGF4wG

- Fri Jun 12, 2015 7:12 pm #20304 I haven't found the time for more investigation until now but i have found a solution:

Original code from the first post)

Code: Select allfunction forwardandrewind()
      if(forward == 1) then
         forward = 0
         gpio.write(3,gpio.HIGH)
      else
         forward = 1
         gpio.write(3,gpio.LOW)
      end
         
      for x=1,16000 do
         gpio.write(4,gpio.HIGH)
         tmr.delay(1)
         gpio.write(4,gpio.LOW)
         tmr.delay(1)
         tmr.wdclr()
      end
      tmr = nil

      print(node.heap())
   end

I found the problem with this code its the tmr.wdclr() when called to often it leaks hell of memory instead it should only be called when needed.

example)

Code: Select allfunction forwardandrewind()
      if(forward == 1) then
         forward = 0
         gpio.write(3,gpio.HIGH)
      else
         forward = 1
         gpio.write(3,gpio.LOW)
      end
         
      for x=1,16000 do
         gpio.write(4,gpio.HIGH)
         tmr.delay(1)
         gpio.write(4,gpio.LOW)
         tmr.delay(1)
         if(x%1000) then
              tmr.wdclr()
         end
      end
      tmr = nil

      print(node.heap())
   end

Here i substitute the
tmr.wdclr()
with
if(x%1000) then
tmr.wdclr()
end

and now it do not leak memory anymore....

So if someone else doing this foolish mistake then you got the solution for it