2x JetNozzleX?

Since the very first version of Torque there has always been a differentiation in the name of all the Jet nozzles in the engine except the backward nozzles (FlyingVehicle.cpp, line 51 ff).

So for the forward thrust there is a "JetNozzle0" and "JetNozzle1", for downward thrusting a "JetNozzle2" and "JetNozzle3", but for the backward thruster, there were always both backward thrusters named "JetNozzleX", instead of for example "JetNozzleX" and "JetNozzleY".

Since the five years of using Torque I have never seen a reason why both backward jet nozzles had the same name - I always renamed them to X and Y and I thought I report it now as "bug" or better as feature request as I think it should be changed to be consistent. Maybe I overlooked something but I think this renaming makes totally sense.

What do you think?

Thanks,
Martin :-)


This is how it looks originally:

const char* FlyingVehicleData::sJetNode[FlyingVehicleData::MaxJetNodes] =
{
   "JetNozzle0",  // Thrust Forward
   "JetNozzle1",
[b]   "JetNozzleX",  // Thrust Backward
   "JetNozzleX",[/b]
   "JetNozzle2",  // Thrust Downward
   "JetNozzle3",
   "contrail0",   // Trail
   "contrail1",
   "contrail2",
   "contrail3",
};

And this is the change I made:

const char* FlyingVehicleData::sJetNode[FlyingVehicleData::MaxJetNodes] =
{
   "JetNozzle0",  // Thrust Forward
   "JetNozzle1",
[b]   "JetNozzleX",  // Thrust Backward
   "JetNozzleY",[/b]
   "JetNozzle2",  // Thrust Downward
   "JetNozzle3",
   "contrail0",   // Trail
   "contrail1",
   "contrail2",
   "contrail3",
};