Hi,

Great work!

These last broken tests were quite tricky because of the not obvious keyworkdsPositions setup. Nice.

While I fixed the previous tests, I added an empty body in the methodDeclaration

"protocol: grammar"
methodDeclaration
^ super methodDeclaration
==> [ :nodes | 
(RBMethodNode
selector: (self buildSelector: nodes first)
keywordsPositions: (nodes first collect: [ :each | each start ])
arguments: nodes second)
body: (RBSequenceNode statements: #());
yourself ]

Could you check if this is the correct way to initialize the RBMethodNode?

Cheers,
Doru


On Sat, Mar 14, 2015 at 1:31 AM, Bernardo Ezequiel Contreras <vonbecmann@gmail.com> wrote:
hi all,
 Many test on PetitSmalltalk were failing so i tried to fix them.
 someone should review the changes, there are many, and it's my
first time with petitparser.
 see attached file.

Steps:
- evaluate
Gofer it
smalltalkhubUser: 'Pharo' project: 'MetaRepoForPharo40';
configurationOf: 'PetitParser';
loadBleedingEdge.

- and merge attached file
Name: PetitSmalltalk-BernardoContreras.84
Author: BernardoContreras
Time: 13 March 2015, 9:24:08.132253 pm
UUID: fc0534b1-3411-485c-87bc-0afa4eaab84d
Ancestors: PetitSmalltalk-TudorGirba.83

PPSmalltalkClassesTests>>#verifyClass:selector:
compare the keywordsPositions instead of the selectorParts

add PPSmalltalkGrammarTests>>#testPragma17
PPSmalltalkGrammarTests>>#testPragma18
PPSmalltalkParserTests>>#testPragma17
PPSmalltalkParserTests>>#testPragma18
tests pragmas with a keyword message.

add PPSmalltalkParser>>#buildSelector:
builds a selector from a array of RB tokens

for every given test add the keywordsPositions assertion.
i think it's needed because the PPSmalltalkClassesTests check the positions.

remove RBValueToken>>#inputValue (noone calls it?)


HTH,
--
Bernardo E.C.

Sent from a cheap desktop computer in South America.

_______________________________________________
Moose-dev mailing list
Moose-dev@iam.unibe.ch
https://www.iam.unibe.ch/mailman/listinfo/moose-dev




--
www.tudorgirba.com

"Every thing has its own flow"