I don't think we ever need setInBytes(false) -- you could leave out the argument. Also, you could assert for safety that m_valueType == VARCHAR.