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.

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

Merge CGame and UI virtual machines #41

Closed
zturtleman opened this issue Jun 16, 2013 · 2 comments
Closed

Merge CGame and UI virtual machines #41

zturtleman opened this issue Jun 16, 2013 · 2 comments

Comments

@zturtleman
Copy link
Member

http://ztm.x10host.com/2013/03/18/why-is-there-a-ui-vm/

Merging cgame and ui will make things better, only negatives I can think of are (possibly?) take longer to load main menu and UI/cgame VM can't be replaced separately.

I created a patch for merging ui into cgame awhile ago, but didn't get Team Arena working. code/ui/ui_shared.c doesn't support loading hud and menus at the same time. :/

@zturtleman
Copy link
Member Author

Edit: disregard this entire message.

Hmm, I should probably decide if I'm going to support Team Arena UI or just replace with new UI see #48. If not supporting TA UI, this is fairly easy. :/

Also I wonder if I should rename cgame after merging ui into it. uxgame.qvm?... probably not. seems like it might make it easier to handling connecting cgame and ui if I had some other name for the connecting part (UX) though..

@zturtleman
Copy link
Member Author

Merged ui into cgame in a3edb6e. Both Q3A UI and Team Arena UI work.

There is still improvements to make (reduce duplicate code, get rid of BG_RegisterClientCvars, etc), but they're merged and working so closing issue.

zturtleman added a commit that referenced this issue Feb 8, 2014
…ling-mingw32

When cross compiling on mingw32, make sure it finds a CC
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant