Jump to content
  • 0

L2Character$Notifyaitask


Question

Posted

 com.l2jesios.gameserver.model.actor.L2Character$NotifyAITask run

WARNING: 
java.lang.StackOverflowError
 at com.l2jesios.gameserver.ai.L2CharacterAI.maybeMoveToPawn(L2CharacterAI.java:934)
 at com.l2jesios.gameserver.ai.L2PlayerAI.thinkCast(L2PlayerAI.java:240)
 at com.l2jesios.gameserver.ai.L2PlayerAI.onEvtThink(L2PlayerAI.java:316)
 at com.l2jesios.gameserver.ai.L2CharacterAI.onEvtArrived(L2CharacterAI.java:673)
 at com.l2jesios.gameserver.ai.AbstractAI.notifyEvent(AbstractAI.java:417)
 at com.l2jesios.gameserver.ai.AbstractAI.notifyEvent(AbstractAI.java:340)
 at com.l2jesios.gameserver.model.actor.L2Character.moveToLocation(L2Character.java:3893)
 at com.l2jesios.gameserver.model.actor.L2Character$AIAccessor.moveTo(L2Character.java:3039)
 at com.l2jesios.gameserver.ai.AbstractAI.moveTo(AbstractAI.java:582)
 at com.l2jesios.gameserver.ai.AbstractAI.moveToPawn(AbstractAI.java:538)
 at com.l2jesios.gameserver.ai.L2CharacterAI.maybeMoveToPawn(L2CharacterAI.java:948)
 
This error is flloding my gs very often and produce lag anyone know how to fix it? this is the line in L2CharacterAI
 
