Cosmetic changes...
try/catch block for loading ship target values xml
make sure we set cache.instance.usedrones to the value specified by our settings file EVERYTIME we start a mission.
Fix PrimaryWeaponPriorityTarget and DronePriorityTarget routines

this needs testing, but should be MUCH better.
remove unnecessary return
make sure when we are choosing the best target that we order the priority targets by their priority number (then by health, then by distance)

this will make sure we are killing scramblers before ECM targets (for instance)
THIS NEEDS TESTING - Iterate on how PrimaryWeaponPriorityTargets and DronePriorityTargets are added and processed - THIS NEEDS TESTING
update directeve to ver: v1.330.4844.24774 Support Instance Needed
bug fix for the panic find any EWAR priority target changes...
more PrimaryWeaponPriorityTarget and DronePriorityTarget changes, this should improve the situation for drone boats quite a bit... if you have a drone pilot and can get me a log of a mission that is not yet working optimally I would appreciate it.
Fix formatting of speed and distance calcs for new logging in Panic that adds Ewar NPCs to the primaryweapon and dron prioritytarget lists
When choosing a new primaryweaponprioritytarget to shoot (that is not the current target) only use the priority number, do not attempt to get the health of each prioritytarget as we may not have it targeted yet and thus not have the shields/armor/hull values to use.
Out of ammo issue, at least don't crash...

this will at least cleanup the log. if the issue reoccurs open a separate issue for it please.
put directeve in the right place ffs. https://github.com/ISeeDEDPpl/Questor/issues/358
We cant use Shields, armor, hull values unless we KNOW it is locked...  if we want to reintegrate this we will need to do this differently.
Untested fix: DirectEve was being instanced once in the login portion and once in questor. Questor now references the DE instance created by the login portion if it exists. - THIS NEEDS TO BE TESTED.
2nd run at this...  uses cache instead of passing the ref from program.cs (which disappears when program.cs finishes?!)
Merge branch 'SingleInstanceDELoginIssue' into BleedingEdge
_directEve can be private as we only use it locally and set it equal to Cache.Instance.DirectEve
fix: https://github.com/ISeeDEDPpl/Questor/issues/363 - ultimately this needs to be fixed differently. feel fre eto submit a patch that corrects this to "know" how many expired bookmarks are left and that the number has decremented from the previous pass.