[GitHub] [geode] lgtm-com[bot] commented on pull request #5400: GEODE-7864: Fix several LGTM warnings
lgtm-com[bot] commented on pull request #5400: URL: https://github.com/apache/geode/pull/5400#issuecomment-663912138 This pull request **fixes 18 alerts** when merging c32020eb8f49ad26aebbf88706793f70b49dbacc into 067194e8f82414b6103c4335beb6567947edcfbc - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-9d96c76acbf0f5124a5aed7cafda33d043853745) **fixed alerts:** * 9 for DOM text reinterpreted as HTML * 3 for Non\-synchronized override of synchronized method * 1 for Useless comparison test * 1 for Superfluous trailing arguments * 1 for Missing space in string concatenation * 1 for Useless null check * 1 for Double escaping or unescaping * 1 for Incomplete string escaping or encoding 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
[GitHub] [geode] lgtm-com[bot] commented on pull request #5400: GEODE-7864: Fix several LGTM warnings
lgtm-com[bot] commented on pull request #5400: URL: https://github.com/apache/geode/pull/5400#issuecomment-663893164 This pull request **introduces 1 alert** and **fixes 17** when merging 82c87573e139d5860f97cd1c85dbccc7ddb90e30 into 067194e8f82414b6103c4335beb6567947edcfbc - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-872796047847ee7a318d33587ab99c428b78c44c) **new alerts:** * 1 for Double escaping or unescaping **fixed alerts:** * 9 for DOM text reinterpreted as HTML * 3 for Non\-synchronized override of synchronized method * 1 for Useless comparison test * 1 for Superfluous trailing arguments * 1 for Missing space in string concatenation * 1 for Useless null check * 1 for Double escaping or unescaping 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
[GitHub] [geode] lgtm-com[bot] commented on pull request #5400: GEODE-7864: Fix several LGTM warnings
lgtm-com[bot] commented on pull request #5400: URL: https://github.com/apache/geode/pull/5400#issuecomment-663777444 This pull request **fixes 7 alerts** when merging 0418008cc33c2e351371088ad78b447d0e84da75 into 067194e8f82414b6103c4335beb6567947edcfbc - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-18be54c5a4d57e413d1a6c87eaab39d3f7fd7cdd) **fixed alerts:** * 3 for Non\-synchronized override of synchronized method * 1 for Useless comparison test * 1 for Superfluous trailing arguments * 1 for Missing space in string concatenation * 1 for Useless null check 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