protected boolean maybeMoveToPawn(L2Object target, int offset)
{
if (target == null || offset < 0) // skill radius -1
return false;


offset += _actor.getTemplate().getCollisionRadius();   //this is the line 934
if (target instanceof L2Character)
offset += ((L2Character) target).getTemplate().getCollisionRadius();

 

Recommended Posts

  • 0
Posted

There shouldn't be an issue there, obviously the original authors of this pack knew about this and added the catch (StackOverflowError). The only way to resolve this, is by checking all the calls of this method (infinite recursive calls are most likely the issue to this problem).

  • 0
Posted

Nop

well i doubt if you tested it, well i am sure you didnt cause if you knew what action char makes and this error appear you would had this bug fixed

  • 0
Posted

well i doubt if you tested it, well i am sure you didnt cause if you knew what action char makes and this error appear you would had this bug fixed

yes of course

  • 0
Posted

yes of course

willing to help you but your irony is of limits, i hope nobody gives a fuck for your trash server and get always this error damn people these days....

  • 0
Posted (edited)

willing to help you but your irony is of limits, i hope nobody gives a fuck for your trash server and get always this error damn people these days....

wtf? my irony? i said nop the thing you posted doesnt work and yes i would fix it if i know where it came from. Where is my irony? just honest replies..

Edited by Xerus
  • 0
Posted

yes of course

this irony

 

and NpcWalkers are the retarded l2j npc and generally offi ones that goes around town etc, i googled it and a random guy got that issue with stackoverflow he disabled NpcWalkers and it helped..

in order to test it disable them and check tommorow if error log still exists so how you test so fast? 

  • 0
Posted

this irony

 

and NpcWalkers are the retarded l2j npc and generally offi ones that goes around town etc, i googled it and a random guy got that issue with stackoverflow he disabled NpcWalkers and it helped..

in order to test it disable them and check tommorow if error log still exists so how you test so fast? 

cause i already tested before and not working....

  • 0
Posted

cause i already tested before and not working....

well obviously cause i saw only stackoverflow and i gaved the 1st answer of google.. you got stackOverFlow on other part of the code and it has also a movement issue i guess, not sure but as i searched more is a badly rework of l2j movement system rework and this issue exist in the most l2j packs so i guess a proper movement rework will fix all those stackoverflows or general-pools errors

  • 0
Posted

well obviously cause i saw only stackoverflow and i gaved the 1st answer of google.. you got stackOverFlow on other part of the code and it has also a movement issue i guess, not sure but as i searched more is a badly rework of l2j movement system rework and this issue exist in the most l2j packs so i guess a proper movement rework will fix all those stackoverflows or general-pools errors

the flood is coming like every 30 minutes i tried everything..

  • 0
Posted

I'm not familiar with esios pack, but seems to me you are getting an error, you don't necessarily need. Most likely this is due to L2J's outdated way of using catch(Throwable t), that's almost never wanted, you want to catch specific exceptions and not generally everything - that should be the case only to the very top of a thread.

 

Go to L2Character.java at NotifyAiTask class and replace 

catch (Throwable t)
{
_log.log(Level.WARNING, "", t);
}

with

catch (Exception e)
{
_log.log(Level.WARNING, "", e);
}
  • 0
Posted

 

I'm not familiar with esios pack, but seems to me you are getting an error, you don't necessarily need. Most likely this is due to L2J's outdated way of using catch(Throwable t), that's almost never wanted, you want to catch specific exceptions and not generally everything - that should be the case only to the very top of a thread.

 

Go to L2Character.java at NotifyAiTask class and replace 

catch (Throwable t)
{
_log.log(Level.WARNING, "", t);
}

with

catch (Exception e)
{
_log.log(Level.WARNING, "", e);
}

 

At the moment its like this

public void run()
{
try
{
getAI().notifyEvent(_evt, null);
}
catch (StackOverflowError e) {
if(Config.DEBUG) // safer
_log.log(Level.WARNING, "", e);
}
catch (Throwable t)
{
_log.log(Level.WARNING, "", t);
}
}

but i think that stackoverflowerror is useless should i change to:

           public void run()
        {
            try
            {
                getAI().notifyEvent(_evt, null);
            }
            catch (Exception e)
            {
                _log.log(Level.WARNING, "NotifyAITask failed. " + e.getMessage() + " Actor " + L2Character.this, e);
            }
  • 0
Posted (edited)

 

 

At the moment its like this

public void run()
{
try
{
getAI().notifyEvent(_evt, null);
}
catch (StackOverflowError e) {
if(Config.DEBUG) // safer
_log.log(Level.WARNING, "", e);
}
catch (Throwable t)
{
_log.log(Level.WARNING, "", t);
}
}

but i think that stackoverflowerror is useless should i change to:

           public void run()
        {
            try
            {
                getAI().notifyEvent(_evt, null);
            }
            catch (Exception e)
            {
                _log.log(Level.WARNING, "NotifyAITask failed. " + e.getMessage() + " Actor " + L2Character.this, e);
            }

 

Yes, you should change to that.

Edited by Versus
Guest
This topic is now closed to further replies.



  • Posts

    • It's also the players' fault, because there have been decent servers implementing some of the things you said plus some other 'innovations', plus many QOL things for newbies (ingame bestiary, with drops searchers, etc). In the end, it's the players who decide to feed into that shit and play the most garbage servers simply because the owners of the servers gave their clan leaders 100 euros, or they insta quit the server because they didn't win the first QA, etc, etc, etc.   In the end, if a server is garbage or great it doesn't really matter if the players don't wanna stay in there.   Players are no better than the devs themselves, in the end it seems there are abusive devs who will milk the shit out of their willingly milkable players, or there are none, goes both ways.
    • In my opinion, L2 is dead because the people who make servers didn’t adapt to today’s reality. People are getting older, life moves faster, there are more responsibilities, and less free time. And I’m not even talking about newcomers—how can you expect someone new to this game to learn by Googling every drop location or quest requirement? These things should’ve been integrated into the game, made accessible with just a few clicks through the interface. Instead, so much time was wasted trying to recreate retail-like features that no one asked for. Everyone hates autofarm, but why? Because admins never found a smart way to implement it. You could have made it available only in specific zones, with reduced drops, working like Adrenaline, or auto-teleporting to farm for a limited time per day—just enough to help people with limited time stay relevant in-game. There should also be zones with better drops, where active farming actually matters. Other features feel pointless—like the Life Stone system. Spamming LS to get a skill? Instead, you could create a system where you level up the skill with low chances per level, something that feels progressive and fair. Crafting should be simpler too. Right-click a recipe, and the required materials should show up right there. As for sieges, why not create daily clan war events at peak hours—one for Europeans, one for Latinos? You could spawn crystals inside or outside castles that give points and trigger PvP. Add a boss during the event that gives even more points, and let the top clan in the ranking take the castle. I could go on forever, but what’s the point? The community died because the people who had the knowledge to improve the game just took the easy way out, copying the same server formula over and over until no one could enjoy playing it anymore.
    • It's not because I'm an admin that he treated me differently. I actually gave him several clients from my side without him even knowing they came from me, and most of them had no issues. I was also waiting 3–4 weeks at times for things I bought from AvE, even when I was in a rush. He still delivered in the end. That said, I'm not defending him blindly. I'm just saying it's unlikely he’d risk scamming someone over 60–100€, especially knowing how quickly word spreads here.
    • For exact same reason - there were accusation that I scammed. When was it? 2016? But in that time, admins actually didn't listen. I got banned, then unbaned (when I prooved I've refunded) but I was trash talking to mods. When few months later same shit happened, Grisom (?) old global mod, banned me anyway. You can read somewhere on forum how I was shitting on him for doing that (from other account because original account was banned) - which was banned too. He is not here anymore I think. Back in the days I was well know for not carring that much if I was talking to mod or admin, I didn't hold my tongue. Now You know. Just like You know - if I delay, I deliver or refund. I'm not a scammer, even if my old time haterz love to repeat themselfs like mantra. I don't care.
    • Okay I respect that but why is your other account banned?   I don't think this happened just because you delayed somebodys work even in 2012
  • Topics

×
×
  • Create New...