Changes based on review.

Author: aardvark179

src/main/java/org/truffleruby/cext/ValueWrapper.java

@@ -13,7 +13,7 @@
 import com.oracle.truffle.api.interop.TruffleObject;
 
 /**
- * This layout represents a VALUE in C which wraps the raw Ruby object. This allows foreign access
+ * This object represents a VALUE in C which wraps the raw Ruby object. This allows foreign access
  * methods to be set up which convert these value wrappers to native pointers without affecting the
  * semantics of the wrapped objects.
  */

src/main/java/org/truffleruby/cext/WrapNode.java

@@ -38,7 +38,7 @@ public abstract class WrapNode extends RubyBaseNode {
     public abstract TruffleObject execute(Object value);
 
     @Specialization
-    public TruffleObject wrapLong(long value,
+    public ValueWrapper wrapLong(long value,
             @Cached("create()") BranchProfile smallFixnumProfile) {
         if (value >= ValueWrapperManager.MIN_FIXNUM_VALUE && value <= ValueWrapperManager.MAX_FIXNUM_VALUE) {
             smallFixnumProfile.enter();
@@ -50,40 +50,40 @@ public TruffleObject wrapLong(long value,
     }
 
     @Specialization
-    public TruffleObject wrapDouble(double value) {
+    public ValueWrapper wrapDouble(double value) {
         return getContext().getValueWrapperManager().doubleWrapper(value);
     }
 
     @Specialization
-    public TruffleObject wrapBoolean(boolean value) {
+    public ValueWrapper wrapBoolean(boolean value) {
         return new ValueWrapper(value, value ? TRUE_HANDLE : FALSE_HANDLE);
     }
 
     @Specialization
-    public TruffleObject wrapUndef(NotProvided value) {
+    public ValueWrapper wrapUndef(NotProvided value) {
         return new ValueWrapper(value, UNDEF_HANDLE);
     }
 
     @Specialization
-    public TruffleObject wrapWrappedValue(ValueWrapper value) {
+    public ValueWrapper wrapWrappedValue(ValueWrapper value) {
         throw new RaiseException(getContext(), coreExceptions().argumentError(RopeOperations.encodeAscii("Wrapping wrapped object", UTF8Encoding.INSTANCE), this));
     }
 
     @Specialization(guards = "isNil(value)")
-    public TruffleObject wrapNil(DynamicObject value) {
+    public ValueWrapper wrapNil(DynamicObject value) {
         return new ValueWrapper(nil(), NIL_HANDLE);
     }
 
     @Specialization(guards = { "isRubyBasicObject(value)", "!isNil(value)" })
-    public TruffleObject wrapValue(DynamicObject value,
+    public ValueWrapper wrapValue(DynamicObject value,
             @Cached("createReader()") ReadObjectFieldNode readWrapperNode,
             @Cached("createWriter()") WriteObjectFieldNode writeWrapperNode,
             @Cached("create()") BranchProfile noHandleProfile) {
-        TruffleObject wrapper = (TruffleObject) readWrapperNode.execute(value);
+        ValueWrapper wrapper = (ValueWrapper) readWrapperNode.execute(value);
         if (wrapper == null) {
             noHandleProfile.enter();
             synchronized (value) {
-                wrapper = (DynamicObject) readWrapperNode.execute(value);
+                wrapper = (ValueWrapper) readWrapperNode.execute(value);
                 if (wrapper == null) {
                     wrapper = new ValueWrapper(value, UNSET_HANDLE);
                     writeWrapperNode.write(value, wrapper);
@@ -106,8 +106,4 @@ public WriteObjectFieldNode createWriter() {
         return WriteObjectFieldNodeGen.create(Layouts.VALUE_WRAPPER_IDENTIFIER);
     }
 
-    public boolean isWrapped(TruffleObject value) {
-        return value instanceof ValueWrapper;
-    }
-
 }