-
Notifications
You must be signed in to change notification settings - Fork 9
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
Patterns documentation #52
Conversation
@@ -147,9 +147,9 @@ private static void SV_Configstrings_f(List<String> args, GameImportsImpl gameIm | |||
final String nextCmd; | |||
|
|||
if (start == Defines.MAX_CONFIGSTRINGS) { | |||
nextCmd = "cmd baselines " + gameImports.spawncount + " 0\n"; | |||
nextCmd = "cmd " + StringCmdMessage.BASELINES + " " + gameImports.spawncount + " 0\n"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
easier to track usages this way
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not use the string templating?
nextCmd = "cmd " + StringCmdMessage.BASELINES + " " + gameImports.spawncount + " 0\n"; | |
nextCmd = String.format("cmd %s %s 0\n", StringCmdMessage.BASELINES, gameImports.spawncount); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing kotlin strings :(
info/Overview.md
Outdated
Although the performance of jake2 is not a problem at the moment (mostly because it uses too little resources), | ||
splitting io and non io operation will improve execution structure. | ||
Introduction of non blocking io can improve resource consumption even more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This statement needed clarification
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
improved threading section
10e1fe7
to
b149108
Compare
Kudos, SonarCloud Quality Gate passed! |
No description provided.