Has someone seen something happening with Monticello Mcz handling extended characters ?

If I add a non-ascii character to MCMockClassA comment, such as �, then MCMczInstallerTest>>testInstallFromFile fails. If I add an ascii character instead to MCMockClassA comment, then this test pass.

(On 2.0 / 20610)

It's recent and has appeared between mid-june and now.

Thierry
--

Not sure you can call it recent, the same bug exists and is reproduceable by adding an � to MCMockClassA back to at least Pharo 1.0 :)

MCZInstaller >> #installMember: member

self useNewChangeSetDuring:

[ | str |str := member contentStream text.

str setConverterForCode.

CodeImporter evaluateReadStream: str readStream.

]

setConverterForCode has never been the right thing to do for reading non-ascii .st files written by Monticello, since it's tended to write the raw strings to disk without any regard to encoding.

For the comment you modified, that means it would have been written as latin1, and attempted to be read as MacRoman, where the code point of � is different.

In recent 3.0 versions, it's been fixed by Nicolas Cellier's changes to actually store/read .st files created by Monticello in utf8.

Cheers,

Henry