ZaneDubya / UltimaXNA

Ultima Online client in C#/XNA
GNU General Public License v3.0
142 stars 73 forks source link

Milestone 0.5 Refactor #101

Closed ZaneDubya closed 9 years ago

ZaneDubya commented 9 years ago

As a maintainer of the codebase, I get jeebies every time I browse UltimaClient.cs. I'd like to better organize that file, partially by moving most of the world-modifying packets into a UltimaWorld.WorldClient.cs class.

I'd also like to fold BaseEngine and UltimaEngine into a single class.

Eliminate the extraneous Handlers in Core.Network.Client.cs (only need Typed Handlers).

Get rid of Point3D (not used throughout program).

Reorganize all the mouse picking code into a single class.

Rename UltimaGUI.Control to UltimaGUI.AControl (and make it abstract).

Move GUIManager to UltimaGUI namespace.

ZaneDubya commented 9 years ago

You can see my current progress on these goals in the branch Milestone0.5-Refactor.

robertdeclaux commented 9 years ago

It seems good on sphereserver without errors.

ZaneDubya commented 9 years ago

@robertdeclaux Thank you for confirming!