Change NoYield to use optional coroutine argument for debug.traceback.

This is part of stock Lua, but was only turned on it Roblox fairly
recently. This should stop Rodux from munching your stacks when a
function mistakenly yields.

Author: LPGhatguy

lib/NoYield.lua

@@ -14,7 +14,7 @@ local function resultHandler(co, ok, ...)
 	end
 
 	if coroutine.status(co) ~= "dead" then
-		error("Attempted to yield inside Changed event!", 2)
+		error(debug.traceback(co, "Attempted to yield inside Changed event!"), 2)
 	end
 
 	return ...

lib/NoYield.spec.lua

@@ -23,18 +23,20 @@ return function()
 		local preCount = 0
 		local postCount = 0
 
-		local function test()
+		local function testMethod()
 			preCount = preCount + 1
 			wait()
 			postCount = postCount + 1
 		end
 
-		expect(function()
-			NoYield(test)
-		end).to.throw()
+		local ok, err = pcall(NoYield, testMethod)
 
 		expect(preCount).to.equal(1)
 		expect(postCount).to.equal(0)
+
+		expect(ok).to.equal(false)
+		expect(err:find("wait")).to.be.ok()
+		expect(err:find("NoYield.spec")).to.be.ok()
 	end)
 
 	it("should propagate error messages", function()