FreeTDS Development Group

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

tds_process_row() is getting unwieldy.  Post 0.60 work should include
streamlining this func.  Looks like we can simply test for
column_unicodedata and instead call tds_get_string instead of tds_get_n. 
I don't have access to SQL Server where i am now to test this.  Did you
break the switch out to a separate function?

Brian

> > 
> > 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
> > 
> Opss.. I just tried to apply...
> There are a small problem.
> 
> blob type (and also TEXT) are read with tds_get_n and if unicodedata is true
> it use strlen on code giving just a core dump
> 
> freddy77