Overriding coroutine.yield

On a code I've been working on for a while I've noticed that when I override os.pullEvent to custom handle termination events, usually just me closing a file that's being written to. It doesn't work with the turtle API. That being whenever I try terminating while a turtle api call is happening (turtle.turnLeft, turtle.craft etc) it somehow reverts to the way the old os.pullEvent worked.

I attempted to fix this by overriding coroutine.yield to handle the termination. It works however I'm using error and it seems to not be exactly liking my use of error as that just shut downs the turtle.

local oldPull = coroutine.yield
function coroutine.yield( sFilter )
  local eventData = { oldPull(sFilter) }
  if eventData[1] == "terminate" then
	logger.close()
	error("Terminated",0)
  end
  return unpack( eventData )
end

My coroutine.yield overrider.

Is there any way for me to successfully exit a program, without actually killing the computer? (In this function, I know that error works normally, and I'd rather not use shell.run("shell"))

That's really the wrong place to handle termination. You should be passing the event along, since if the program is using os.pullEventRaw instead of os.pullEvent, they're wanting to handle/ignore terminate events.

Also, turtle calls are using the Java-side yielding, which you may not be able to affect in any way.

The thing is, is if I use this, it works for the turtle yielding. If I put this into my code (which it is already inside of) and try to terminate while a turtle call is happening. It works.

However is there any way for me to exit the running program from coroutine.yield.

The only problem I'm having, before putting this in I was overriding os.pullEvent(), is that if I try to terminate the program during a turtle call, is that it never closed the log file I was writing to. It would always call the old os.pullEvent. So I could never delete it without restarting the PC.

Valithor helped out and told me to instead of error, just return "terminate" Which makes it count as the termination event that's being handled by the turtle call, however it allows me to close the file i needed.

local oldPull = coroutine.yield
function coroutine.yield( sFilter )
  local eventData = { oldPull(sFilter) }
  if eventData[1] == "terminate" then
    logger.close()
    return "terminate"
  end
  return unpack( eventData )
end

you could have also done

local oldYield = coroutine.yield
function coroutine.yield( fliter )
  local event = { oldYield( filter ) }
  if event[1] == "terminate" then
	logger.close()
  end
  return unpack( event )
end

as unpack( event ) will still return the terminate event.

Well that's a big derp lol. Thanks though :D/> Glad someone else also helped me. Didn't actually realize that the unpack still worked after the if. Yay for not noticing that. Thanks a lot theoriginalbit

Dragon53535, on 01 December 2014 - 03:55 AM said:

Valithor helped out and told me to instead of error, just return "terminate"

Lyqyd, on 30 November 2014 - 10:52 PM said:

You should be passing the event along

The name is Lyqyd, not Valithor. :P/>

Lyqyd, on 01 December 2014 - 04:57 AM said:

The name is Lyqyd, not Valithor. :P/>

The one who helped me that actually got the code running at the time was valithor who talked to me on the LuaLand server.