Please let me know what you come up with for a final patch and I will try to incorporate this weekend...
> -----Original Message----- > From: Alexander Foken [mailto:[EMAIL PROTECTED] > Sent: Tuesday, March 21, 2006 2:56 AM > To: Martin J. Evans > Cc: dbi-users@perl.org > Subject: Re: [dbi] Unicode-Patch for DBD::ODBC > > Hello Martin, > > just a quick reply, I'm in a hurry, so please let me verify this later. > > Yes, this really looks wrong. But if I remember right, there was a > reason for the "+". I think I (ab)used phs->sv to store Unicode in a > char[], so I just made sure there was enough room for the trailing > U+0000. If that's right, phs->maxlen counts bytes, not characters. > > Or maybe I really missed the Shift key twice. ;-) > > Alexander > > Martin J. Evans wrote: > > >Alexander, > > > >Thanks for this. I've not had a chance to try it but a quick scan showed: > > > >- SvGROW(phs->sv, (phs->maxlen < 28) ? 28 : phs->maxlen+1); > >+ SvGROW(phs->sv, (phs->maxlen+sizeof(WCHAR) < 28) ? 28 : > >phs->maxlen+sizeof(WCHAR)); > > > >Only because I was looking at this very line the other day it stood out - > >should that be "*sizeof(WCHAR)". > > > >Martin > >-- > >Martin J. Evans > >Easysoft Ltd, UK > >http://www.easysoft.com > > > >On 15-Mar-2006 Alexander Foken wrote: > > > > > >>Hello DBI-users, > >> > >>I've written a first patch for DBD::ODBC to support Unicode. The patch > >>was developed to make a custom project management software work with > >>Unicode on Win32 with Microsoft's SQL Server via ODBC. It is in use at > >>several clients since about six months. So far, no bugs have been > >>reported for the modified DBD::ODBC driver. > >> > >>The patch is available for download at from > >>http://www.alexander-foken.de/unicode-patch.txt.gz. You find detailed > >>information in the README.unicode-patch file, HTML version available at > >>http://www.alexander-foken.de/README.unicode-patch.html. > >> > >>Please remember to read the README.unicode-patch file, especially the > >>"Known Problems" section. There are still some rough edges where things > >>work different than you may expect. > >> > >>Alexander > >> > >>-- > >>Alexander Foken > >>mailto:[EMAIL PROTECTED] http://www.foken.de/alexander/ > >> > >> > > > > > > > > > -- > Alexander Foken > mailto:[EMAIL PROTECTED] http://www.foken.de/alexander/