- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] - 250 * 1.5 is not the same as 250 * 1.5

by ClÃ©ment Bera

2015-09-24 8:19 GMT+02:00 Peter UhnÃ¡k <i.uhnak(a)gmail.com>: > > - 250 * 1.5 returns -2.25 > > why is this valid syntax? > It should not. RBParser bug. This is not valid Smalltalk syntax. As Ben showed similar cases raise an error but this one generate incorrect compiledMethod instead. > > On Thu, Sep 24, 2015 at 5:35 AM, Ben Coman <btc(a)openinworld.com> wrote: > >> RBParser parseExpression: '-250 * 1.5' => "RBMessageNode(-250 * 1.5)" >> RBParser parseExpression: '- 250 * 1.5' => "RBMessageNode(*1.5 * 1.5)" >> >> RBParser parseExpression: '250 * -1.5' => "RBMessageNode(250 * -1.5)" >> RBParser parseExpression: '250 * - 1.5' => MNU RBToken>>realvalue. >> >> RBParser parseExpression: '-250' => "RBLiteralValueNode(-250)" >> RBParser parseExpression: '- 250' => MNU RBToken>>realvalue. >> >> So how significant should be the space between the negative sign and its >> number? >> >> cheers -ben >> >> On Thu, Sep 24, 2015 at 6:34 AM, ClÃ©ment Bera <bera.clement(a)gmail.com> >> wrote: >> > RBParser parseExpression: '- 250 * 1.5' => Answers >> RBMessageNode(*1.5 * >> > 1.5) >> > >> > Second expression looks correct. >> > >> > Hence RBParser bug. >> > >> > 2015-09-23 23:42 GMT+02:00 phil(a)highoctane.be <phil(a)highoctane.be>: >> >> >> >> debug it on the first one gives "MNU: receiver of >> "doSemanticAnalyisIn:" >> >> is nil. >> >> >> >> Weird. >> >> >> >> On Wed, Sep 23, 2015 at 8:40 AM, stepharo <stepharo(a)free.fr> wrote: >> >>> >> >>> Hi guys >> >>> >> >>> I think that this is quite bad that >> >>> - 250 * 1.5 returns -2.25 >> >>> >> >>> while >> >>> >> >>> -250 * 1.5 return 375 >> >>> >> >>> stef >> >>> >> >> >> > >> >> >

Sept. 24, 2015

Ctrl-b-m Ctrl-b-n and Ctrl-b-N should work in all code panes

by stepharo

Hi guys I can understand that nautilus has its own shortcuts. Now it would be good that have Ctrl-b-m Ctrl-b-n and Ctrl-b-N should work in all code panes in addition to Ctrl-m and Ctrl-n So that it is the same for students. Stef

Sept. 24, 2015

Re: [Pharo-dev] - 250 * 1.5 is not the same as 250 * 1.5

by Nicolai Hess

