Skip to content

More performant o.plugins - #487

Open
Tryhard999 wants to merge 8 commits into
OxideMod:developfrom
Tryhard999:develop
Open

More performant o.plugins#487
Tryhard999 wants to merge 8 commits into
OxideMod:developfrom
Tryhard999:develop

Conversation

@Tryhard999

Copy link
Copy Markdown
Contributor

Reduced allocations and overheard from running the O.plugins command, useful for RCON tools and such that might run it often

@MrBlue

Copy link
Copy Markdown
Member

I'm not opposed to changing the way this command works but I feel like we can make it a little bit more readable and clean it up using StringBuilder.AppendFormat for example:

output.AppendLine();output.AppendFormat(string.Format(" {0} \"{1}\" ({2}) by {3} ({4}s) - {5}",pluginCount++.ToString("00"),plugin.Title,plugin.Version,plugin.Author,plugin.TotalHookTime.ToString("0.00"),plugin.Filename.Basename()));

@MrBlue
MrBlueforce-pushed the develop branch 5 times, most recently from 342e2fb to ed689a6CompareApril 2, 2025 05:20
@MrBlue
MrBlueforce-pushed the develop branch 2 times, most recently from d29b0c8 to bfe3314CompareAugust 30, 2025 20:15
Sign up for freeto 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.

2 participants

@Tryhard999@MrBlue