Refactor Fleet movement code for AI about open-realms-of-stars

[tuomount]

MissionHandling.fleetMakeMove(final Game game, final int nx, final int ny,
      final PlayerInfo info, final Fleet fleet)
``` is used only when fleet are moving with FTL routes.

MissionHandling.fleetMakeMove(final Game game, final PathPoint point,
final PlayerInfo info, final Fleet fleet)
``` is used when fleet is moving with standard moves where PathPoints were calculated with A* search. Both of these method then call game.fleetMakeMove() which does the actual fleet moving. Reason why this is in game is that there can be game state change during fleet move for example attack or diplomatic trade and only place where to make game state changes is Game class.

They have different logic since they check a bit different things while moving in FTL and while moving with regular moves. For example FTL movement should not start combat, only when moved with same sector with regular move. Javadocs are horrible, since they are identical. I also think that this part in pathpoint fleetMakeMove is dead code.

 else {
          Message msg = new Message(MessageType.FLEET,
              "Fleet encounter another fleet while moving in FTL!",
              Icons.getIconByName(Icons.ICON_HULL_TECH));
          msg.setCoordinate(fleet.getCoordinate());
          msg.setMatchByString(fleet.getName());
          info.getMsgList().addNewMessage(msg);
        }

[BottledByte]

Ah, I see, so it is a primarily naming and copied javadoc issue. 👍
One method is technically fleetMoveFTL(), while the other is fleetMoveRegular().

I will rename them, try to extract repeated code, fix Javadoc, remove the presumably dead code and open a PR. Or two, since I did some other smaller refactors in MissionHandling.java already.

Also, as an experiment, I tried to blindly refactor the methods, leaving only the PathPoint code version, added like 2 null checks and mixed in the A* search code... All the tests passed and I have not observed any weird behavior in-game. 😄
Maybe the AI now starts combats in FTL, but as a player, I haven't noticed that 😄