Cached mob despawning failure

I'm trying to despawn some cached mobs:

Code:java

for(MyMob mymob : mobList.values()){
            if(!mymob.getMob().getLocation().getChunk().isLoaded())
                mymob.getMob().getLocation().getChunk().load();
            if(mymob.getMob() != null){
                mymob.getMob().setRemoveWhenFarAway(true);
                if(!mymob.getMob().isEmpty()){
                    Entity passenger = mymob.getMob().getPassenger();
                    ((LivingEntity) passenger).setRemoveWhenFarAway(true);
                    passenger.leaveVehicle();
                    passenger.remove();
                }
 
                mymob.getMob().remove();
            }
        }

• mobList is a HashMap<Integer, MyMob>
• mymob.getMob() returns a LivingEntity
• previously I: mobList.put(mymob.getMob().getEntityId(), this) where this is MyMob
• If I do a player.teleport(mymob) I get teleported to the mob spawn location (for some reason doesn't teleport to actual mob location)

When I try to despawn them sometimes (really few times) I get them removed but normally they aren't despawned, they simply still there. Something really strange is that if I do a mymob.getMob().isDead(), it return "true".
Thanks in advantage for any help.

 

NemesisMate
I think there's a memory leak when using LivingEntity#remove() (not sure though, I'll have to check), and that's sure a fancy way of looping though a hashmap. Are you sure it works? Usually this is achieved by looping though the entry set of the hashmap

 

Yes, I use it on many other hashmaps and it works fine.

If it is helpful, I'm using Bukkit 1.6.4 R-0.1
If I check if that mob is valid: mymob.getMob().isValid(), it returns false, as with isDead() it returns true.

I just saw the reason, the mob change it ID so I suppose is a different mob when I try to remove him (his name is the same and all it values are my customized mob but the server despawn/spawn it changing the entity so I can't save it on a variable, now I need a way of doing this).​

I can save the mob by UUID so it never changes, by I still having the problem about removing the mob if this is not loaded so I need a way to load a mob having his UUID stored (I can't find it).

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

NemesisMate
Try looping through the world.getEntities() and check if the entity's UUID matches that in the loop

 

The problem there is if the mob isn't loaded. Whatever, the only way I found is saving the mob location on each chunk unloading (taking that chunk entities and looking for the mob and saving if so).