Timer.Simple "p" nil Value :( Pls Help me to fix it

[aoa3005]

It Timer.Simple make error

timer.Simple(0.5,function(p)
			p:SetPos(MenuPos.pos-(p:EyePos()-p:GetPos()));
		end,p);

Full Code

function JB:PlayerSpawn(p)
	p:ConCommand("fov_desired 90")
	p.Crouch = false;
	if (p:Team() < 3) then
		p:GodDisable();
		p:SetColor(Color(255,255,255,255));
		p:UnSpectate()
		p:SetMoveType(MOVETYPE_WALK);

		p:SetHealth(100);
		p:SetArmor(0);

		p:SetRunSpeed(250);
		p:SetWalkSpeed(180);

		p:SetNoDraw(false)

		p:SetModel(JB:TranslateModel((p.character or JB.Characters.Prisoner[1]).model));
		(p.character or JB.Characters.Prisoner[1]).OnSpawn(p);

		p:Give("jb_knife");
		p:Give("jb_hands");

		p:StripAmmo();

		p:GiveAmmo(500,"SMG1");
		p:GiveAmmo(500,"Pistol");
	elseif (p:Team() == TEAM_GUARD_DEAD or p:Team() == TEAM_PRISONER_DEAD or p:Team() == TEAM_SPECTATOR) and p.character then
		p:Spectate( OBS_MODE_CHASE );
		p:ConCommand("jb_spectate_switch_plus")
		p:SetMoveType( MOVETYPE_OBSERVER );
		p:SetNoDraw(true)
	elseif not p.character then 
		p:SetTeam(TEAM_SPECTATOR);
		p:SetMoveType(MOVETYPE_OBSERVER);
		p:SetNotSolid(true);
		p:SetPos(MenuPos.pos-(p:EyePos()-p:GetPos()));
		p:SetAngles(Angle(0.403056, 90.340294, 0));
		timer.Simple(0.5,function(p)
			p:SetPos(MenuPos.pos-(p:EyePos()-p:GetPos()));
		end,p);
		p:Freeze(true);
		p:SetNoDraw(true);
		p.character = nil;
	end	
end

pls tell me how to fix it :(

[jaooe]

What's the error/ which line gives you an error/ can you paste the console error?

[editline]30th March 2013[/editline]

Also, I don't think you need to pass p into your timer if you're creating it in the function that p is passed to.

[aoa3005]

[img]http://img46.imageshack.us/img46/457/afqwfqwfv.png[/img]

it error line 71

timer.Simple(0.5,function(p)
			p:SetPos(MenuPos.pos-(p:EyePos()-p:GetPos()));
		end,p);

Sorry for my bad english :<

[Rubat]

[QUOTE=aoa3005;40094417][img]http://img46.imageshack.us/img46/457/afqwfqwfv.png[/img]

it error line 71

timer.Simple(0.5,function(p)
			p:SetPos(MenuPos.pos-(p:EyePos()-p:GetPos()));
		end,p);

Sorry for my bad english :<[/QUOTE]

timer.Simple(0.5,function()
			p:SetPos(MenuPos.pos-(p:EyePos()-p:GetPos()));
		end);

[Ludicium]

O.o

[Willox]

[QUOTE=Ludicium;40094515]...[/QUOTE]

Robotboy is right

Check this out: [URL]http://lua-users.org/wiki/ScopeTutorial[/URL]

[Ludicium]

How can you reference p when it wasn't passed to the function though.

[MakeR]

[QUOTE=Ludicium;40094577]How can you reference p when it wasn't passed to the function though.[/QUOTE]

p is accessible from the timer function as an upvalue.

[Ludicium]

Yeah but we still don't know how he declared his code, but I guess I am wrong, thanks for the lesson.

[Willox]

[QUOTE=Ludicium;40094619]Yeah but we still don't know how he declared his code, but I guess I am wrong, thanks for the lesson.[/QUOTE]

Look at the first post

[Ludicium]

[QUOTE=Willox;40094620]Look at the first post[/QUOTE]

The timer declaration is taken out of context.

[Willox]

It's within the block of full code, just use ctrl+f and you'll see it.

[Ludicium]

Oh well, I was thinking about this all the wrong way, didn't see it. I really feel dumb now. Feel free to give me the dumbs.

[aoa3005]

[QUOTE=Robotboy655;40094484]

timer.Simple(0.5,function()
			p:SetPos(MenuPos.pos-(p:EyePos()-p:GetPos()));
		end);

[/QUOTE]

it work it work
thank you very much :)

[Ruzza]

You can't pass arguments with timer

s since gmod13 update.