What consecuences can this give to me?

Hi, when we know, the project have a feature which do the mobs moves arround the players when you are farming trains.

To disable this i deleted a part of code from core:

on L2AtackableIA.java

from line 1437 to 1474.

Code:

if (!_actor.isMovementDisabled() && Rnd.nextInt(100) <= 33)
      {
         for (L2Object nearby : _actor.getKnownList().getKnownObjects().values())
         {
            if (nearby instanceof L2Attackable && _actor.isInsideRadius(nearby, collision, false, false) && nearby != _mostHatedAnalysis.character)
            {
               int newX = combinedCollision + Rnd.get(40);
               if (Rnd.nextBoolean())
               {
                  newX = _mostHatedAnalysis.character.getX() + newX;
               }
               else
               {
                  newX = _mostHatedAnalysis.character.getX() - newX;
               }

               int newY = combinedCollision + Rnd.get(40);
               if (Rnd.nextBoolean())
               {
                  newY = _mostHatedAnalysis.character.getY() + newY;
               }
               else
               {
                  newY = _mostHatedAnalysis.character.getY() - newY;
               }
               
               if (!_actor.isInsideRadius(newX, newY, collision, false))
               {
                  int newZ = _actor.getZ() + 30;
                  if (GeoData.getInstance().canMove(_actor.getX(), _actor.getY(), _actor.getZ(), newX, newY, newZ))
                  {
                     moveTo(newX, newY, newZ);
                  }
               }
               return;
            }
         }
      }

The thing is... this can generate any secondary efect to me? i cant understand fine that part of code, but i know that is who do mobs moves arround the player.

It seems that this is necessary to prevent mobs get stucked in each other chasing the player.
If this is not important for you, then nothing terrible will happen.
The main thing is that you like the result