pharo icon indicating copy to clipboard operation
pharo copied to clipboard

Review if #scanToken: in RBScanner is still needed

Open astares opened this issue 2 years ago • 0 comments

The comment in RBScanner(class)>>on: method recommends to use #scanTokens:

"Reader may consider scanTokens: as a better way to use the class."

image

but #scanToken: is only used in two tests:

image

I think we can deprecate and remove it and also remove the unnecessary part in the comment on RBScanner(class)>>on: then.

Side note: there is also a #scanToken: in MCScanner class as well, which I guess can get removed too

astares avatar Aug 18 '22 22:08 astares