Las Venturas Playground
Pawn & json Complie - Printable Version

+- Las Venturas Playground (https://forum.sa-mp.nl)
+-- Forum: Main Talk (https://forum.sa-mp.nl/forum-3.html)
+--- Forum: Development (https://forum.sa-mp.nl/forum-16.html)
+--- Thread: Pawn & json Complie (/thread-33259.html)

Pages: 1 2 3 4


Pawn & json Complie - Yassine - 07-30-2016

Hello !!

I Build Event.pwn and Updated lvp.pwn + lvp.ppr 

But i Have Some errors 

Quote:Dialog.pwn(43) : error 017: undefined symbol "__annotation_switch_OnDialogResponse"
Dialog.pwn(46) : error 017: undefined symbol "deprecated_OnDialogResponse"
Dialog.pwn(42) : warning 203: symbol is never used: "listItem"
Dialog.pwn(42) : warning 203: symbol is never used: "response"
Dialog.pwn(42) : warning 203: symbol is never used: "dialogId"
Dialog.pwn(42) : warning 203: symbol is never used: "playerId"
ZoneManager.pwn(82) : error 017: undefined symbol "__annotation_switch_OnPlayerEnterZone"
ZoneManager.pwn(100) : error 017: undefined symbol "__annotation_switch_OnPlayerLeaveZone"
Player.pwn(113) : error 017: undefined symbol "__annotation_list_OnIndentifiedHuman"
PlayerEvents.pwn(26) : error 017: undefined symbol "__annotation_list_OnPlayerConnect"
PlayerEvents.pwn(28) : error 017: undefined symbol "OnPlayerLVPConnect"
PlayerEvents.pwn(39) : error 017: undefined symbol "__annotation_list_OnPlayerDisconnect"
PlayerEvents.pwn(47) : error 017: undefined symbol "OnPlayerLVPDisconnect"
PlayerEvents.pwn(38) : warning 203: symbol is never used: "reason"
PlayerEvents.pwn(81) : error 017: undefined symbol "__annotation_list_OnPlayerClickMap"
PlayerEvents.pwn(76) : warning 203: symbol is never used: "positionZ"
PlayerEvents.pwn(76) : warning 203: symbol is never used: "positionY"
PlayerEvents.pwn(76) : warning 203: symbol is never used: "positionX"
PlayerEvents.pwn(100) : error 017: undefined symbol "__annotation_list_OnPlayerClickTextDraw"
PlayerManager.pwn(101) : error 017: undefined symbol "IsNumeric"
VehicleModel.pwn(306) : error 017: undefined symbol "IsNumeric"
VehicleStorageManager.pwn(138) : error 017: undefined symbol "CExport__NewWantedVehicle"
VehicleCommands.pwn(66) : error 017: undefined symbol "__annotation_switch_GenericVehicleCommand"
VehicleCommands.pwn(82) : error 017: undefined symbol "__annotation_switch_SpecificVehicleCommand"
VehicleCommands.pwn(158) : error 017: undefined symbol "Admin"
VehicleCommands.pwn(255) : error 017: undefined symbol "Admin"
VehicleCommands.pwn(301) : error 017: undefined symbol "Admin"
VehicleCommands.pwn(528) : error 017: undefined symbol "Admin"
VehicleCommands.pwn(595) : error 017: undefined symbol "Admin"
VehicleCommands.pwn(661) : error 017: undefined symbol "Admin"
VehicleEvents.pwn(132) : error 017: undefined symbol "sprayTagOnVehicleSpawn"
VehicleEvents.pwn(194) : error 017: undefined symbol "__annotation_list_OnVehicleStreamIn"
VehicleEvents.pwn(224) : error 017: undefined symbol "radioPlayerExitVehicle"
VehicleEvents.pwn(223) : warning 203: symbol is never used: "playerId"
VehicleEvents.pwn(262) : error 017: undefined symbol "Admin"

Thanks 

Yassine


RE: Pawn Complie - Shark - 08-01-2016

that's what happens when you have variables that you don't use. if you're not using those things just erase them from the code, then you won't have any problems.

thanks,

el pedro


RE: Pawn Complie - Xanland - 08-01-2016

The problem here is that there is somewhere missing a }, ) or perhaps ;. To my knowledge all those things are definitely used unless Russell broke it!


RE: Pawn Complie - Yassine - 08-01-2016

(08-01-2016, 11:04 AM)Xanland Wrote: The problem here is that there is somewhere missing a }, ) or perhaps ;. To my knowledge all those things are definitely used unless Russell broke it!

