Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Modernize the code, remove clutter and optimize functionality #13

Open
4 of 9 tasks
xx-shitai-xx opened this issue May 23, 2019 · 2 comments
Open
4 of 9 tasks

Modernize the code, remove clutter and optimize functionality #13

xx-shitai-xx opened this issue May 23, 2019 · 2 comments
Assignees
@xtjoeytx @LoneBoco @xx-shitai-xx
Labels
enhancement New feature or request gserver
Milestone
v3.1.0

Comments

@xx-shitai-xx
Copy link
Collaborator

xx-shitai-xx commented May 23, 2019 •
edited by LoneBoco
Loading

  • Fix for loops to use for (auto var : vector)
  • Use shared_pointer
  • Code style cleanup
  • Remove code duplication
  • Re-organize project
  • Break down class interconnections
  • Fix up includes
  • Switch to default member initialization
  • Introduce proper namespaces
@xx-shitai-xx xx-shitai-xx added the enhancement New feature or request label May 23, 2019
@xx-shitai-xx xx-shitai-xx added this to the v3.1.0 milestone May 23, 2019
@xx-shitai-xx xx-shitai-xx pinned this issue Jun 3, 2019
@xx-shitai-xx
Copy link
Collaborator Author

This is obviously still on-going and might never be completed :D

@xx-shitai-xx
Copy link
Collaborator Author

@LoneBoco Fill in with tasks (even those you've already compeled) and check them off the list. It's nice to keep track of the changes we've done. :)

@LoneBoco LoneBoco linked a pull request Jun 7, 2024 that will close this issue
Refactor the GServer #96
Closed
7 tasks
@LoneBoco LoneBoco removed a link to a pull request Jun 10, 2024
Refactor the GServer #96
Closed
7 tasks
@LoneBoco LoneBoco moved this to In progress in Long term fixing Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@xtjoeytx xtjoeytx

@LoneBoco LoneBoco

@xx-shitai-xx xx-shitai-xx

Labels
enhancement New feature or request gserver
Projects
Long term fixing
Status: In progress
Milestone
v3.1.0
Development

No branches or pull requests
3 participants
@xtjoeytx @LoneBoco @xx-shitai-xx