You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by "nabarunnag (GitHub)" <gi...@apache.org> on 2018/11/26 17:07:17 UTC

[GitHub] [geode] nabarunnag commented on issue #2827: GEODE-6028: Remove checks that are not necessary.

@mcmellawatt Bruce also had the same question with an earlier similar PR. I generally add .diff at the end of the PR hyperlink (https://github.com/apache/geode/pull/2827.diff) and copy the page and in intelliJ I do the "apply  patch from clipboard".  Doing that you will be able easily see why the checks are unnecessary. It helps to look at PRs when it is tough to analyze it from the github webpage.

[ Full content available at: https://github.com/apache/geode/pull/2827 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org