tinkerpop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From FlorianHockmann (GitHub) <git...@apache.org>
Subject [GitHub] [tinkerpop] FlorianHockmann commented on pull request #1257: dotnet: add session connection
Date Fri, 06 Mar 2020 15:36:23 GMT
One other idea I just got, how about adding a private property like this:

private bool IsSessionEnabled => !string.IsNullOrEmpty(_sessionId);

then this if clause could be changed to:

if (IsSessionEnabled)

which is easer to understand in my opinion.
But this if course only a suggestion. Ignore it if you disagree.

[ Full content available at: https://github.com/apache/tinkerpop/pull/1257 ]
This message was relayed via gitbox.apache.org for dev@tinkerpop.apache.org

  • Unnamed multipart/mixed (inline, None, 0 bytes)
View raw message