Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scanner no longer included with Pharo (Pharo 7) #9

Open
cdavidshaffer opened this issue Jul 23, 2019 · 3 comments
Open

Scanner no longer included with Pharo (Pharo 7) #9

cdavidshaffer opened this issue Jul 23, 2019 · 3 comments

Comments

@cdavidshaffer
Copy link

The Scanner class has disappeared from Pharo some time at or before Pharo 7. I tried substituting RBScanner and MCScanner but neither seemed suitable for the job.

@cdavidshaffer
Copy link
Author

cdavidshaffer commented Jul 24, 2019

I have attached a parser (which uses PetitParser). I'm not sure if you want to introduce that dependency but I'm using your book for a course and needed a quick solution. This is the first time I've used PP so I can't attest to the quality of this parser.
PParser.txt
PParserTest.txt

@cdavidshaffer
Copy link
Author

cdavidshaffer commented Jul 24, 2019

I have moved this parser to its own package:

Metacello new
	baseline: 'PhsycheParser';
	repository: 'github://cdavidshaffer/phsyche-parser';
	load

The following change needs to be made to the code in the textbook if this parser is adopted:

Phsyche>>parse: aString

	^PParser new parse: aString

@Ducasse
Copy link
Member

Ducasse commented Jan 30, 2024

Thanks I'm just discovering your feedback.just 4 years later :(.
I will fix it during my next holidays.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants