
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"><html><head><meta content="text/html;charset=UTF-8" http-equiv="Content-Type"></head><body ><div style='font-size:10pt;font-family:Verdana,Arial,Helvetica,sans-serif;color:#00000;'><div><br></div><div><br></div><div class="zmail_extra"><div id="Zm-_Id_-Sgn1"><div>my applogies<br></div><div><br></div><div>I accidentally hit send with a control key while attempting to format the chain of reasoning.<br></div></div></div><div><br></div><div>What this test does is loop through each subclass of GRCodec and asks each one if they support the encoding 'mac-roman'<br></div><div><br></div><div><br></div><div>those classes are:<br></div><div><br></div><blockquote style="border: 1px solid rgb(204, 204, 204); padding: 7px; background-color: rgb(245, 245, 245);"><div><div>GRNullCodec<br></div><div>GRPharoGenericCodec<br></div><div>GRPharoUtf8Codec<br></div><div>GRPharoLatin1Codec<br></div></div></blockquote><div><div><br></div><div>The subclasses either hard-code their encodings in the class side supportedEncodingNames method or they define it as a subset of TextConverter allEncodingNames<br></div></div><div><br></div><div><br></div><blockquote style="border: 1px solid rgb(204, 204, 204); padding: 7px; background-color: rgb(245, 245, 245);"><div><div>TextConverter allEncodingNames inspect<br></div><div><br></div><div>TextConverter allEncodingNames select:[:element | element asString beginsWith:'mac']<br></div><div><br></div></div></blockquote><div>Like GRCodec, TextConverter loops through all its subclasses and asks them what to populate its set of names with.<br></div><div><br></div><div>The one we want is:<br></div><div><br></div><div class="zmail_extra"><div id="Zm-_Id_-Sgn1"><div>MacRomanTextConverter encodingNames<br></div><div><br></div><div>If you print that, you see two: #('macroman' 'mac-roman')<br></div><div><br></div><div>The issue, is in the TextConverte allEncodingNames method<br></div><div><br></div><div><blockquote style="border: 1px solid rgb(204, 204, 204); padding: 7px; background-color: rgb(245, 245, 245);"><div>allEncodingNames<br></div><div>"TextConverter allEncodingNames"<br></div><div>| encodingNames |<br></div><div>encodingNames := Set new.<br></div><div>self allSubclasses<br></div><div>do: [:each |<br></div><div>| names |<br></div><div>names := each encodingNames.<br></div><div>names notEmpty<br></div><div>ifTrue: [encodingNames add: names first asSymbol]].<br></div><div>^encodingNames<br></div></blockquote><br></div><div>looking at the ifTrue: block you see it only adds the FIRST name to the set.<br></div><div><br></div><div>This could be a bug in TextConverter.<br></div><div><br></div><div>Since 'mac-roman' and 'macroman' both point to the same MacRomanTextConverter, it stands to reason that changing the test from 'mac-roman' to 'macroman' is safe.<br></div><div><br></div><div>Do that, and the test passes.</div><div><br></div><div>cheers,<br></div><div><br></div><div>t</div><div><br></div><div><br></div><div>---- On Wed, 05 Dec 2018 08:33:02 -0500 <b>David T. Lewis <lewis@mail.msen.com></b> wrote ----<br></div></div><div><br></div><blockquote style="border-left: 1px solid #cccccc; padding-left: 6px; margin:0 0 0 5px"><div><div>Replying on the squeak-dev list to get some more eyes on the issue.<br></div><div><br></div><div>Dave<br></div><div><br></div><div>On Tue, Dec 04, 2018 at 10:17:17PM +0100, Johan Brichau wrote:<br></div><div>> Hi,<br></div><div>> <br></div><div>> Two tests are currently still failing for Squeak [1].<br></div><div>> - testNamedTempAt -> fails on travis but works in an interactive image??? dunno what???s going on but it???s an expected failure in Pharo for a long time as well???<br></div><div>> - testGenericCodecMacRoman -> it seems the preferential name for the Mac-Roman encoding became ???macroman??? in Squeak instead of ???mac-roman???. Not sure what to think about this one??? <br></div><div>> <br></div><div>> Anybody from the Squeak community who wants to step in and check on these?<br></div><div>> <br></div><div>> cheers<br></div><div>> Johan<br></div><div>> <br></div><div>> <br></div><div>> [1] <a target="_blank" href="https://travis-ci.org/SeasideSt/Seaside/jobs/463542906">https://travis-ci.org/SeasideSt/Seaside/jobs/463542906</a><br></div><div>> _______________________________________________<br></div><div>> seaside mailing list<br></div><div>> <a target="_blank" href="mailto:seaside@lists.squeakfoundation.org">seaside@lists.squeakfoundation.org</a><br></div><div>> <a target="_blank" href="http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside">http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside</a><br></div></div></blockquote></div><div><br></div></div><br></body></html>