It is the subject of this thread. The fix itself is trivial, (changing one byte), but I didn't know how to come up with a testcase. Henry and you have provide that. I think Henry will fix it when he has spare time.
Чт, 09 июн 2016, JSource написал(а): > Even if this is "MS's fault", could windows get the same behaviour as linux > on this? > > > > > ----- Original Message ----- > From: bill lam <bbill....@gmail.com> > To: Source forum <sou...@jsoftware.com> > Sent: Thursday, June 9, 2016 9:20 AM > Subject: Re: [Jsource] strtol > > No problem on linux64 > > a_12345678901234_ =: 3 > cocreate '' > +--------------+ > |12345678901235| > +--------------+ > JVERSION > Engine: j805/j64/linux > Beta: commercial/2016-05-10T12:23:26 > Library: 8.04.15 > Platform: Linux 64 > Installer: unknown > InstallPath: /usr/share/j/8.0.4 > Contact: www.jsoftware.com > > I suspect it only breaks on windows64 where strtol is 32-bit. > > a_12345678901234_ =: 3 > cocreate'' > +----------+ > |2147483648| > +----------+ > JVERSION > Engine: j804/j64/windows > Release: commercial/2015-12-21 16:18:48 > Library: 8.04.15 > Platform: Win 64 > Installer: j801 beta install > InstallPath: z:/usr/share/j/8.0.4 > Contact: www.jsoftware.com > > > > Чт, 09 июн 2016, bill lam написал(а): > > Did you test on windows? Does it also failed on linux or mac? > > On Jun 9, 2016 8:11 PM, "'Pascal Jasmin' via Source" <sou...@jsoftware.com> > > wrote: > > > > > On a (afaiu) related note, could the numbered locale limit be raised on 64 > > > bit systems? > > > > > > > > > a_12345678901234_ =: 3 > > > > > > cocreate '' > > > ┌───────────┐ > > > │-2147483646│ > > > └───────────┘ > > > > > > > > > using unlimited extended integers (to track next object number) on all > > > systems (or just on 32bit) would seem viable as well. > > > > > > The constraint becomes an issue with systems that hope to have high > > > uptime. > > > > > > > > > > > > > > > ----- Original Message ----- > > > From: Henry Rich <henryhr...@gmail.com> > > > To: sou...@jsoftware.com > > > Sent: Wednesday, June 8, 2016 12:25 PM > > > Subject: Re: [Jsource] strtol > > > > > > The way to create a testcase is to create a locale with a very large > > > number, by > > > > > > a_2147483648_ =: 5 > > > > > > locale numbering will start after the largest number that has been used. > > > > > > Henry > > > > > > On 6/8/2016 8:54 AM, bill lam wrote: > > > > I think using strtoI should be harmless, but I am not sure if u will > > > exceed > > > > range of 32-bit integer. And how to design a test script to show strtol > > > > will fail while strtoI will pass. > > > > > > > > PS. I realized it is nearly impossible to distinguish strtol (last > > > > character is el) and strtoI (last is capital i) on android gmail app. > > > > On Jun 8, 2016 8:36 PM, "Raul Miller" <rauldmil...@gmail.com> wrote: > > > > > > > >> Oh! > > > >> > > > >> There is also a microsoft supplied definition for strtoI, I was > > > >> similarly lazy and forgot to look in the source code to see if it > > > >> redefined the name (which it looks like it does). > > > >> > > > >> So one part of my objection (that strtoI would cause problems which > > > >> currently do not happen) should not be the case. > > > >> > > > >> But the other issue - that it does not seem like strtoI would fix > > > >> anything (would not make J perform in a fashion which is closer to > > > >> what was prescribed by the dictionary), either - might still be an > > > >> issue. > > > >> > > > >> Do you disagree? > > > >> > > > >> Thanks, > > > >> > > > >> -- > > > >> Raul > > > >> > > > >> > > > >> On Wed, Jun 8, 2016 at 1:12 AM, bill lam <bbill....@gmail.com> wrote: > > > >>> I am sorry that I was too lazy to include the definition of > > > >>> strtoI for discussion > > > >>> > > > >>> #if SY_64 > > > >>> #define IMAX 9223372036854775807L > > > >>> #define FMTI "%lli" > > > >>> #define FMTI02 "%02lli" > > > >>> #define FMTI04 "%04lli" > > > >>> #define FMTI05 "%05lli" > > > >>> > > > >>> #if SY_WIN32 > > > >>> #define strtoI _strtoi64 > > > >>> #else > > > >>> #define strtoI strtoll > > > >>> #endif > > > >>> > > > >>> #else > > > >>> #define IMAX 2147483647L > > > >>> #define FMTI "%li" > > > >>> #define FMTI02 "%02li" > > > >>> #define FMTI04 "%04li" > > > >>> #define FMTI05 "%05li" > > > >>> #define strtoI strtol > > > >>> #endif > > > >>> > > > >>> For *nix, strtol and strtoll are almost equivalent. > > > >>> > > > >>> Вт, 07 июн 2016, Raul Miller написал(а): > > > >>>> I think strtol (long result) should be better than strtoI (int > > > result). > > > >>>> > > > >>>> The distinction should only matter when a numeric local name exceeds > > > >>>> the value which can be stored in a 32 bit integer. That's a rare > > > >>>> problem and it's probably more important that the behavior be correct > > > >>>> for that case on 64 bit systems than on 32 bit systems. Worse, > > > >>>> changing to strtoI would not actually fix any failure case, but it > > > >>>> would introduce additional failure cases. > > > >>>> > > > >>>> That said, it might be worthwhile investigating a solution for the > > > >>>> resulting problem. I guess you would first want to cocreate/coerase > > > >>>> about 5 billion times to see if that works. And, if it fails, work > > > >>>> from there. > > > >>>> > > > >>>> One possible failure mode would be overlapping locales. For example, > > > >>>> locale 4294967296 on a 32 bit system might be the same as locale 0. > > > >>>> Other failure modes (crashes or whatever) might also be possible? > > > >>>> > > > >>>> Thanks, > > > >>>> > > > >>>> -- > > > >>>> Raul > > > >>>> > > > >>>> On Tue, Jun 7, 2016 at 9:00 PM, bill lam <bbill....@gmail.com> wrote: > > > >>>>> There is still one artefact of strtol in sl.c > > > >>>>> > > > >>>>> A jtstfind(J jt,B b,I n,C*u){I old;L*v; > > > >>>>> if(!n){n=4; u="base";} > > > >>>>> if('9'>=*u)R stfindnum(b,strtol(u,NULL,10)); > > > >>>>> else{ > > > >>>>> old=jt->tbase+jt->ttop; v=probe(nfs(n,u),jt->stloc); > > > >>>>> tpop(old); > > > >>>>> R v?v->val:b?stcreate(0,jt->locsize[0],n,u):0; > > > >>>>> }} /* find the symbol table for locale u, create if b and > > > >>>>> non-existent */ > > > >>>>> > > > >>>>> Is it safe to use strtol here? > > > >>>>> > > > >>>>> Is it ok to change to strtoI? > > > >>>>> > > > >>>>> * strtol returns 32-bit integers under 32-bit windows. > > > >>>>> > > > >>>>> -- > > > >>>>> regards, > > > >>>>> ==================================================== > > > >>>>> GPG key 1024D/4434BAB3 2008-08-24 > > > >>>>> gpg --keyserver subkeys.pgp.net --recv-keys 4434BAB3 > > > >>>>> gpg --keyserver subkeys.pgp.net --armor --export 4434BAB3 > > > >>>>> > > > ---------------------------------------------------------------------- > > > >>>>> For information about J forums see > > > >> http://www.jsoftware.com/forums.htm > > > >>>> ---------------------------------------------------------------------- > > > >>>> For information about J forums see > > > http://www.jsoftware.com/forums.htm > > > > > > >>> -- > > > >>> regards, > > > >>> ==================================================== > > > >>> GPG key 1024D/4434BAB3 2008-08-24 > > > >>> gpg --keyserver subkeys.pgp.net --recv-keys 4434BAB3 > > > >>> gpg --keyserver subkeys.pgp.net --armor --export 4434BAB3 > > > >>> ---------------------------------------------------------------------- > > > >>> For information about J forums see http://www.jsoftware.com/forums.htm > > > >> ---------------------------------------------------------------------- > > > >> For information about J forums see http://www.jsoftware.com/forums.htm > > > > ---------------------------------------------------------------------- > > > > For information about J forums see http://www.jsoftware.com/forums.htm > > > > > > > ---------------------------------------------------------------------- > > > For information about J forums see http://www.jsoftware.com/forums.htm > > > ---------------------------------------------------------------------- > > > For information about J forums see http://www.jsoftware.com/forums.htm > > -- > regards, > ==================================================== > GPG key 1024D/4434BAB3 2008-08-24 > gpg --keyserver subkeys.pgp.net --recv-keys 4434BAB3 > gpg --keyserver subkeys.pgp.net --armor --export 4434BAB3 > ---------------------------------------------------------------------- > For information about J forums see http://www.jsoftware.com/forums.htm > ---------------------------------------------------------------------- > For information about J forums see http://www.jsoftware.com/forums.htm -- regards, ==================================================== GPG key 1024D/4434BAB3 2008-08-24 gpg --keyserver subkeys.pgp.net --recv-keys 4434BAB3 gpg --keyserver subkeys.pgp.net --armor --export 4434BAB3 ---------------------------------------------------------------------- For information about J forums see http://www.jsoftware.com/forums.htm