2015-09-24 8:19 GMT+02:00 Peter UhnÃ¡k <i.uhnak(a)gmail.com>: > > - 250 * 1.5 returns -2.25 > > why is this valid syntax? > I don't know if this is valid syntax (I always wondered that there is one place in PointTest, that is not compilable with old compiler but compiles fine with opal). But the error happens in RBParser>>#parseNegatedNumber. If a #- is recognized, it tests if a literalnumber follows (but from the token stream, that is, the spaces are ignored). Now the real bug is, that we do two "steps" and concstruct a new literalvaluenode from the now following tokens. RBParser parseExpression:'- 2' -> throws an error, because no following tokens RBParser parseExpression:'- 2 * 3' -> works but actually duplicates the two last tokens "*3 *3. and some funny other things '- 2@1' -> '-1@1' instead, we should: save sign and number token do the two steps construct the literalvaluenode fromt the two saved tokens: parseNegatedNumber |tokenSign tokenNumber| (self nextToken isLiteralToken not or: [ self nextToken realValue isNumber not ]) ifTrue: [ ^ self parserError: 'The ''-'' prefix works only for literal numbers (use #negated instead)' ]. tokenSign := currentToken. self step. tokenNumber := currentToken. self step. ^ self literalValueNodeClass value: tokenNumber realValue negated start: tokenSign start stop: tokenNumber stop source: tokenSign value, tokenNumber source. But only if we want to support spaces between sign and number tokens. BTW, if we use "-2" without space, parseNegatedNumber isn't even called, I guess the scanner already creates a single (negated)number token. > > On Thu, Sep 24, 2015 at 5:35 AM, Ben Coman <btc(a)openinworld.com> wrote: > >> RBParser parseExpression: '-250 * 1.5' => "RBMessageNode(-250 * 1.5)" >> RBParser parseExpression: '- 250 * 1.5' => "RBMessageNode(*1.5 * 1.5)" >> >> RBParser parseExpression: '250 * -1.5' => "RBMessageNode(250 * -1.5)" >> RBParser parseExpression: '250 * - 1.5' => MNU RBToken>>realvalue. >> >> RBParser parseExpression: '-250' => "RBLiteralValueNode(-250)" >> RBParser parseExpression: '- 250' => MNU RBToken>>realvalue. >> >> So how significant should be the space between the negative sign and its >> number? >> >> cheers -ben >> >> On Thu, Sep 24, 2015 at 6:34 AM, ClÃ©ment Bera <bera.clement(a)gmail.com> >> wrote: >> > RBParser parseExpression: '- 250 * 1.5' => Answers >> RBMessageNode(*1.5 * >> > 1.5) >> > >> > Second expression looks correct. >> > >> > Hence RBParser bug. >> > >> > 2015-09-23 23:42 GMT+02:00 phil(a)highoctane.be <phil(a)highoctane.be>: >> >> >> >> debug it on the first one gives "MNU: receiver of >> "doSemanticAnalyisIn:" >> >> is nil. >> >> >> >> Weird. >> >> >> >> On Wed, Sep 23, 2015 at 8:40 AM, stepharo <stepharo(a)free.fr> wrote: >> >>> >> >>> Hi guys >> >>> >> >>> I think that this is quite bad that >> >>> - 250 * 1.5 returns -2.25 >> >>> >> >>> while >> >>> >> >>> -250 * 1.5 return 375 >> >>> >> >>> stef >> >>> >> >> >> > >> >> >

Sept. 24, 2015

Re: [Pharo-dev] call "compress file " from the FileList

by stepharo

yes probably. Stef Le 13/9/15 06:49, Marcus Denker a Ã©crit : > >> On 13 Sep 2015, at 00:26, Nicolai Hess <nicolaihess(a)web.de >> <mailto:nicolaihess@web.de>> wrote: >> >> There was a bug on compress file >> (see issue >> https://pharo.fogbugz.com/f/cases/10235/StandardFileStream-compressFile-can…) >> >> This is fixed know for Pharo 4.0, anyone knows why we can not call >> "compress file" >> anymore in Pharo 5.0 ? >> > > I think this could be a side effect (bug) from the new pragma based > registration mechanism for registering actions > in the FileList? > > Marcus

Sept. 24, 2015

Re: [Pharo-dev] - 250 * 1.5 is not the same as 250 * 1.5

by Peter UhnÃ¡k

> - 250 * 1.5 returns -2.25 why is this valid syntax? On Thu, Sep 24, 2015 at 5:35 AM, Ben Coman <btc(a)openinworld.com> wrote: > RBParser parseExpression: '-250 * 1.5' => "RBMessageNode(-250 * 1.5)" > RBParser parseExpression: '- 250 * 1.5' => "RBMessageNode(*1.5 * 1.5)" > > RBParser parseExpression: '250 * -1.5' => "RBMessageNode(250 * -1.5)" > RBParser parseExpression: '250 * - 1.5' => MNU RBToken>>realvalue. > > RBParser parseExpression: '-250' => "RBLiteralValueNode(-250)" > RBParser parseExpression: '- 250' => MNU RBToken>>realvalue. > > So how significant should be the space between the negative sign and its > number? > > cheers -ben > > On Thu, Sep 24, 2015 at 6:34 AM, ClÃ©ment Bera <bera.clement(a)gmail.com> > wrote: > > RBParser parseExpression: '- 250 * 1.5' => Answers > RBMessageNode(*1.5 * > > 1.5) > > > > Second expression looks correct. > > > > Hence RBParser bug. > > > > 2015-09-23 23:42 GMT+02:00 phil(a)highoctane.be <phil(a)highoctane.be>: > >> > >> debug it on the first one gives "MNU: receiver of "doSemanticAnalyisIn:" > >> is nil. > >> > >> Weird. > >> > >> On Wed, Sep 23, 2015 at 8:40 AM, stepharo <stepharo(a)free.fr> wrote: > >>> > >>> Hi guys > >>> > >>> I think that this is quite bad that > >>> - 250 * 1.5 returns -2.25 > >>> > >>> while > >>> > >>> -250 * 1.5 return 375 > >>> > >>> stef > >>> > >> > > > >

Sept. 24, 2015

Re: [Pharo-dev] - 250 * 1.5 is not the same as 250 * 1.5

by Ben Coman

RBParser parseExpression: '-250 * 1.5' => "RBMessageNode(-250 * 1.5)" RBParser parseExpression: '- 250 * 1.5' => "RBMessageNode(*1.5 * 1.5)" RBParser parseExpression: '250 * -1.5' => "RBMessageNode(250 * -1.5)" RBParser parseExpression: '250 * - 1.5' => MNU RBToken>>realvalue. RBParser parseExpression: '-250' => "RBLiteralValueNode(-250)" RBParser parseExpression: '- 250' => MNU RBToken>>realvalue. So how significant should be the space between the negative sign and its number? cheers -ben On Thu, Sep 24, 2015 at 6:34 AM, ClÃ©ment Bera <bera.clement(a)gmail.com> wrote: > RBParser parseExpression: '- 250 * 1.5' => Answers RBMessageNode(*1.5 * > 1.5) > > Second expression looks correct. > > Hence RBParser bug. > > 2015-09-23 23:42 GMT+02:00 phil(a)highoctane.be <phil(a)highoctane.be>: >> >> debug it on the first one gives "MNU: receiver of "doSemanticAnalyisIn:" >> is nil. >> >> Weird. >> >> On Wed, Sep 23, 2015 at 8:40 AM, stepharo <stepharo(a)free.fr> wrote: >>> >>> Hi guys >>> >>> I think that this is quite bad that >>> - 250 * 1.5 returns -2.25 >>> >>> while >>> >>> -250 * 1.5 return 375 >>> >>> stef >>> >> >

Sept. 24, 2015

Re: [Pharo-dev] - 250 * 1.5 is not the same as 250 * 1.5

by ClÃ©ment Bera

RBParser parseExpression: '- 250 * 1.5' => Answers RBMessageNode(*1.5 * 1.5) Second expression looks correct. Hence RBParser bug. 2015-09-23 23:42 GMT+02:00 phil(a)highoctane.be <phil(a)highoctane.be>: > debug it on the first one gives "MNU: receiver of "doSemanticAnalyisIn:" > is nil. > > Weird. > > On Wed, Sep 23, 2015 at 8:40 AM, stepharo <stepharo(a)free.fr> wrote: > >> Hi guys >> >> I think that this is quite bad that >> - 250 * 1.5 returns -2.25 >> >> while >> >> -250 * 1.5 return 375 >> >> stef >> >> >

Sept. 23, 2015

Re: [Pharo-dev] Some references to transcript should be changed into File stdout

by phil＠highoctane.be

NonInteractiveTranscript is usually the class then, so no need to File stdout anything I guess. NonInteractiveTranscript install is "de rigueur" for CLI style apps. Phil On Wed, Sep 23, 2015 at 1:16 PM, stepharo <stepharo(a)free.fr> wrote: > Hi > > I have the impression that some references to Transcript should be > rewritten as File stdout > > For example > > DummyUIManager>>unhandledErrorDefaultAction: anException > > Transcript show: '*** EXCEPTION ***'; cr. > Transcript show: anException; cr. > Transcript show: (thisContext stack first: (20 min: thisContext stack > size)); cr. > anException isResumable ifTrue: [ anException resume ] > > Stef > >

Sept. 23, 2015

Re: [Pharo-dev] - 250 * 1.5 is not the same as 250 * 1.5

by phil＠highoctane.be

debug it on the first one gives "MNU: receiver of "doSemanticAnalyisIn:" is nil. Weird. On Wed, Sep 23, 2015 at 8:40 AM, stepharo <stepharo(a)free.fr> wrote: > Hi guys > > I think that this is quite bad that > - 250 * 1.5 returns -2.25 > > while > > -250 * 1.5 return 375 > > stef > >

Sept. 23, 2015

Re: [Pharo-dev] call "compress file " from the FileList

by stepharo

yes probably. Stef Le 13/9/15 06:49, Marcus Denker a Ã©crit : > >> On 13 Sep 2015, at 00:26, Nicolai Hess <nicolaihess(a)web.de >> <mailto:nicolaihess@web.de>> wrote: >> >> There was a bug on compress file >> (see issue >> https://pharo.fogbugz.com/f/cases/10235/StandardFileStream-compressFile-can…) >> >> This is fixed know for Pharo 4.0, anyone knows why we can not call >> "compress file" >> anymore in Pharo 5.0 ? >> > > I think this could be a side effect (bug) from the new pragma based > registration mechanism for registering actions > in the FileList? > > Marcus

Sept. 23, 2015