[]patch][8182] Updating map system

[soyunnico]

Thanks for this will give it a go later on

patch 1.10 error applying rev 8098 - src/game/MovementHandler.cpp line 329

I get the same error

[Ambal]

Updated patch for rev 8099+. First post updated

[Ambal]

Patch updated to rev 8103 since part of a patch already in GIT. First post updated.

[alexluana]

hey Ambal can u write down a changelog? the goals or the curent features implemented and those not yet implemented?

With respect,

Axel

[painejake]

Working great now thanks for the fix No difference in stability which is a good thing

[divisorxx]

I'm still testing and don't see any crash related to this patch. Tested with ~700 characters ON. I believe that it's ready to master branch but only a developer can decide. One more time Thanks Ambal for all your hard work into mangos performance!

[nanounico]

Testing this patch!

[painejake]

I'm still testing and don't see any crash related to this patch. Tested with ~700 characters ON. I believe that it's ready to master branch but only a developer can decide. One more time Thanks Ambal for all your hard work into mangos performance!

I agree no stability issues from that patch at all. Would be nice if more people tested it and gave their reports.

[nanounico]

well, now i test in my server without this patch, becouse this patch turn a little unstable my server!

[Porsj]

I've been using this past since my last compile 2 days ago. It's been very stable :-)

Thank you!

[Moska]

Why not include this patch in master brach of mangos?

[Ambal]

well, now i test in my server without this patch, becouse this patch turn a little unstable my server!

Guys, do I have to remind you this everytime: if you have crashes, post crashdump, jeez! I can't help if you will just remove patch and say 'it turns server unstable'...

[Porsj]

I've been running this patch for the past few days now and I must say I'm very very happy with it!

I haven't noticed any stability issues what-so-ever - on the contrary even - i believe this patch actually helps performance !

Thanks a bunch!

[xeross155]

This won't conflict with ScriptDev2 ?

[riddick]

This won't conflict with ScriptDev2 ?

Why would it?

[divisorxx]

I'm using it with sd2 and there isn't any problem. Thanks again Ambal, hope a dev when he have a little time can review it, mangos need performance.

[nanounico]

ambal, but, something is reall,i test, and my server crash every 3 max 5 hours, i remove, and i have 8-16 hours on, the problem, is when i have the crash with this path, is only "segmentation fault"

[Ambal]

i have the crash with this path, is only "segmentation fault"

Provide crashdump, please. With this patch you can get only 'assertion failed'.

[Sephiroth1983]

Looking forward to see this patch tested as deserves.

[xeross155]

Why would it?

I am not too familiar with the source yet, So I thought it might affect code in SD2

[filo]

Can you update it please? I get this error, when I run compile (Iam too use your speedup grid object search)

error: patch failed: src/game/Map.cpp:184

error: src/game/Map.cpp: patch does not apply

Thanks

[xeross155]

He'll update and if you're impatient, Apply it manually

[Homenixx]

Can you update it please? I get this error, when I run compile (Iam too use your speedup grid object search)

error: patch failed: src/game/Map.cpp:184

error: src/game/Map.cpp: patch does not apply

Thanks

Same Pleazeeeeeeeeee :D

Keep up the good work ;-)

[Ambal]

Code cleanups in WorldSession::Logout() procedure. Patch updated to latest 8158 rev. First post updated

P.S. If you have troubles with merging this patch with CellSearch one in Map.cpp, this conflict can be easely solved by adding m_VisibleDistance(DEFAULT_VISIBILITY_DISTANCE) to Map::Map() constructor.

[Ambal]

Patch 1.13 introduced possible crash, when accessing information about player who logged out... New patch uploaded. Sry