simplify - smarter use of constructor means we can drop a setter

h2/src/main/org/h2/mvstore/db/ValueDataType.java

@@ -612,13 +612,16 @@ public class ValueDataType implements DataType {
                     precision = readVarLong(buff);
                     compression = buff.get() == 1;
                 }
-                ValueLob lob = ValueLob.open(type, handler, tableId, objectId, precision, compression);
                 if (smallLen == -2) {
-                    lob.setFileName(readString(buff), false);
-                }
+                    String filename = readString(buff);
+                    ValueLob lob = ValueLob.openUnlinked(type, handler, tableId, objectId, precision, compression, filename);
+                    return lob;
+                } else {
+                    ValueLob lob = ValueLob.openLinked(type, handler, tableId, objectId, precision, compression);
                     return lob;
                 }
             }
+        }
         case Value.ARRAY: {
             int len = readVarInt(buff);
             Value[] list = new Value[len];

h2/src/main/org/h2/store/Data.java

@@ -806,13 +806,16 @@ public class Data {
                     precision = readVarLong();
                     compression = readByte() == 1;
                 }
-                ValueLob lob = ValueLob.open(type, handler, tableId, objectId, precision, compression);
                 if (smallLen == -2) {
-                    lob.setFileName(readString(), false);
-                }
+                    String filename = readString();
+                    ValueLob lob = ValueLob.openUnlinked(type, handler, tableId, objectId, precision, compression, filename);
+                    return lob;
+                } else {
+                    ValueLob lob = ValueLob.openLinked(type, handler, tableId, objectId, precision, compression);
                     return lob;
                 }
             }
+        }
         case Value.ARRAY: {
             int len = readVarInt();
             Value[] list = new Value[len];

h2/src/main/org/h2/value/ValueLob.java

@@ -127,10 +127,26 @@ public class ValueLob extends Value {
      * @param compression if compression is used
      * @return the value object
      */
-    public static ValueLob open(int type, DataHandler handler,
+    public static ValueLob openLinked(int type, DataHandler handler,
             int tableId, int objectId, long precision, boolean compression) {
         String fileName = getFileName(handler, tableId, objectId);
-        return new ValueLob(type, handler, fileName, tableId, objectId, true, precision, compression);
+        return new ValueLob(type, handler, fileName, tableId, objectId, true/*linked*/, precision, compression);
+    }
+
+    /**
+     * Create a LOB value with the given parameters.
+     *
+     * @param type the data type
+     * @param handler the file handler
+     * @param tableId the table object id
+     * @param objectId the object id
+     * @param precision the precision (length in elements)
+     * @param compression if compression is used
+     * @return the value object
+     */
+    public static ValueLob openUnlinked(int type, DataHandler handler,
+            int tableId, int objectId, long precision, boolean compression, String fileName) {
+        return new ValueLob(type, handler, fileName, tableId, objectId, false/*linked*/, precision, compression);
     }
     
     /**
@@ -762,17 +778,6 @@ public class ValueLob extends Value {
         }
     }
 
-    /**
-     * Set the file name of this lob value.
-     *
-     * @param fileName the file name
-     * @param linked if the lob is linked
-     */
-    public void setFileName(String fileName, boolean linked) {
-        this.fileName = fileName;
-        this.linked = linked;
-    }
-
     public int getMemory() {
         if (small != null) {
             return small.length + 104;