isapir commented on issue #157: 
https://bz.apache.org/bugzilla/show_bug.cgi?id=63333
URL: https://github.com/apache/tomcat/pull/157#issuecomment-481861275
 
 
   > Ok, let me know what you think we should put as a comment additionally 
that is so greatly missed and it is not obvious enough for you or someone else 
and I will add it
   
   > because if you to have another thread trying to access the LockOutRealm 
and you put the 'invocationSuccess = true' on line 405 you could just change 
the failed invocation e.g. invocationSuccess from false to true on line 405 to 
only put the variable back to false when the next attempt fails again
   
   Well then, here's an excellent opportunity for a comment so that someone 
wouldn't edit it later without taking this into account.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to