i uploaded the pwn file for u can see it 

https://github.com/LVPYassine/Las-Venturas-Playground-Event


RE: Pawn Complie - Russell - 08-01-2016

Yes, you have loads of missing closing brackets.

This is why it's important to indent your code correctly. Each block (basically the code within the curly brackets) should be indented four spaces from its parent block.

I've ran your code through a C prettifier which illustrates the problem:

https://gist.github.com/RussellLVP/aa5a7c15f2202891b1056211a42dc25f

To give you a few higher level comments however:

- You may want to give credit to vittorio, who made this map.
- This seems to be a race track rather than its own minigame. Why not introduce this as a new race? That avoids needing your code altogether, because we have all the necessary infrastructure in place.


RE: Pawn Complie - Yassine - 08-01-2016

(08-01-2016, 03:01 PM)Russell Wrote: Yes, you have loads of missing closing brackets.

This is why it's important to indent your code correctly. Each block (basically the code within the curly brackets) should be indented four spaces from its parent block.

I've ran your code through a C prettifier which illustrates the problem:

https://gist.github.com/RussellLVP/aa5a7c15f2202891b1056211a42dc25f

To give you a few higher level comments however:

- You may want to give credit to vittorio, who made this map.
- This seems to be a race track rather than its own minigame. Why not introduce this as a new race? That avoids needing your code altogether, because we have all the necessary infrastructure in place.


Another Place Better for Event :D

Players Already See All maps

Now I Have those errors 


Quote:Event.pwn(446) : error 017: undefined symbol "playerId"
Event.pwn(449) : warning 202: number of arguments does not match definition
Event.pwn(449) : warning 202: number of arguments does not match definition
Event.pwn(450) : error 017: undefined symbol "Color"
Event.pwn(450) : error 029: invalid expression, assumed zero
Event.pwn(450) : error 017: undefined symbol "Warning"
Event.pwn(450) : fatal error 107: too many error messages on one line

with u file ofc


RE: Pawn Complie - Russell - 08-01-2016

Your code has a lot of issues. Some particular to our infrastructure (there should be no spaces around "->" operators, apologies that it matters), others normal to Pawn (you never define variables such as specid).

Taking a step back, could you detail why you believe it's better to introduce this as a new feature rather than using the existing infrastructure?

In the past, we had a series of maps that were visible for all players all the time, similar to xSF. This was popular with some players, but very unpopular with others, so we decided to take them out in favor of more scoped maps. In this case, the location of this map is far outside of the normal San Andreas map, making it very unlikely that players would randomly find it.

Furthermore, the cost of introducing a second race-like system in our code is significant, it makes it very hard to reason about which command to use when— especially when the existing infrastructure has been optimised to allow for this. We work very hard to make sure that our features are integrated well with each other.

Because of this, it really should be a regular race, and I'd be willing to accept it as one.


RE: Pawn Complie - Yassine - 08-01-2016

[DELETED]

Now I Have Some Warnings :D



Quote:Event.pwn(292) : warning 202: number of arguments does not match definition

Event.pwn(292) : warning 202: number of arguments does not match definition

Hope u fix it :D
Yassine

New Code Here : http://pastebin.com/uviNNHRA


RE: Pawn Complie - Russell - 08-01-2016

Yeah, that's like, completely wrong.

Sorry, I won't dedicate more time explaining what's going on if you don't reply to my questions.


RE: Pawn Complie - Yassine - 08-01-2016

(08-01-2016, 05:39 PM)Russell Wrote: Yeah, that's like, completely wrong.

Sorry, I won't dedicate more time explaining what's going on if you don't reply to my questions.

questions ?