RE: Hint to bug #12127,#10896 .... (Function fgetcsv() lost some lett ers) (fwd)

Forgot to forward this to the list. Could someone check this out? --Jani ---------- Forwarded message ---------- Date: Wed, 26 Sep 2001 14:54:51 +0200 From: "Tesacek, Jan" <jan.tesacek@sap.com> To: 'Jani Taskinen' <sniper@iki.fi> Subject: RE: Hint to bug #12127, #10896 .... (Function fgetcsv() lost some lett ers) Hello again, I locate and repair this bug, but I am behind firewall and I cannot access cvs. In file.c file in function fgetcsv is used this syntax to call function isspace : "isspace((int)*ptr)", replace it with "isspace((unsigned char)*ptr). Jan Tesacek > -----Original Message----- > From: Jani Taskinen [mailto:sniper@iki.fi] > Sent: Tuesday, September 18, 2001 12:41 PM > To: Tesacek, Jan > Subject: Re: Hint to bug #12127,#10896 .... (Function > fgetcsv() lost some lett ers) > > > > Sorry, but I'm not the correct person to send these since > I don't have any windows compiler and my knowledge of that system > is quite limited. :) > > Please send this information to php-dev@lists.php.net > > --Jani > > > On Tue, 18 Sep 2001, Tesacek, Jan wrote: > > >Hello, > >I send email to rasmus@php.net, but no respond to me. Today > I try developer > >build 4.0.8 from www.php4win.de and this bug is also in this > build (I try > >version 4.0.4pl1,4.0.5,4.0.6 and this bug is in all builds). I try to > >reproduce this bug on 5 OS (1xW2K Advanced server English > with sp1 and sp2, > >2x W2K Profesional English with sp1 and sp2,1x W2K > Profesional Czech with > >sp2, 1xWinNT 4.0 with sp6) - and result all W2K OS has same > bug result, NT > >has right result. I try to explore binding to Regional > Setting - but no > >binding to Regional Settings. > >My result is that, problem is not in fgetcsv function, but > in somewhere in C > >function isspace, which is used in file.c - When replace > isspace(c) with > >(c==' ') and it works right. (I dont make new build to test > this, but i path > >php4ts.dll in hex editor). In attachment i send you my > php.ini, test php, > >and diff (offset <tab> original value <tab> modified value) > to php4ts.dll > >(v. 4.0.6). > >I hope that you excuse my terrible english ... > > Jan Tesacek > > > > >