tx I will look at it. Stef
On Jun 7, 2013, at 8:57 PM, José Comesaña <jose.comes...@gmail.com> wrote: > Done. > > I hope I have made it correctly. If not, please tell me. > > Regards > > 2013/6/7 Stéphane Ducasse <stephane.duca...@inria.fr> > Here it is https://pharo.fogbugz.com/default.asp?10869 > > Stef > > > > > On Jun 7, 2013, at 8:17 PM, Stéphane Ducasse <stephane.duca...@inria.fr> > wrote: > >> >> >>> Hi again. >>> >>> I have one first test for #readFrom:pattern:. What can I do now to send it, >>> where, to whom?. >> >> I will create a bug entry for you and you create a slice with the test and >> the fix and you publish it in the pharo inbox. >> >> Stef >> >>> >>> Regards >>> >>> 2013/6/7 Sven Van Caekenberghe <s...@stfx.eu> >>> >>> On 07 Jun 2013, at 14:28, José Comesaña <jose.comes...@gmail.com> wrote: >>> >>> > Answers in red. >>> > >>> > Regards >>> > >>> > 2013/6/7 Sven Van Caekenberghe <s...@stfx.eu> >>> > >>> > On 07 Jun 2013, at 13:26, José Comesaña <jose.comes...@gmail.com> wrote: >>> > >>> > > Thank you Sven. >>> > > >>> > > I had made the fixes myself, creating a new class for dates (I don't >>> > > need the complexity of Date just for determining if a date is past, I >>> > > don't need date arithmetic either). I just wanted to inform, mainly >>> > > because the bug seems to be still present in Pharo 3.0. >>> > > >>> > > Anyway, your solution does not work for the 'd/m/y' pattern, although >>> > > it does for 'd/m/yy'. >>> > >>> > Depends on what a single y means, right ? >>> > >>> > According to documentation, one y means ...year, no matter how many >>> > digits supplied... (my own words, not litteral). It means it will either >>> > accept ../../13 or ../../2013 >>> >>> Yes, that is how I understood it as well. My change adds 2000 only in the >>> case of 2 y's unless I am mistaken. But I must admin I didn't test it. >>> >>> > > I could write the tests you suggest, if you think they are interesting. >>> > >>> > Yes, please do: with a good set of tests, we can discuss using concrete >>> > examples. >>> > >>> > >>> > Sure I will. Will keep you informed! >>> >>> Super. >>> >>> > > Regards >>> > > >>> > > 2013/6/7 Sven Van Caekenberghe <s...@stfx.eu> >>> > > Hi José, >>> > > >>> > > On 07 Jun 2013, at 12:02, José Comesaña <jose.comes...@gmail.com> wrote: >>> > > >>> > > > Hello everybody. >>> > > > >>> > > > I would like to add a new improvement to this thread. >>> > > > >>> > > > I have noticed that Date class has a bug, or at least something to >>> > > > improve in readFrom:pattern: method. The initial comment states that >>> > > > "A year given using only two decimals is considered to be >2000.". >>> > > > >>> > > > But this seems to be incorrect: if you evaluate >>> > > > >>> > > > Date readFrom: '07/06/13' readStream pattern: 'd/m/y' >>> > > > >>> > > > you get: >>> > > > >>> > > > 7 June 0013 instead of 7 June 2013. >>> > > > >>> > > > I am working with version #20605 >>> > > >>> > > The fix would be quite easy: >>> > > >>> > > year := (inputStream next: 2) asInteger >>> > > >>> > > should be >>> > > >>> > > year := 2000 + (inputStream next: 2) asInteger >>> > > >>> > > A more important problem is that there are no users of >>> > > #readFrom:pattern: in the system, not even unit tests. If we want to >>> > > keep the methods, someone should start by writing a couple of unit >>> > > tests. >>> > > >>> > > Sven >>> > > >>> > > > Thank you all for your great work. >>> > > > >>> > > > Regards. >>> > > > >>> > > > >>> > > > 2013/6/6 Camillo Bruni <camillobr...@gmail.com> >>> > > > can you save an image with the inspector opened on both x and y, x >>> > > > asTime and y asTime >>> > > > and provide a link to it? like that I can inspect it in all detail... >>> > > > >>> > > > On 2013-06-06, at 17:51, dmacq <dm...@instantiations.com> wrote: >>> > > > >>> > > > > I tried it again this morning and still had the problem. >>> > > > > >>> > > > > This is interesting. >>> > > > > >>> > > > > | x y | >>> > > > > x := DateAndTime year: 1991 day: 196 hour: 20 minute: 5 second: 7. >>> > > > > y := DateAndTime year: 1991 month: 7 day: 15 hour: 20 minute: 5 >>> > > > > second: 7. >>> > > > > x = y <------------------ True >>> > > > > >>> > > > > But >>> > > > > | x y | >>> > > > > x := DateAndTime year: 1991 day: 196 hour: 20 minute: 5 second: 7. >>> > > > > y := DateAndTime year: 1991 month: 7 day: 15 hour: 20 minute: 5 >>> > > > > second: 7. >>> > > > > x asTime = y asTime <------------- False >>> > > > > >>> > > > > >>> > > > > >>> > > > > -- >>> > > > > View this message in context: >>> > > > > http://forum.world.st/DateAndTime-asTime-tp4691870p4691988.html >>> > > > > Sent from the Pharo Smalltalk Users mailing list archive at >>> > > > > Nabble.com. >>> > > > > >>> > > > >>> > > > >>> > > > >>> > > >>> > > >>> > > >>> > >>> > >>> > >>> >>> >>> >> > >