-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fix bug with network packet splitting
Fixes #1403
- Loading branch information
1 parent
0d59454
commit 2dfcb49
Showing
1 changed file
with
5 additions
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this fix the error logs displayed below?
()
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No. You have probably made modifications to your source code which are causing these crashes.
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the reply, i'm having alot of difficulties finding the cause of the crashes i hoped this would solve it.
Can u see what is causing the crash (i cant find valuable information in the crash logs)? just to push me in the right direction? The server randomly crashes with 1 player hunting cyclops's.
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not the right place to discuss that, and besides, I have no interest in helping people who turn their back on the community when the knowledge is in their hands. What you, and other community members, did in a previous flash client discussion thread on OTLand is a disgrace.
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems you're not up-to-date with the facts. I've been threatened and harassed. But your answer is clear. Besides that please take a look at what I've contributed in the past.
AND besides that, this is for ORTS update. I've already let ORTS members know that i'm going to release all map parts, monsters, quests, npcs, and alot of bug fixes, Premium shop, Unjustified counter, etc. Znote and Gesior are working to build flash client into their website.
Please reconsider your opinion about me, i've done nothing wrong to receive the "disgrace".
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will reconsider my opinion of you when what you are saying can be publicly verifiable. I am well aware of your past contributions which date back to long before TFS 1.0 was released, but haven't seen any meaningful contribution from you since then.
Here's a quote of what you said in that thread: "To be honest, i only wish to share Flash client with people who contribute. I'll be maiking a privet Github project soon.". So I don't think the word "disgrace" was undeserved, but here's more if you need help remembering: https://otland.net/threads/using-flash-client-in-ots.230791/page-2#post-2228533
The kind of behavior that you, WibbenZ and other people in that thread are practicing does not belong in an open source community. It is a disgrace. It is elitism, it does not move OpenTibia forward. In fact, it moves us backwards by discouraging other people from contributing because they know certain people refuse to share their findings with the community. If you want to discuss this further, you may start a thread about it on OTLand, because it may be generating noise for people who are watching this repository.
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, if you don't mind i will post this message towards those people and those who threatened me. And yes, i really wish to release everything. I'm just scared of the minor bugs, those will make people hunt me down for support. That's why i'm trying to resolve the issue above.
Going to create a thread soon~ Let me fix a couple of bugs (like the random crash bug above :() so i can try to make a release.
Proof of contributing (see date!) to ORTS, I've been working on the update for a couple of months now.
http://puu.sh/oLuyA/7216efdc9b.png
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fyi me and milice had the same ide, we would get spammed abt bugs and ppl wanting help.
And mark I can throw the same rock back, why are you not making ex the unjust window public or other things you did with shadowcores?
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@marksamman about this commit, it should be added to this wikipage?
https://github.com/otland/forgottenserver/wiki/Changelog-1.2
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love how @WibbenZ doesn't get a reply about all these features that weren't released here haha
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@BenDol Im hardely active in ots anymore, mainly to reasons like this.
I also "love" how Mark thinks that we should release everything we have to be welcome here, as he who still has done alot for this repo keeps his files to himself.
You can see here below, that unless we give everything out that we make are the ones who are the problem.
But then again, if he already has the files / fixes w/e why not release them?
It's such a double moral.
If is he allowed to keep his files we should also, so why not just stop this long lost discussion and agree that we release what we wanna release.
Idc if mark releases his files, its up to him as it should be.
But don't go saying everyone else should when you don't.
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@marksamman I'm the one that report #1403 and i was using tfs 1.1 with old networking code. So its something that was already "broken" in tfs 1.1 and probably 1.0.
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In that case #1403 might not be fixed. Can you check if it is?
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On tfs 1.1 the whole networking stack is different from 1.2, so i had to copy your fix into 1.1 code like this:
http://pastebin.com/tP3FdwK5
the problem is, Connection::send(OutputMessage_ptr msg) got a different code and a FIXME in tfs 1.1 that has been fixed in 1.2:
http://pastebin.com/CXfPJG0M
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should test it on 1.2, not by trying to backport the fix, because it's not compatible with TFS 1.1 network stack unless you replace the entire network code.
2dfcb49
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have tested with both 1.1 and 1.2. With 1.1 i have tested two times and there was a client debug with the same house and amount of items. 1.2 got no debug with the same amount of items and same house as tfs 1.1 test, i have tested 3 times.