Marcus Denker
Marcus Denker
I propose to do two steps: 1) fix the subclassResponsibility, yes, knowing that this is not good. But we do it for WideString already, and there are users 2) open...
PR is merged
Failing tests as a PR are a bit bad to handle... PRs are there to be integrated, while this one can not. Maybe just a fileout attached to the issue...
Interestingly it works if I used they shortcuts (cmd m). And *after that* it even works via the menu. Maybe the error leads to some initialisation not done correctly for...
This is not that easy, see discussion on the PR of the first try of what needs to be done: https://github.com/pharo-project/pharo/pull/10553
@svenvc: I looked at the Symbol interning code before, I found it confusing and a bit of a mess, it looks to be more complex than it needs to be,...
@MarcusDenker But: ``` 'ABC' = 'ABC' asWideString. "true" ``` Thus, the search for an existing symbol (using #like on the WeakSet) will find the symbol regardless if it looks for...
@svenvc I think we can try the change. I would add a specific test case and maybe a comment.
After merging https://github.com/pharo-project/pharo/pull/11291, the method where the check is done is WideString >> createSymbol
It seems that a lot of symbols are small... and would be perfectly ok to be immediate objects - no interning for most cases needed (identical by construction) - we...