FreeTDS Development Group

["Brian Bruns" <camber AT ais.org>]

> > 
> > Ok, I think I know what is going on.  In 
> > tds7_process_result() there is a
> > switch statement that sets curcol->column_unicodedata if the 
> > column type
> > is a unicode capable datatype.  ntext is missing from the 
> > list. I don't
> > have time (nor am i somewhere convenient) to make a patch 
> > right now but we
> > really should be calling is_unicode() on the datatype instead 
> > of having
> > this int the switch statement, and then fixing it in the 
> > header, nice and
> > clean.
> > 
> > Brian
> > 
> Yes but after that you should correct column_type to SYBTEXT.
> I think this is a good and correct solution.
> I expect that in the next release (after 0.60) column_unicodedata just
> disappear...
> 
> freddy77

Good point, we used to have a column_xtype of something of the sort so we
didn't lose this info.  column_type was the cardinal type and column_xtype
was the actual type.  I'd prefer to have a tds_get_cardinal_type()
function instead of inlining this switch.  If I apply the patch, I'll do
just that, if someone else applies it, please consider doing it.

Brian

Brian