From 5f711f8271cdbd2db48c2b8d349dc00e386b7050 Mon Sep 17 00:00:00 2001 From: jtarvydas <5713582+jtarvydas@users.noreply.github.com> Date: Mon, 12 Nov 2018 14:57:47 -0500 Subject: [PATCH] Fix Marshallable's fromCC Check that property value is defined by wrapping in Option rather than comparing to null, fixes compiler warnings on value's whose types can never be null. --- macros/src/main/scala/gremlin/scala/Marshallable.scala | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/macros/src/main/scala/gremlin/scala/Marshallable.scala b/macros/src/main/scala/gremlin/scala/Marshallable.scala index 72cf0f18..7bc81bf3 100644 --- a/macros/src/main/scala/gremlin/scala/Marshallable.scala +++ b/macros/src/main/scala/gremlin/scala/Marshallable.scala @@ -194,11 +194,10 @@ object Marshallable { this.FromCC( $idParam, $label, - _root_.scala.collection.immutable.List(..$fromCCParams) + _root_.scala.collection.immutable.List[_root_.scala.collection.immutable.List[(_root_.scala.Predef.String,_root_.scala.Any)]](..$fromCCParams) .flatten .filter { kv => - kv.isInstanceOf[Product2[_, _]] && - kv.asInstanceOf[Product2[_, _]]._2 != null + _root_.scala.Option(kv._2).isDefined } ) def toCC(element: _root_.gremlin.scala.Element): $tpe = $companion(..$toCCParams)