Skip to content
This repository has been archived by the owner on Sep 16, 2024. It is now read-only.

ProcessString methods are long and unweildy #20

Open
clowenhg opened this issue Feb 8, 2014 · 2 comments
Open

ProcessString methods are long and unweildy #20

clowenhg opened this issue Feb 8, 2014 · 2 comments

Comments

@clowenhg
Copy link

clowenhg commented Feb 8, 2014

The ProcessString methods on the server and client are just way too inclusive. I want to move the individual processing into sub methods.

@clowenhg clowenhg self-assigned this Feb 8, 2014
@clowenhg
Copy link
Author

clowenhg commented Feb 8, 2014

Starting a new branch to work this on.
Hope is that this will make it easier to work inside the code, and easier to fix bugs / write improvements.

clowenhg added a commit that referenced this issue Feb 9, 2014
clowenhg added a commit that referenced this issue Feb 9, 2014
@clowenhg
Copy link
Author

I've moved more processes to modProcessEvents on the client side.
I also created a debug log feature on the server that let's us see packets as they come. Hoping this lets us debug any potential issues while migrating the process code

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant