Short circuit.

pdal/DimUtil.hpp

@@ -183,6 +183,47 @@ inline Type type(const std::string& baseType, size_t size)
     return static_cast<Type>((size_t)(base) | size);
 }
 
+template<typename T>
+inline constexpr Type type()
+{ return Type::None; }
+
+template<>
+inline constexpr Type type<float>()
+{ return Type::Float; }
+
+template<>
+inline constexpr Type type<uint64_t>()
+{ return Type::Unsigned64; }
+
+template<>
+inline constexpr Type type<uint32_t>()
+{ return Type::Unsigned32; }
+
+template<>
+inline constexpr Type type<uint16_t>()
+{ return Type::Unsigned16; }
+
+template<>
+inline constexpr Type type<uint8_t>()
+{ return Type::Unsigned8; }
+
+
+template<>
+inline constexpr Type type<int64_t>()
+{ return Type::Signed64; }
+
+template<>
+inline constexpr Type type<int32_t>()
+{ return Type::Signed32; }
+
+template<>
+inline constexpr Type type<int16_t>()
+{ return Type::Signed16; }
+
+template<>
+inline constexpr Type type<int8_t>()
+{ return Type::Signed8; }
+
 /// Extract a dimension name of a string.  Dimension names start with an alpha
 /// and continue with numbers or underscores.
 /// \param s  String from which to extract dimension name.

pdal/PointRef.hpp

@@ -75,6 +75,13 @@ class PDAL_DLL PointRef
         Everything e;
         Dimension::Type type = m_layout.dimDetail(dim)->type();
 
+        if (Dimension::type<T>() == type)
+        {
+            T t;
+            m_container.getFieldInternal(dim, m_idx, &t);
+            return t;
+        }
+
         m_container.getFieldInternal(dim, m_idx, &e);
         switch (type)
         {
@@ -135,6 +142,10 @@ class PDAL_DLL PointRef
     void setField(Dimension::Id dim, T val)
     {
         Dimension::Type type = m_layout.dimDetail(dim)->type();
+
+        if (Dimension::type<T>() == type)
+            m_container.setFieldInternal(dim, m_idx, &val);
+
         Everything e;
         bool success = false;
 

pdal/PointView.hpp

@@ -525,6 +525,9 @@ void PointView::setField(Dimension::Id dim, PointId idx, T val)
 {
     const Dimension::Detail *dd = layout()->dimDetail(dim);
 
+    if (Dimension::type<T>() == dd->type())
+        setFieldInternal(dim, idx, &val);
+
     bool ok = true;
     switch (dd->type())
     {