Skip to content

Extend the scope of the project#4

Closed
xaviergmail wants to merge 9 commits into
Mista-Tea:masterfrom
xaviergmail:master
Closed

Extend the scope of the project#4
xaviergmail wants to merge 9 commits into
Mista-Tea:masterfrom
xaviergmail:master

Conversation

@xaviergmail

@xaviergmail xaviergmail commented Apr 24, 2019

Copy link
Copy Markdown

This is a fork I've been using personally for a while and I decided I might as well give back to the community so here it is. I'm making this PR to hopefully get it some extra publicity from the existing thread on the facepunch forum than trying to market my own fork.

These additions are what I feel make this library complete. I can't really think of anything else I'd like to add other than GM:NetRequest/Net/Client/GlobalVarChanged hook to capture every change regardless or entity or key but I can't think of a good use case for that, so I decided against it. I also think adding hooks to NetRequests should be relatively easy so that might be my next move.

This PR adds the following:

  • NetVar Hooks ( Currently does not support net requests )
  • ClientVars (NetVars that are only networked to a single client for say HUD elements and whatnot)
  • GlobalVars (NetVars on game.GetWorld())
  • Persistent Vars (Save and restore player volatile (current lua state only) variables (NetVar, NetRequest, ClientVar) on reconnect)

I suggest you view the README as well as the examples

@Mista-Tea

Copy link
Copy Markdown
Owner

Hey, I'm just following up to say I'm not ignoring this PR, I'm just incredibly busy outside of GMod anymore and haven't had a chance to check this out. I think it's awesome how much you've expanded the project.

@FoksVHox

Copy link
Copy Markdown

Any news on this PR @Mista-Tea ?

@xaviergmail

Copy link
Copy Markdown
Author

Just going through and cleaning my open PRs!

In case anyone finds this in the future, I've essentially moved on away from Garry's Mod development, but the changes outlined in this PR along with more utilities can be found in xlib

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants