-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Refactored code to improve readability and understand complex logic #1934
Conversation
|
||
private ConstrainableInputStream(InputStream in, int bufferSize, int maxSize) { | ||
super(in, bufferSize); | ||
Validate.isTrue(maxSize >= 0); | ||
this.maxSize = maxSize; | ||
remaining = maxSize; | ||
capped = maxSize != 0; | ||
startTime = System.nanoTime(); | ||
this.timeoutManager = timeoutManager; |
Check failure
Code scanning / CodeQL
Self assignment Error
@@ -50,39 +64,68 @@ | |||
XmlTreeBuilder newInstance() { | |||
return new XmlTreeBuilder(); | |||
} | |||
|
|||
abstract class TokenProcessor { |
Check notice
Code scanning / CodeQL
Inner class could be static Note
} | ||
} | ||
|
||
public class EofProcessor extends TokenProcessor { |
Check notice
Code scanning / CodeQL
Inner class could be static Note
Thanks, but I don't see any particular improvement here. |
No description provided.