Ennoying syntax error

#1 You left the { out after the switch but before the first case, use this:

function GameConnection::createPlayer(%this, %spawnPoint,%Char)
{
   if (%this.player > 0)  {
      // The client should not have a player currently
      // assigned.  Assigning a new one could result in 
      // a player ghost.
      error( "Attempting to create an angus ghost!" );
   }
Switch(%Char)
{
case 1:
   // Create the player object
   %player = new Player() {
      dataBlock = TerroristBody;
      client = %this;
   };
   MissionCleanup.add(%player);

   // Player setup...
   %player.setTransform(%spawnPoint);
   %player.setShapeName(%this.name);
   
   // Update the camera to start with the player
   %this.camera.setTransform(%player.getEyeTransform());

   // Give the client control of the player
   %this.player = %player;
   %this.setControlObject(%player);


   // Set Equip
   togglehelmet();
   %player.setSkinName(iraq);

case 2:
   // Create the player object
   %player = new Player() {
   datablock = BrittishBody;
   client = %this;
   };
   MissionCleanup.add(%player);

   // Player setup...
   %player.setTransform(%spawnPoint);
   %player.setShapeName(%this.name);
   
   // Update the camera to start with the player
   %this.camera.setTransform(%player.getEyeTransform());

   // Give the client control of the player
   %this.player = %player;
   %this.setControlObject(%player);


   // Set Equip
   togglehelmet();
   %player.setSkinName(jungle1);

Case 3:

   // Create the player object
   %player = new Player() {
      dataBlock = SASBody;
      client = %this;
   };
   MissionCleanup.add(%player);

   // Player setup...
   %player.setTransform(%spawnPoint);
   %player.setShapeName(%this.name);
   
   // Update the camera to start with the player
   %this.camera.setTransform(%player.getEyeTransform());

   // Give the client control of the player
   %this.player = %player;
   %this.setControlObject(%player);


   // Set Equip
   togglegasmask();
   %player.setSkinName(gign);

case 4:

   // Create the player object
   %player = new Player() {
      dataBlock = AmericanBody;
      client = %this;
   };
   MissionCleanup.add(%player);

   // Player setup...
   %player.setTransform(%spawnPoint);
   %player.setShapeName(%this.name);
   
   // Update the camera to start with the player
   %this.camera.setTransform(%player.getEyeTransform());

   // Give the client control of the player
   %this.player = %player;
   %this.setControlObject(%player);


   // Set Equip
   togglegasmask();
   %player.setSkinName(classic);
}

#2 Kl thanks, now i look like a dunce :p

#3 It still says there is still a syntax error

>>> Advanced script error report.  Line 19.
>>> Some error context, with ## on sides of error halt:
      error( "Attempting to create an angus ghost!" );

   }

Switch(%Char)

{##
##
case 1:

   // Create the player object

   %player = new Player() {
>>> Error report complete.

#4 Try using switch$() ("switch string") and put your individual case matches in quotes, such as:

case "1":

Also, assuming that you cut/pasted the entire function, you are missing at least one ending } for the function itself.

#5 IE Replace the entire funtion with

function GameConnection::createPlayer(%this, %spawnPoint,%Char)
{
   if (%this.player > 0)  {
      // The client should not have a player currently
      // assigned.  Assigning a new one could result in 
      // a player ghost.
      error( "Attempting to create an angus ghost!" );
   }
switch(%Char)
{
case 1:
   // Create the player object
   %player = new Player() {
      dataBlock = TerroristBody;
      client = %this;
   };
   MissionCleanup.add(%player);

   // Player setup...
   %player.setTransform(%spawnPoint);
   %player.setShapeName(%this.name);
   
   // Update the camera to start with the player
   %this.camera.setTransform(%player.getEyeTransform());

   // Give the client control of the player
   %this.player = %player;
   %this.setControlObject(%player);


   // Set Equip
   togglehelmet();
   %player.setSkinName(iraq);

case 2:
   // Create the player object
   %player = new Player() {
   datablock = BrittishBody;
   client = %this;
   };
   MissionCleanup.add(%player);

   // Player setup...
   %player.setTransform(%spawnPoint);
   %player.setShapeName(%this.name);
   
   // Update the camera to start with the player
   %this.camera.setTransform(%player.getEyeTransform());

   // Give the client control of the player
   %this.player = %player;
   %this.setControlObject(%player);


   // Set Equip
   togglehelmet();
   %player.setSkinName(jungle1);

Case 3:

   // Create the player object
   %player = new Player() {
      dataBlock = SASBody;
      client = %this;
   };
   MissionCleanup.add(%player);

   // Player setup...
   %player.setTransform(%spawnPoint);
   %player.setShapeName(%this.name);
   
   // Update the camera to start with the player
   %this.camera.setTransform(%player.getEyeTransform());

   // Give the client control of the player
   %this.player = %player;
   %this.setControlObject(%player);


   // Set Equip
   togglegasmask();
   %player.setSkinName(gign);

case 4:

   // Create the player object
   %player = new Player() {
      dataBlock = AmericanBody;
      client = %this;
   };
   MissionCleanup.add(%player);

   // Player setup...
   %player.setTransform(%spawnPoint);
   %player.setShapeName(%this.name);
   
   // Update the camera to start with the player
   %this.camera.setTransform(%player.getEyeTransform());

   // Give the client control of the player
   %this.player = %player;
   %this.setControlObject(%player);


   // Set Equip
   togglegasmask();
   %player.setSkinName(classic);
}
}

#6 Hmmm well it gives a different syntax error now

>>> Advanced script error report.  Line 117.
>>> Some error context, with ## on sides of error halt:

   togglehelmet();

   %player.setSkinName(jungle1);



Case 3:##
##


   // Create the player object

   %player = new Player() {

      dataBlock = SASBody;
>>> Error report complete.

#7 Ok well nobody has said this yet but: DON'T USE CAPS (oh the irony)

You were using caps for all of those (Switch(%var) should be switch(%var)) and the engine doesn't like that. Just like you should never capitalize "Function name(blah)" never capitalize things like that ;)

Hope that helps ;)

#8 Ok thanks never knew that

#9 ROFL. Didn't notice that. Torque Script should be treated like C++ where everything is case sensitive. The only difference between the two in the case sensitive way is that Torque Script does not use case sensitive variables. (Even though I think it should.)