FlorianHockmann commented on code in PR #2610:
URL: https://github.com/apache/tinkerpop/pull/2610#discussion_r1609407967
##########
gremlin-dotnet/src/Gremlin.Net/Structure/IO/GraphSON/EdgeDeserializer.cs:
##########
@@ -31,26 +31,26 @@ internal class EdgeDeserializer : IGraphSONDeserializer
public dynamic Objectify(JsonElement graphsonObject, GraphSONReader
reader)
{
var outVId = reader.ToObject(graphsonObject.GetProperty("outV"));
- var outVLabel = graphsonObject.TryGetProperty("outVLabel", out var
outVLabelProperty)
- ? outVLabelProperty.GetString()
+ string outVLabel = graphsonObject.TryGetProperty("outVLabel", out
var outVLabelProperty)
Review Comment:
> Visual Studio 17.9.5 think GetString()! is not enough
OK, I see. I'm using Rider and that even complains if `string` is used
instead of `var`, but then we can stick with `string` of course.
--
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.
To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
